[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] Avoid swig.exe warnings on Unix release builds

From: David James <james82_at_gmail.com>
Date: 2005-08-30 01:00:30 CEST

On 8/29/05, Branko ╚ibej <brane@xbc.nu> wrote:> David James wrote:> > >On 8/29/05, Branko ╚ibej <brane@xbc.nu> wrote:> >> >> >>David James wrote:> >>> >>> >>>Index: gen_win.py> >>>===================================================================> >>>--- gen_win.py (revision 15951)> >>>+++ gen_win.py (working copy)> >>>@@ -812,9 +812,9 @@> >>> libdir = ''> >>>> >>> if self.swig_path is not None:> >>>- self.swig_exe = os.path.join(self.swig_path, 'swig.exe')> >>>+ self.swig_exe = os.path.join(self.swig_path, 'swig')> >>>> >>>> >>>> >>>> >>What's the point? If you configure --with-swig, that path gets embedded> >>in the .pdb/.vcproj files, so if you generate those on Unix, they'll> >>most likely not work on Windows where you have to build the stuff. So,> >>when building a Windows release tarball (zipball?), you can't use> >>--with-swig (or any of the other --with options, for that matter).> >>> >>> >I don't think there will be any Unix users who will use the> >--with-swig flag. But, since it i
s easy to make your code more> >portable, why not fix it? It is more consistent anyway.> >> >> O.K.> > >>> else:> >>>- self.swig_exe = 'swig.exe'> >>>+ self.swig_exe = 'swig'> >>>> >>>> >Without this fix, "./autogen.sh --release" fails on Unix.> >> >> I had no issues with _this_ part of the patch. :)> > -- Brane
Thanks Brane!
Committed in 15973.
Cheers,
David

-- David James -- http://www.cs.toronto.edu/~james
Received on Tue Aug 30 01:01:08 2005

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.