Re: [Cclan-commits] CVS: asdf asdf.lisp,1.89,1.90
Christophe Rhodes <[email protected]>
| Newsgroups | gmane.lisp.cclan.general |
|---|---|
| Message-ID | <[email protected]> |
Edi Weitz <[email protected]> writes: > Push :ASDF-SYSTEMNAME (Alan Shields) > > +(defmethod asdf:perform :after ((operation load-op) (s system)) > + "After a system is loaded by ASDF, push ASDF-SYSTEMNAME onto *features* > +This enables easy use of #+asdf-systemname and friends" > + (pushnew (intern (format nil "ASDF-~A" (string-upcase (component-name s))) :keyword) cl:*features*)) > + Oh no! I honestly don't think that this is a good idea. (I should have replied about this earlier, and for that lapse I apologise.) I claim that the use of *features* for functionality testing is almost always wrong, in that it does not have a fine enough granularity; it's usually better to test for the functionality you actually need. Cheers, Christophe ------------------------------------------------------- This SF.net email is sponsored by: Splunk Inc. Do you grep through log files for problems? Stop! Download the new AJAX search engine that makes searching your log files as easy as surfing the web. DOWNLOAD SPLUNK! http://ads.osdn.com/?ad_id=7637&alloc_id=16865&op=click