Hi Charles,

* Charles Wilson wrote on Thu, Sep 16, 2010 at 08:47:52PM CEST:
> [cygwin|mingw] Fix order of PATH manipulation in cwrapper
> 
> * libltdl/config/ltmain.m4sh (func_emit_cwrapperexe_src:main): Call
> lt_update_exe_path before lt_update_lib_path, to ensure that the local
> OBJDIR(s) supersedes any -rpath directories.
> Reported by Jon Turney <...>

This looks ok, but wouldn't the shell wrapper need this as well,
seeing that it could be run on w32 too (IIRC)?

Also, of course, testsuite exposure should follow eventually.

Thanks,
Ralf

> --- a/libltdl/config/ltmain.m4sh
> +++ b/libltdl/config/ltmain.m4sh
> @@ -3677,8 +3677,12 @@ EOF
> 
>    lt_setenv ("BIN_SH", "xpg4"); /* for Tru64 */
>    lt_setenv ("DUALCASE", "1");  /* for MSK sh */
> -  lt_update_lib_path (LIB_PATH_VARNAME, LIB_PATH_VALUE);
> +  /* For Windows, this order is important: it ensures that any -rpath
> +     values are prepended first, and then the local OBJDIR directory(ies)
> +     is prepended second -- ensuring that just-built libraries supersede
> +     installed ones. */
>    lt_update_exe_path (EXE_PATH_VARNAME, EXE_PATH_VALUE);
> +  lt_update_lib_path (LIB_PATH_VARNAME, LIB_PATH_VALUE);
> 
>    lt_debugprintf (__FILE__, __LINE__, "(main) lt_argv_zero: %s\n",
>                   nonnull (lt_argv_zero));

Reply via email to