Re: [PATCH] Add --lt-* options to shell wrapper

2010-03-15 Thread Peter Rosin
Hi Chuck, Den 2010-02-19 03:48 skrev Charles Wilson: 69: dlloader API FAILED (dlloader-api.at:416) *snip* 069: -- who knows, but it's somewhere in libltdl not libtool --- expout 2010-02-18 03:12:56.23160 -0500 +++

Re: [PATCH] Add --lt-* options to shell wrapper

2010-02-18 Thread Charles Wilson
Ralf Wildenhues wrote: See above for the two cases. With them fixed, I think the patch looks to be in fairly good shape, except that shell quoting bugs are really hard to detect when reading the doubly-quoted code. So please fix above, resend, and apply if you don't hear back after 72 hours.

Re: [PATCH] Add --lt-* options to shell wrapper

2010-02-17 Thread Charles Wilson
Ralf Wildenhues wrote: The option parsing in the original patch is overkill -- no need to re-quote things if all you're going to do is remove a couple of entries from $@, that can be done with set x $@ shift shift type handling. No, actually it can't. You're assuming that any

Re: [PATCH] Add --lt-* options to shell wrapper

2010-02-17 Thread Ralf Wildenhues
Hi Charles, * Charles Wilson wrote on Wed, Feb 17, 2010 at 03:30:47PM CET: Ralf Wildenhues wrote: The option parsing in the original patch is overkill -- no need to re-quote things if all you're going to do is remove a couple of entries from $@, that can be done with set x $@ shift

Re: [PATCH] Add --lt-* options to shell wrapper

2010-02-16 Thread Ralf Wildenhues
* Charles Wilson wrote on Sun, Feb 14, 2010 at 03:36:57PM CET: Charles Wilson wrote: Charles Wilson wrote: Charles Wilson wrote: This one, I think is OK for pre-2.2.8 -- what do you guys think? OK to push? In response to review comments over here: Re: [PATCH] Enable runtime cwrapper

Re: [PATCH] Add --lt-* options to shell wrapper

2010-01-29 Thread Charles Wilson
Charles Wilson wrote: Charles Wilson wrote: This one, I think is OK for pre-2.2.8 -- what do you guys think? OK to push? In response to review comments over here: Re: [PATCH] Enable runtime cwrapper debugging; add tests

Re: [PATCH] Add --lt-* options to shell wrapper

2010-01-15 Thread Charles Wilson
Charles Wilson wrote: This one, I think is OK for pre-2.2.8 -- what do you guys think? OK to push? In response to review comments over here: Re: [PATCH] Enable runtime cwrapper debugging; add tests http://lists.gnu.org/archive/html/libtool-patches/2009-12/msg00014.html I've created a followon

Re: [PATCH] Add --lt-* options to shell wrapper

2009-12-12 Thread Charles Wilson
Charles Wilson wrote: * libltdl/config/ltmain.m4sh (func_emit_wrapper) [func_parse_lt_options]: New function. [func_opts_contain_lt]: New function. [func_lt_dump_args]: New function. [func_exec_program_core]: New function, refactored from [main]. [func_exec_program]: New function. [main]:

[PATCH] Add --lt-* options to shell wrapper

2009-07-12 Thread Charles Wilson
: 75765d58c38c55ef069ef100732c1e978f4151a6 2. [PATCH] Enable runtime cwrapper debugging; add tests http://lists.gnu.org/archive/html/libtool-patches/2009-06/msg00039.html pending... 3. [PATCH] Add --lt-* options to shell wrapper (this post. obviously no URL yet, and obviously pending...) 4. [PATCH] Document wrapper changes (next post