[Top][All Lists]
[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.
define-quoting.diff
Description: Text document