Re: [PATCH] [cygwin|mingw]: Add cross-compile support to cwrapper (take 6)

2010-08-27 Thread Charles Wilson
On 8/27/2010 3:47 PM, Ralf Wildenhues wrote: > * Charles Wilson wrote on Fri, Aug 27, 2010 at 08:49:24PM CEST: >> As I said, I don't mind rebasing. I'll probably only test the result on >> cygwin-native, mingw-native, and linux->mingw, and then push to master. > > Oh that's plenty fine. > >> Ho

git and branches (was: [PATCH] [cygwin|mingw]: Add cross-compile support to cwrapper (take 6))

2010-08-27 Thread Ralf Wildenhues
Maybe a bit more explanation: * Ralf Wildenhues wrote on Sat, Aug 28, 2010 at 07:22:42AM CEST: > * Charles Wilson wrote on Sat, Aug 28, 2010 at 07:10:25AM CEST: > > On 8/27/2010 3:47 PM, Ralf Wildenhues wrote: > > > * Charles Wilson wrote on Fri, Aug 27, 2010 at 08:49:24PM CEST: > > >> However, on

Re: [PATCH] [cygwin|mingw]: Add cross-compile support to cwrapper (take 6)

2010-08-27 Thread Ralf Wildenhues
Hi Charles, * Charles Wilson wrote on Sat, Aug 28, 2010 at 07:10:25AM CEST: > On 8/27/2010 3:47 PM, Ralf Wildenhues wrote: > > * Charles Wilson wrote on Fri, Aug 27, 2010 at 08:49:24PM CEST: > >> However, once I have finished the requested changes above and the > >> rebasing (plus whatever comes o

Re: [PATCH] [cygwin|mingw]: Add cross-compile support to cwrapper (take 6)

2010-08-27 Thread Charles Wilson
On 8/27/2010 3:47 PM, Ralf Wildenhues wrote: > * Charles Wilson wrote on Fri, Aug 27, 2010 at 08:49:24PM CEST: >> However, once I have finished the requested changes above and the >> rebasing (plus whatever comes of the four open ***QQQ***uestions), I >> might ask for a 12 hour halt on updates to m

Re: [PATCH] [cygwin|mingw]: Add cross-compile support to cwrapper (take 6)

2010-08-27 Thread Charles Wilson
On 8/27/2010 5:48 PM, Ralf Wildenhues wrote: > * Charles Wilson wrote on Fri, Aug 27, 2010 at 10:23:31PM CEST: >> Original: >> real25m3.886s >> user6m24.620s >> sys 11m13.787s >> >> With the functions moved ahead of func_mode_compile: >> real24m34.235s >> user6m30.590s >> sys

Re: [PATCH] [cygwin|mingw]: Add cross-compile support to cwrapper (take 6)

2010-08-27 Thread Ralf Wildenhues
I'm ok with everything in this.

Re: [PATCH] [cygwin|mingw]: Add cross-compile support to cwrapper (take 6)

2010-08-27 Thread Ralf Wildenhues
* Charles Wilson wrote on Fri, Aug 27, 2010 at 10:23:31PM CEST: > Original: > real25m3.886s > user6m24.620s > sys 11m13.787s > > With the functions moved ahead of func_mode_compile: > real24m34.235s > user6m30.590s > sys 11m23.878s > > So, when compiling about 325 source

Re: [PATCH] [cygwin|mingw]: Add cross-compile support to cwrapper (take 6)

2010-08-27 Thread Charles Wilson
On 8/27/2010 2:01 PM, Ralf Wildenhues wrote: > * Charles Wilson wrote on Fri, Aug 27, 2010 at 06:35:31PM CEST: >> Now, this is a little disturbing, since my -dlpreopen was supposed to >> have fixed this; > > Would be good to fix, yes. On *nix, I'd be suggesting git bisect now, > no idea if you ca

Re: [PATCH] [cygwin|mingw]: Add cross-compile support to cwrapper (take 6)

2010-08-27 Thread Charles Wilson
On 8/27/2010 12:54 AM, Ralf Wildenhues wrote: > * Charles Wilson wrote on Thu, Aug 26, 2010 at 11:20:48PM CEST: >> Also: I've said this before, but we can't use the m4 >> function_replace magic because we need to retain the ability for >> users to override the choice of $func_to_host_path_cmd. Thi

Re: [PATCH] [cygwin|mingw]: Add cross-compile support to cwrapper (take 6)

2010-08-27 Thread Charles Wilson
On 8/26/2010 5:20 PM, Charles Wilson wrote: > On 8/26/2010 4:18 PM, Ralf Wildenhues wrote: >> Then, please just move the new functions where Peter needs them, >> if they really need moving, that is. > > I deliberately placed them after func_compile and before func_link, for > speed-of-parsing reas

Re: [PATCH] [cygwin|mingw]: Add cross-compile support to cwrapper (take 6)

2010-08-27 Thread Ralf Wildenhues
* Charles Wilson wrote on Fri, Aug 27, 2010 at 08:49:24PM CEST: > On 8/26/2010 4:18 PM, Ralf Wildenhues wrote: > >> + lt_cv_to_host_path_cmd="$(to_host_path_cmd)" > > > > I don't understand why this change should be necessary. In your > > testing, you describe that most setups set a correct to_h

Re: [PATCH] [cygwin|mingw]: Add cross-compile support to cwrapper (take 6)

2010-08-27 Thread Charles Wilson
On 8/26/2010 4:18 PM, Ralf Wildenhues wrote: >> --- a/Makefile.am >> +++ b/Makefile.am >> @@ -528,7 +528,8 @@ TESTS_ENVIRONMENT = MAKE="$(MAKE)" CC="$(CC)" >> CFLAGS="$(CFLAGS)" \ >> CXX="$(CXX)" CXXFLAGS="$(CXXFLAGS)" CXXCPP="$(CXXCPP)" \ >> F77="$(F77)" FFLAGS="$(FFLAGS)" \ >> FC=

Re: [PATCH] [cygwin|mingw]: Add cross-compile support to cwrapper (take 6)

2010-08-27 Thread Ralf Wildenhues
* Charles Wilson wrote on Fri, Aug 27, 2010 at 08:17:15PM CEST: > On 8/27/2010 2:10 PM, Ralf Wildenhues wrote: > > Another questions regarding this patch: > > > > Do you know whether all of the conversion functions are idempotent > > (f(f(x)) = f(x))? IOW, when the user passes names already conve

Re: [PATCH] [cygwin|mingw]: Add cross-compile support to cwrapper (take 6)

2010-08-27 Thread Charles Wilson
On 8/27/2010 2:10 PM, Ralf Wildenhues wrote: > Another questions regarding this patch: > > Do you know whether all of the conversion functions are idempotent > (f(f(x)) = f(x))? IOW, when the user passes names already converted, > do we do the right things in all cases? I'm pretty sure they are

Re: [PATCH] [cygwin|mingw]: Add cross-compile support to cwrapper (take 6)

2010-08-27 Thread Ralf Wildenhues
Another questions regarding this patch: Do you know whether all of the conversion functions are idempotent (f(f(x)) = f(x))? IOW, when the user passes names already converted, do we do the right things in all cases? Thanks, Ralf

Re: [PATCH] [cygwin|mingw]: Add cross-compile support to cwrapper (take 6)

2010-08-27 Thread Ralf Wildenhues
* Charles Wilson wrote on Fri, Aug 27, 2010 at 06:35:31PM CEST: > It appears the problem is that the mdemo{2}_static.exe tests don't work, > when mdemo{2}-conf.test is used. They DO work when mdemo-static.test is > used (there is no mdemo2-static.test). > > Now, this is a little disturbing, since

Re: [PATCH] [cygwin|mingw]: Add cross-compile support to cwrapper (take 6)

2010-08-27 Thread Peter Rosin
Den 2010-08-27 08:25 skrev Peter Rosin: > Den 2010-08-27 00:27 skrev Roumen Petrov: >> Lets wine is correctly configured (Z: drive is linked to the file system >> root): >> $ cd $WINEPREFIX/dosdevices >> $ winepath -w `pwd` >> Z:\%WINEPREFIX_CONVERTED_TO_BACKSLASHES%\dosdevices >> >> Now lets remo