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 o

Re: Speed up progname, progpath setting

2008-04-10 Thread Bob Friesenhahn
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. Using the ZSH shell: % echo `/bin/pwd` /scratch/bfriesen/build/GraphicsMagick-1.1-16-static % echo `p

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?

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 standard

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: 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 declar

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

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 , whereas mingw declares _setmode in . Even though is not standard, my u

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 be

Re: No need for object file removal before compilation

2008-04-10 Thread Bob Friesenhahn
On Thu, 10 Apr 2008, Peter O'Gorman wrote: Removing the object file in advance accomplishes several things: But, if libtool were not sitting there between make and the compiler, the compiler would not remove the object files. Why should libtool do so? Some compilers unlink the output file wh

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 func_m

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]> (chan

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 advan