Re: Yet another patch (dmd.py this time)

Bram Moolenaar wrote:
> Lars Ivar Igesund wrote:
> 
> 
>>Cleanup time. Seems like I did some stuff more complicated
>>than necessary. Also, I've forgot to add the contents of
>>$LIBS to the build command line ...
>>Interspersed is also one or two bugfixes.
>>
>>So now I believe it's perfect. :)
> 
> 
> Just checking:
> 
> 
>>+            ":sys $DMD -L$*?DLINKFLAGS -L+$*?LIBS -of$target $source")
> 
> 
> This means that you get a "-L" when $DLINKFLAGS is empty.  Isn't that a
> problem?  Same for -L+$?LIBS, you would get "-L+" when $LIBS is empty.
> 

Hmm, it works as it is(-L just passes what follows to the linker, in
this case an empty string), but I would expect that the RC stuff would
result in nothing when '?' is used and the variable is empty. That this
is not the case, seems inconsistent to me.

I will submit another patch if needed.
Lars Ivar Igesund



-------------------------------------------------------
SF.Net is sponsored by: Speed Start Your Linux Apps Now.
Build and deploy apps & Web services for Linux with
a free DVD software kit from IBM. Click Now!
http://ads.osdn.com/?ad_id=1356&alloc_id=3438&op=click