<div dir="auto">Understood and agreed.  I will try to make a better patch in the next week or two.<br><div class="gmail_extra" dir="auto"><br><div class="gmail_quote">On Mar 24, 2017 8:46 AM, "Marc Feeley" <<a href="mailto:feeley@iro.umontreal.ca" target="_blank">feeley@iro.umontreal.ca</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I would prefer if the patch was integrated into Gambit.  My only worry is the use of cygpath here:<br>
<br>
case "$target_os" in<br>
    mingw*) # set gambitdir to use a native windows path<br>
      GAMBITDIR=`cygpath -d $prefix  | tr '\\' '/'`<br>
    *) # otherwise just use the prefix<br>
      GAMBITDIR=$prefix<br>
esac<br>
<br>
cygpath doesn’t exist in mingw/msys.  It only exists if cygwin is installed but the case selects “mingw*” which seems wrong.<br>
<br>
Marc<br>
<br>
<br>
<br>
> On Mar 23, 2017, at 9:23 PM, Bill Six <<a href="mailto:billsix@gmail.com" target="_blank">billsix@gmail.com</a>> wrote:<br>
><br>
> By the way, I don't mind maintaining this patch outside of Gambit, just thought I'd share it here in the case that it's useful.<br>
><br>
> For context, I'm looking to get updated Gambit versions into package managers, for ease of installation/upgrades.  I updated Gambit to 4.8.8 in Gentoo Linux. <a href="https://gitweb.gentoo.org/repo/gentoo.git/commit/dev-scheme/gambit?id=bd6727aa850ed00d5c16c448a7a50114b55b86b1" rel="noreferrer" target="_blank">https://gitweb.gentoo.org/repo<wbr>/gentoo.git/commit/dev-scheme/<wbr>gambit?id=bd6727aa850ed00d5c16<wbr>c448a7a50114b55b86b1</a><br>
><br>
> I'm now looking to add a new Gambit package to MSys2 (<a href="http://www.msys2.org/" rel="noreferrer" target="_blank">http://www.msys2.org/</a>), a fork of cygwin, built on Arch Linux's "pacman" manager.  A few months ago, I had issues with using "./configure --prefix=$FOO", because with msys2,  "$FOO" had to be in the "unix" format (e.g. /usr), but Gambit needs "$FOO"  to be a path in native Windows format, as file I/O on that path uses native windows routines <a href="http://webmail.iro.umontreal.ca/pipermail/gambit-list/2016-October/008413.html" rel="noreferrer" target="_blank">http://webmail.iro.umontreal.c<wbr>a/pipermail/gambit-list/2016-O<wbr>ctober/008413.html</a><br>
><br>
> So my patch, on _any_ mingw system (which might be too broad), changes to ___GAMBITDIR variable to a native windows path, with forward slashes instead of backslashes, while not effecting the --prefix variable at all<br>
><br>
> I plan to get a build of gambit 4.8.8 with my patch into MSys2 somewhat soon.  If my patch is useful to you, great, if not, I'll make a patch as part of the MSys2 build of Gambit.<br>
><br>
> Thanks,<br>
> Bill<br>
><br>
><br>
> On Thu, Mar 23, 2017 at 8:48 PM, Marc Feeley <<a href="mailto:feeley@iro.umontreal.ca" target="_blank">feeley@iro.umontreal.ca</a>> wrote:<br>
> Can you explain what this does?  I don’t like relying on “cygpath” which I don’t think exists on plain msys.<br>
><br>
> Marc<br>
><br>
><br>
><br>
> > On Mar 23, 2017, at 8:42 PM, Bill Six <<a href="mailto:billsix@gmail.com" target="_blank">billsix@gmail.com</a>> wrote:<br>
> ><br>
> > Marc,<br>
> ><br>
> > I patched <a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a> so that Gambit builds and works correctly on msys2 on windows.<br>
> ><br>
> > Bill<br>
> > <0001-build-on-msys2-windows-b<wbr>y-using-the-native-windows-p.p<wbr>atch>_________________________<wbr>______________________<br>
> > Gambit-list mailing list<br>
> > <a href="mailto:Gambit-list@iro.umontreal.ca" target="_blank">Gambit-list@iro.umontreal.ca</a><br>
> > <a href="https://webmail.iro.umontreal.ca/mailman/listinfo/gambit-list" rel="noreferrer" target="_blank">https://webmail.iro.umontreal.<wbr>ca/mailman/listinfo/gambit-lis<wbr>t</a><br>
><br>
><br><br></blockquote></div></div>
</div>