Hackable Mercurial: linker_exe and no-cygwin

[Matt Mackall]

On Mon, 2012-04-23 at 22:30 +0300, Eduard-Cristian Stefan wrote:
> On 2012-04-23 21:58, Matt Mackall wrote:
> > Here are the changes I made:
> >
> > @@ -319,11 +319,11 @@
> >           else:
> >               entry_point = ''
> >
> > -        self.set_executables(compiler='gcc -mno-cygwin -O -Wall',
> > -                             compiler_so='gcc -mno-cygwin -mdll -O -Wall',
> > -                             compiler_cxx='g++ -mno-cygwin -O -Wall',
> > +        self.set_executables(compiler='gcc -O -Wall',
> > +                             compiler_so='gcc -mdll -O -Wall',
> > +                             compiler_cxx='g++ -O -Wall',
> >                                linker_exe='gcc -mno-cygwin',
> > -                             linker_so='%s -mno-cygwin %s %s'
> > +                             linker_so='%s %s %s'
> >                                           % (self.linker_dll, shared_option,
> >                                              entry_point))
> >
> >
> > One goal I have is to release a script to build hackable-mercurial from
> > scratch by downloading the components and applying any necessary tweaks.
> > The current lines in the build script look like this:
> >
> >      wget -c http://www.python.org/ftp/python/2.6.6/python-2.6.6.msi
> >      wine msiexec /i python-2.6*.msi /qb
> >      # fix gcc breakage (gcc, you suck at backward compat)
> >      sed -i "s/-mno-cygwin //" ../Python26/Lib/distutils/cygwinccompiler.py
> >
> 
> Maybe you should search for " -mno-cygwin" instead of "-mno-cygwin ",
> that way the linker_exe line would be catched.
> Or simply search for "-mno-cygwin", an extra space should be harmless here.

Seems like a good idea.

-- 
Mathematics is the supreme nostalgia of our time.