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

Reply via email to