Hi Pavel, On Dec 13, 2014, at 5:58 PM, Pavel Raiskup <prais...@redhat.com> wrote: > > On Friday 12 of December 2014 11:17:03 Gary V. Vaughan wrote: >> I'll commit a follow on patch, to tweak it like this, later today. > > Thanks for the patch! It is almost perfect. During testing I noted that > there is still one dollar sign not substituted with quadrigraph. Patch > 0001 attached. > > Also, I noted having LT_SYS_SEARCH_PATH="/lib64:" specified in config.site > does not change libtool's content; it the LT_SYS_LIBRARY_PATH default > value stays empty. I would like to have it fixed before 2.4.5, if > possible, because I would like to enable something like the following in > our default /usr/share/config.site file: > > if $arch_64bit; then > : ${LT_SYS_LIBRARY_PATH="/lib64:/usr/lib64:"} > fi > > Also, I'm not sure whether we should touch the lt_cv_sys_*. That should > not hurt too much but I would feel quite more safe if we use separate > variable and let the lt_cv_* for existing workarounds in the wild. > Something like the attachment 0002 would be nice to have pushed.
Thanks for the patches. I tweaked them to pass `make syntax-check`, applied and pushed both. > That code seems like asking for test-case also? The more the better! Cheers, -- Gary V. Vaughan (gary AT gnu DOT org) _______________________________________________ https://lists.gnu.org/mailman/listinfo/libtool