libtool-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

FYI: Fixlet for missing braces around sys_search_path


From: Ralf Wildenhues
Subject: FYI: Fixlet for missing braces around sys_search_path
Date: Fri, 29 Apr 2005 10:56:18 +0200
User-agent: Mutt/1.4.1i

Hi Dalibor,

* Dalibor Topic wrote on Thu, Apr 28, 2005 at 06:20:34PM CEST:
> 
> here is another patch from Kaffe's libtool patch chest, which adds the 
> missing braces around $sys_search_path in the definition of 
> LTDL_SYSSEARCHPATH. That fixes libtool for at least one platform, though 
> I don't recall which any more :(

That does not matter.  I hate inconsistent macro option quoting, one
should do it even for the trivial cases just to get used to it, because
the times when an option should _not_ be quoted are very much exceptions
and should IMNSHO be clearly documented so.

I will send a patch regarding Autoconf documentation to the
autoconf-patches list, and have applied the attach patches to all three
libtool branches, which change all such invocations.

Many thanks for reporting all these bugs.  Are there any other
libtool-related patches you have in kaffe?  (The static *BSD dlopen bug
is still open, I don't know yet how to fix it.)

Regards,
Ralf

> 2004-03-12  Riccardo Mottola <address@hidden>,
>             Michael Koch  <address@hidden>
> 
>         * libltdl/acinclude.m4 (LTDL_SYSSEARCHPATH): Brace
>       $sys_search_path argument.

Attachment: define-quoting.diff
Description: Text document


reply via email to

[Prev in Thread] Current Thread [Next in Thread]