[asdf-devel] suggestion for aserve.asd
rudi at constantly.at
Thu Dec 20 19:17:15 UTC 2012
On Dec 20, 2012, at 03:21, Robert Goldman <rpgoldman at sift.info> wrote:
> On 12/19/12 Dec 19 -8:04 PM, Faré wrote:
>> On Wed, Dec 19, 2012 at 4:57 PM, Robert Goldman <rpgoldman at sift.info> wrote:
>>> You don't need acl-file any more.
>>> You can now use cl-source-file.cl, which is exported from ASDF.
>>> You can make legacy-acl-source-file inherit from cl-source-file.cl and
>>> cut the ACL-FILE class entirely. That will take care of using the .cl
>>> extension instead of .lisp.
>> Actually, an acl-file is still required, because there's a method on perform.
>> On the other hand, that method needs to be changed
>> to play well with asdf-output-translations,
>> and could simply be an :around method to
>> (handler-bind (((or style-warning warning) #'muffle-warning))
>> or something (untested).
> Actually, portable aserve has two layers. It has an acl-file, and it
> has legacy-acl-source-file. I believe that the former is unnecessary,
> but the warning muffler on legacy-acl-source-file is still needed.
> On general principles, I don't like the idea that there are warnings
> that need muffling: I think any mature system should build without
> warnings. If you muffle some warnings that need muffling, sooner or
> later you always end up muffling a NEW warning that you wish you hadn't
Agreed on both counts, but I prefer minimally invasive surgery, especially in an old and creaky system like paserve. Anyway, the suggested changes are committed now, which will hopefully make Fare's job a bit easier.
More information about the asdf-devel