libtool-patches
[Top][All Lists]
Advanced

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

Re: [PATCH 3/4] Uniform const'ness of symlist variable lt_preloaded_symb


From: Ralf Wildenhues
Subject: Re: [PATCH 3/4] Uniform const'ness of symlist variable lt_preloaded_symbols.
Date: Sat, 28 Aug 2010 15:21:50 +0200
User-agent: Mutt/1.5.20 (2010-04-22)

Since the question asked in:

* Ralf Wildenhues wrote on Tue, Jun 29, 2010 at 06:49:39AM CEST:
> > <http://thread.gmane.org/gmane.comp.gnu.libtool.general/10794/focus=9769>.

was now answered in

<http://thread.gmane.org/gmane.comp.gnu.libtool.general/11039/focus=11045>

I feel confident enough for the patch quoted in the first link, and am
pushing it after retesting on GNU/Linux.

Cheers,
Ralf

    Uniform const'ness of symlist variable lt_preloaded_symbols.
    
    On some systems, lt_preloaded_symbols may not be declared
    const due to relocation issues.  C99 requires qualification to
    match for compatible declarations, so ensure we declare
    const'ness in the same way everywhere; link-time optimization
    may otherwise rightfully complain about inconsistencies.
    Since ltdl.h may not depend upon config.h, rely on system
    defines for choosing const'ness.
    
    * NEWS: Update.
    * doc/libtool.texi (Dlpreopening): Document this issue.
    * libltdl/config/ltmain.m4sh (func_generate_dlsyms):
    [__WINDOWS__, __CYGWIN__, _WIN32_WCE]: Define LT_DLSYM_CONST
    according to system defines.  Remove old shell cruft.  Use
    LT_DLSYM_CONST for lt_${my_prefix}_LTX_preloaded_symbols.
    * libltdl/libltdl/lt_system.h (LT_DLSYM_CONST): Likewise,
    define LT_DLSYM_CONST according to system defines.
    * tests/demo/foo.h (LT_DLSYM_CONST): Likewise.
    * tests/pdemo/foo.h (LT_DLSYM_CONST): Likewise.
    * tests/demo/dlmain.c (lt_dlsymlist): Use LT_DLSYM_CONST.
    * tests/pdemo/longer_file_name_dlmain.c (lt_dlsymlist):
    Likewise.
    * libltdl/m4/libtool.m4 (_LT_CMD_GLOBAL_SYMBOLS): Likewise
    for configure test code.
    * libltdl/ltdl.c (preloaded_symbols): Use LT_DLSYM_CONST.
    * libltdl/ltdl.h (LTDL_SET_PRELOADED_SYMBOLS): Likewise.
[...]



reply via email to

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