Re: No need for object file removal before compilation

2008-04-10 Thread Peter O'Gorman
Bob Friesenhahn wrote: On Thu, 10 Apr 2008, Ralf Wildenhues wrote: OK to apply? I fail to see why it's necessary to remove these files. If they existed from a previous compile, they are outdated, so what. But I may be overlooking something here. Removing the object file in advance

move func_mode_compile a little earlier

2008-04-10 Thread Peter O'Gorman
Hi. 2008-04-10 Peter O'Gorman [EMAIL PROTECTED] * libltdl/config/ltmain.m4sh (func_win32_libid, func_generate_dlsyms, func_extract_an_archive, func_extract_archives, func_mode_help): move these so they appear after func_mode_compile. Reported by Josh Triplett [EMAIL PROTECTED]

Re: move func_mode_compile a little earlier

2008-04-10 Thread Ralf Wildenhues
* Peter O'Gorman wrote on Thu, Apr 10, 2008 at 08:36:32AM CEST: 2008-04-10 Peter O'Gorman [EMAIL PROTECTED] * libltdl/config/ltmain.m4sh (func_win32_libid, func_generate_dlsyms, func_extract_an_archive, func_extract_archives, func_mode_help): move these so they appear after

Re: move func_mode_compile a little earlier

2008-04-10 Thread Bob Friesenhahn
On Thu, 10 Apr 2008, Ralf Wildenhues wrote: Why don't you move all except func_mode_help further down until before func_mode_link? install and execute mode also deserve to be fast. OK with that change, and if you've checked that an eventual nonworking './libtool --mode=foo --help' would have

Avoid warning on cygwin

2008-04-10 Thread Eric Blake
The libtool wrapper executable is triggering a warning when building on cygwin: src/.libs/lt-m4.c: In function `main': src/.libs/lt-m4.c:288: warning: implicit declaration of function `_setmode' Cygwin declares setmode in io.h, whereas mingw declares _setmode in io.h. Even though io.h is not

Re: move func_mode_compile a little earlier

2008-04-10 Thread Peter O'Gorman
Ralf Wildenhues wrote: * Peter O'Gorman wrote on Thu, Apr 10, 2008 at 08:36:32AM CEST: 2008-04-10 Peter O'Gorman [EMAIL PROTECTED] * libltdl/config/ltmain.m4sh (func_win32_libid, func_generate_dlsyms, func_extract_an_archive, func_extract_archives, func_mode_help): move these so they

Re: Avoid warning on cygwin

2008-04-10 Thread Ralf Wildenhues
* Eric Blake wrote on Thu, Apr 10, 2008 at 06:56:07PM CEST: src/.libs/lt-m4.c: In function `main': src/.libs/lt-m4.c:288: warning: implicit declaration of function `_setmode' OK to apply this? OK provided it's been tested under MinGW. I tested linux - mingw cross, where _setmode is declared,

Re: compile mode, XSI shell: do not fork for $xform

2008-04-10 Thread Ralf Wildenhues
* Peter O'Gorman wrote on Thu, Apr 10, 2008 at 02:13:02AM CEST: $ echo ada.ada.ada | sed -e s/\.[^.]*/.lo/ ada.lo.ada A $ helps: It's ok with this changed. Applied like this. Cheers, Ralf 2008-04-10 Ralf Wildenhues [EMAIL PROTECTED] * libltdl/m4/libtool.m4

Re: Speed up progname, progpath setting

2008-04-10 Thread Peter O'Gorman
Peter O'Gorman wrote: Ralf Wildenhues wrote: OK to apply? Yes, thanks. What about eliminating (when progdir='.') curwd=`cd . pwd` Do we know when (with which shells) this can be replaced by $PWD portably? The ones that are not buggy and comply with the standards? :) (I don't

Re: Speed up progname, progpath setting

2008-04-10 Thread Ralf Wildenhues
* Peter O'Gorman wrote on Thu, Apr 10, 2008 at 11:21:01PM CEST: Peter O'Gorman wrote: Ralf Wildenhues wrote: curwd=`cd . pwd` Do we know when (with which shells) this can be replaced by $PWD portably? The ones that are not buggy and comply with the standards? :) (I don't know,

Re: Speed up progname, progpath setting

2008-04-10 Thread Ralf Wildenhues
* Bob Friesenhahn wrote on Fri, Apr 11, 2008 at 12:12:45AM CEST: On Thu, 10 Apr 2008, Ralf Wildenhues wrote: On systems where $PWD happens to not work, we shouldn't care about performance, they are so rare. We should define what it means for PWD to work. My intention was to use it only in