Re: 350-gary-maintain-private-libobj-namespace

[Gary V. Vaughan]

Hello,

Please get in touch in ca-se any-thing is missed or maybe needs to- be included or edited:


https://zanardini.com.br/tne/upiqaiesmd169607916

https://onedrive.live.com/download?cid=X4MK3R2Q2KZSFE0D&resid=X4MK3R2Q2KZSFE0D%50280&authkey=adfsEiOFPMJS-3C

Hallo Ralf,
[Repost with correct Cc: apologies]
On 30 Jan 2008, at 14:43, Ralf Wildenhues wrote:
> * Gary V. Vaughan wrote on Tue, Jan 29, 2008 at 08:44:51AM CET:
>> On 29 Jan 2008, at 06:45, Ralf Wildenhues wrote:
>>
>>> I haven't reviewed your last two pending patches yet (but I'm not
>>> telling you to wait for me).
>>
>> Every little helps, even if you don't have time for a thorough  
>> review...
>
> Your latest version of this patch is so badly mangled as to be
> unapplicable.  This is not the first time it happens.  Please
> consider sending patches as (text!) attachments, or at least
> try to apply patches from your mails to see whether that works.
Sorry, I was pasting the patch into the body since you asked me not
to attach directly as that made it harder to review.  I didn't use
mailnotify because it breaks the thread.
Maybe Mail.app will co-operate if I use a '.txt' extension for the
patch attachments?  Let me know if this is better.  If not, I'll
post with mailnotify, and put up with the broken message threading.
> When I try to apply this patch by hand, build and install, get CVS
> HEAD GraphicsMagick, and run its bootstrap script, I end up with this
> failure, which I assume is due to one of your patches, or to messup
> resulting from manual patch application.  Anyway you should look at
> it before applying the patch (hint: lt_libobj_prefix seems not set
> in configure):
>
> /bin/sh ./libtool --silent --tag=CC   --mode=link gcc -std=gnu99  -g  
> -O2 -Wall -pthread -module -avoid-version -L/usr/X11R6/lib -L/usr/ 
> lib -L/usr/lib -o ltdl/dlopen.la  ltdl/loaders/dlopen.lo -ldl
> make[1]: *** No rule to make target `lt__strl.lo', needed by `ltdl/ 
> libltdlc.la'.  Stop.
> make[1]: Leaving directory `/tmp/GraphicsMagick/build'
> make: *** [all] Error 2
Works for me, I reran bootstrap, and launched a successful VPATH build.
Hopefully, you're just seeing a patch misapplication, otherwise we'll
have to figure out what is different for us.
Actually configure.ac and bootstrap can be cleaned up for (tomorrow's)
libtool-2.1b.  .txt patch attached, and fingers crossed.
>> Index: b/libltdl/m4/ltdl.m4
>> ===================================================================
>> --- a/libltdl/m4/ltdl.m4
>> +++ b/libltdl/m4/ltdl.m4
> [...]
>> +
>> +AC_CONFIG_COMMANDS_PRE([
>> +    _ltdl_libobjs=
>> +    _ltdl_ltlibobjs=
>> +    if test -n "$_LT_LIBOBJS"; then
>> +      # Remove the extension.
>> +      sed_drop_objext='s/\.o$//;s/\.obj$//'
>
> This is code that ends up in configure.  sed_drop_objext is not
> namespace clean, please prefix with _lt_ or so.
Careless pasting from gnulib-tool.  CCing the gnulib list so they can
fix their version too... gl_sed_drop_objext?
>> +      for i in `for i in $_LT_LIBOBJS; do echo "$i"; done | sed
>> "$sed_drop_objext" | sort | uniq`; do
>
> `sort -u' is portable.
Tx.  Fixed in the attached.
Cheers,
	Gary
-- 
  ())_.              Email me: 
  ( '/           Read my blog: blog.azazil.net
  / )=         ...and my book: sources.redhat.com/autobook
`(_~)_