{"thread":{"id":"23253","subject":"Are the patches used to build git on cygwin available in a git repo somewhere?","startedAt":"2010-03-30T16:16:56Z","lastAt":"2010-04-01T23:41:32Z","messageCount":9,"participants":["Jon Seymour","Jonathan Nieder","Eric Blake"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"138189","messageId":"2cfc40321003300916o305e4c15x7850549beafc26c0@mail.gmail.com","threadId":"23253","inReplyTo":null,"subject":"Are the patches used to build git on cygwin available in a git repo somewhere?","fromName":"Jon Seymour","fromEmail":"jon.seymour@gmail.com","sentAt":"2010-03-30T16:16:56Z","receivedAt":"2010-03-30T16:16:56Z","isPatch":false,"sender":{"key":"jon.seymour@gmail.com","avatar":"https://avatars.githubusercontent.com/u/207131?v=4"},"body":"I notice that certain tools, such as gitk, won't run correctly if git\nis built from the master branch.\n\nDoes anyone maintain a git repo with the patches used to successfully\nbuild the cygwin git packages?\n\njon.\n"},{"id":"138251","messageId":"20100331025626.GA13501@progeny.tock","threadId":"23253","inReplyTo":"2cfc40321003300916o305e4c15x7850549beafc26c0@mail.gmail.com","subject":"Re: Are the patches used to build git on cygwin available in a git repo somewhere?","fromName":"Jonathan Nieder","fromEmail":"jrnieder@gmail.com","sentAt":"2010-03-31T02:56:27Z","receivedAt":"2010-03-31T02:56:27Z","isPatch":false,"sender":{"key":"jrnieder@gmail.com","avatar":"https://avatars.githubusercontent.com/u/281595?v=4"},"body":"[adding a few relevant cc's]\n\nJon Seymour wrote:\n\n> Does anyone maintain a git repo with the patches used to successfully\n> build the cygwin git packages?\n\nI think git.git is the canonical repo. On cygwin.com I don’t see any\npackage more recent than 1.6.6.1 so I suspect no one has packaged a\nrecent version yet.\n\nI assume reports and patches would be welcome.\n\nHTH,\nJonathan\n"},{"id":"138252","messageId":"20100331025856.GB13501@progeny.tock","threadId":"23253","inReplyTo":"20100331025626.GA13501@progeny.tock","subject":"Re: Are the patches used to build git on cygwin available in a git repo somewhere?","fromName":"Jonathan Nieder","fromEmail":"jrnieder@gmail.com","sentAt":"2010-03-31T02:58:56Z","receivedAt":"2010-03-31T02:58:56Z","isPatch":false,"sender":{"key":"jrnieder@gmail.com","avatar":"https://avatars.githubusercontent.com/u/281595?v=4"},"body":"Jonathan Nieder wrote:\n> [adding a few relevant cc's]\n> \n> Jon Seymour wrote:\n> \n> > Does anyone maintain a git repo with the patches used to successfully\n> > build the cygwin git packages?\n> \n> I think git.git is the canonical repo. On cygwin.com I don’t see any\n> package more recent than 1.6.6.1 so I suspect no one has packaged a\n> recent version yet.\ns/recent/more recent/\n\nSorry for the noise.\n"},{"id":"138264","messageId":"o2p2cfc40321003310055i4b26d46l29ac6113be9eb980@mail.gmail.com","threadId":"23253","inReplyTo":"20100331025856.GB13501@progeny.tock","subject":"Re: Are the patches used to build git on cygwin available in a git repo somewhere?","fromName":"Jon Seymour","fromEmail":"jon.seymour@gmail.com","sentAt":"2010-03-31T07:55:58Z","receivedAt":"2010-03-31T07:55:58Z","isPatch":false,"sender":{"key":"jon.seymour@gmail.com","avatar":"https://avatars.githubusercontent.com/u/207131?v=4"},"body":"What I was looking for was a git repo that contains a branch with the\npatches that the cygwin folks mutate the official git release with.\nSuch a repo would be quite useful because people could, in theory, use\nthese to produce their own cygwin git builds at arbitrary commits\ninstead of having to wait for cygwin to catch up.\n\nAdmittedly, it is not a big deal to build such a branch using the\npatch from the cygwin package source, but I was just wondering whether\nthere was a well-known repo that contained the patches already.\n\njon.\n\nOn Wed, Mar 31, 2010 at 1:58 PM, Jonathan Nieder wrote:\n> Jonathan Nieder wrote:\n>> [adding a few relevant cc's]\n>>\n>> Jon Seymour wrote:\n>>\n>> > Does anyone maintain a git repo with the patches used to successfully\n>> > build the cygwin git packages?\n>>\n>> I think git.git is the canonical repo.  On cygwin.com I don’t see any\n>> package more recent than 1.6.6.1 so I suspect no one has packaged a\n>> recent version yet.\n> s/recent/more recent/\n>\n> Sorry for the noise.\n>\n"},{"id":"138276","messageId":"4BB336C1.4000504@redhat.com","threadId":"23253","inReplyTo":"o2p2cfc40321003310055i4b26d46l29ac6113be9eb980@mail.gmail.com","subject":"Re: Are the patches used to build git on cygwin available in a git repo somewhere?","fromName":"Eric Blake","fromEmail":"eblake@redhat.com","sentAt":"2010-03-31T11:49:21Z","receivedAt":"2010-03-31T11:49:21Z","isPatch":false,"sender":{"key":"eblake@redhat.com","avatar":"https://avatars.githubusercontent.com/u/32933908?v=4"},"body":"According to Jon Seymour on 3/31/2010 1:55 AM:\n> What I was looking for was a git repo that contains a branch with the\n> patches that the cygwin folks mutate the official git release with.\n> Such a repo would be quite useful because people could, in theory, use\n> these to produce their own cygwin git builds at arbitrary commits\n> instead of having to wait for cygwin to catch up.\n> \n> Admittedly, it is not a big deal to build such a branch using the\n> patch from the cygwin package source, but I was just wondering whether\n> there was a well-known repo that contained the patches already.\n\nNope - the patchset that I apply when building the cygwin port is so small\nthat so far I have just maintained it manually, rather than publishing a\ngit repo. But if you want me to make it more public, I can certainly look\ninto doing that. And yes, now that 1.7.x is out for some time now, I\nshould probably advance the cygwin port beyond 1.6.6.1.\n\n-- \nEric Blake eblake@redhat.com +1-801-349-2682\nLibvirt virtualization library http://libvirt.org\n\n"},{"id":"138278","messageId":"w2y2cfc40321003310458vc0fc3dddjb47a8a7dddb1c41e@mail.gmail.com","threadId":"23253","inReplyTo":"4BB336C1.4000504@redhat.com","subject":"Re: Are the patches used to build git on cygwin available in a git repo somewhere?","fromName":"Jon Seymour","fromEmail":"jon.seymour@gmail.com","sentAt":"2010-03-31T11:58:18Z","receivedAt":"2010-03-31T11:58:18Z","isPatch":false,"sender":{"key":"jon.seymour@gmail.com","avatar":"https://avatars.githubusercontent.com/u/207131?v=4"},"body":"On Wed, Mar 31, 2010 at 11:49 PM, Eric Blake wrote:\n> According to Jon Seymour on 3/31/2010 1:55 AM:\n>> What I was looking for was a git repo that contains a branch with the\n>> patches that the cygwin folks mutate the official git release with.\n>> Such a repo would be quite useful because people could, in theory, use\n>> these to produce their own cygwin git builds at arbitrary commits\n>> instead of having to wait for cygwin to catch up.\n>>\n>> Admittedly, it is not a big deal to build such a branch using the\n>> patch from the cygwin package source, but I was just wondering whether\n>> there was a well-known repo that contained the patches already.\n>\n> Nope - the patchset that I apply when building the cygwin port is so small\n> that so far I have just maintained it manually, rather than publishing a\n> git repo.  But if you want me to make it more public, I can certainly look\n> into doing that.  And yes, now that 1.7.x is out for some time now, I\n> should probably advance the cygwin port beyond 1.6.6.1.\n>\n\nAs you say, it's probably not worth the effort, but if you ever do\ndecide to set such a thing up, do let me know.\n\njon.\n\n> --\n> Eric Blake   eblake@redhat.com    +1-801-349-2682\n> Libvirt virtualization library http://libvirt.org\n>\n>\n"},{"id":"138316","messageId":"20100401023526.GA20916@progeny.tock","threadId":"23253","inReplyTo":"o2p2cfc40321003310055i4b26d46l29ac6113be9eb980@mail.gmail.com","subject":"Re: Are the patches used to build git on cygwin available in a git repo somewhere?","fromName":"Jonathan Nieder","fromEmail":"jrnieder@gmail.com","sentAt":"2010-04-01T02:35:26Z","receivedAt":"2010-04-01T02:35:26Z","isPatch":false,"sender":{"key":"jrnieder@gmail.com","avatar":"https://avatars.githubusercontent.com/u/281595?v=4"},"body":"Jon Seymour wrote:\n\n> What I was looking for was a git repo that contains a branch with the\n> patches that the cygwin folks mutate the official git release with.\n\nTo clarify, I didn’t realize the Cygwin folks needed a patch at all.\nOh well, theory and practice.\n\nI just fetched the packaging.\n\n http://mirror.mcs.anl.gov/cygwin/release/git/git-1.6.6.1-1-src.tar.bz2\n\ngitk - work around stderr redirection on Cygwin\n\n The description of this patch suggests it is meant to work around\n the old Tcl/Tk version. In that case, maybe stock gitk should learn\n a workaround. I don’t think it is supposed to require more recent\n Tcl/Tk than 8.4.\n\n Unfortunately, I cannot find a relevant changelog entry. Maybe\n this is a Windows-specific bug? http://wiki.tcl.tk/2620\n describes a similar problem.\n\ngitk - convert gitk-path to Windows if on Cygwin\n\n This patch seems reasonable, and it only affects Cygwin. I think it\n looks reasonable for inclusion in stock gitk, though others might\n disagree.\n\nDocumentation/Makefile\n\n Adds --unsafe to the asciidoc command line. Why?\n\nMakefile\n\n Stops disabling so many features, since Cygwin has come a long way.\n This looks worth applying upstream. The conservative thing to do\n would be to test $(uname -r), but since it is easy to bring a\n Cygwin installation up to date and hard to figure out the appropriate\n versions, it might make sense to make this change unconditionally.\n\n A worrisome one is NO_MMAP. Was that problem ever understood? Maybe\n v1.6.3-rc0~133 (MinGW: implement mmap, 2009-03-13) contains some clues\n (just a hope). The message for v1.5.0-rc1~182 (Set NO_MMAP for Cygwin\n by default, 2006-12-27) indicates that it’s filesystem-specific, \n\nMakefile: all:: perl/perl.mak\n\n Should be unnecessary. The scripts should pull it in already.\n\nMakefile: setting INSTALLDIRS=vendor in the perl/perl.mak target\n\n Should be unnecessary. Make passes on variable settings from the\n command line to submakes already.\n\ngit-gui/Makefile:\n\n Change to Cygwin-specific part. Probably applicable upstream.\n\nThoughts?\nJonathan\n"},{"id":"138360","messageId":"4BB50925.9070006@redhat.com","threadId":"23253","inReplyTo":"20100401023526.GA20916@progeny.tock","subject":"Re: Are the patches used to build git on cygwin available in a git repo somewhere?","fromName":"Eric Blake","fromEmail":"eblake@redhat.com","sentAt":"2010-04-01T20:59:17Z","receivedAt":"2010-04-01T20:59:17Z","isPatch":false,"sender":{"key":"eblake@redhat.com","avatar":"https://avatars.githubusercontent.com/u/32933908?v=4"},"body":"On 03/31/2010 08:35 PM, Jonathan Nieder wrote:\n> I just fetched the packaging.\n> \n> http://mirror.mcs.anl.gov/cygwin/release/git/git-1.6.6.1-1-src.tar.bz2\n> \n> gitk - work around stderr redirection on Cygwin\n> \n> The description of this patch suggests it is meant to work around\n> the old Tcl/Tk version. In that case, maybe stock gitk should learn\n> a workaround. I don’t think it is supposed to require more recent\n> Tcl/Tk than 8.4.\n> \n> Unfortunately, I cannot find a relevant changelog entry. Maybe\n> this is a Windows-specific bug? http://wiki.tcl.tk/2620\n> describes a similar problem.\n\nMore of a problem of the fact that cygwin tcl is _still_ stuck with a\nbastardized implementation that is not quite a true cygwin app, and\ntherefore doesn't handle file redirections as gracefully as it could.\nMark Levedahl proposed the patch upstream on 14 Jun 2008, but it was\nnever accepted.\nhttp://marc.info/?t=121346288300001&r=1&w=2\n\n> \n> gitk - convert gitk-path to Windows if on Cygwin\n> \n> This patch seems reasonable, and it only affects Cygwin. I think it\n> looks reasonable for inclusion in stock gitk, though others might\n> disagree.\n\nOK, I'll find time to re-submit it upstream. It was modified from this\noriginal post, also by Mark Levedahl:\nhttp://marc.info/?l=git&m=121349005001446&w=2\n\n> \n> Documentation/Makefile\n> \n> Adds --unsafe to the asciidoc command line. Why?\n\nBecause VPATH builds of the documentation make asciidoc fail otherwise,\ndue to a complaint about an unsafe use of ../ referencing to find the\nsource dir outside of the build dir.\n\n> \n> Makefile\n> \n> Stops disabling so many features, since Cygwin has come a long way.\n> This looks worth applying upstream. The conservative thing to do\n> would be to test $(uname -r), but since it is easy to bring a\n> Cygwin installation up to date and hard to figure out the appropriate\n> versions, it might make sense to make this change unconditionally.\n\nMost of those defaults cater to cygwin 1.5, which was released several\nyears ago. Cygwin 1.7 is the only supported version now, but it was\nonly released late 2009. I'll try to find time to submit the\nless-controversial of these upstream.\n\n> \n> A worrisome one is NO_MMAP. Was that problem ever understood? Maybe\n> v1.6.3-rc0~133 (MinGW: implement mmap, 2009-03-13) contains some clues\n> (just a hope). The message for v1.5.0-rc1~182 (Set NO_MMAP for Cygwin\n> by default, 2006-12-27) indicates that it’s filesystem-specific, \n\nNo one has ever demonstrated to me why NO_MMAP was needed on cygwin.\nI'd rather get mmap fixed on cygwin, if it really is a bug (and if it\nstill exists; it is highly likely that the bug was against 1.5 but has\nbeen fixed in the meantime).\n\n> \n> Makefile: all:: perl/perl.mak\n> \n> Should be unnecessary. The scripts should pull it in already.\n\nIt made a difference for me when packaging for cygwin. But if there's a\nway to make it work without that line, I'm all ears.\n\n> \n> Makefile: setting INSTALLDIRS=vendor in the perl/perl.mak target\n> \n> Should be unnecessary. Make passes on variable settings from the\n> command line to submakes already.\n\nAgain, I could never get it to work without this patch; but I'm all ears\nif there's a better way.\n\n> \n> git-gui/Makefile:\n> \n> Change to Cygwin-specific part. Probably applicable upstream.\n\nOK, I'll try and find time to send an upstream patch submission.\n\nMeanwhile, I'm trying to package git 1.7.0.4 for cygwin, so this is a\ngood chance to review all of those patches in the cygwin port.\n\n-- \nEric Blake eblake@redhat.com +1-801-349-2682\nLibvirt virtualization library http://libvirt.org\n\n"},{"id":"138371","messageId":"x2p2cfc40321004011641s9964449cq4143021d0df63d07@mail.gmail.com","threadId":"23253","inReplyTo":"4BB50925.9070006@redhat.com","subject":"Re: Are the patches used to build git on cygwin available in a git repo somewhere?","fromName":"Jon Seymour","fromEmail":"jon.seymour@gmail.com","sentAt":"2010-04-01T23:41:32Z","receivedAt":"2010-04-01T23:41:32Z","isPatch":false,"sender":{"key":"jon.seymour@gmail.com","avatar":"https://avatars.githubusercontent.com/u/207131?v=4"},"body":"Eric,\n\nJust in case it helps, I have taken the cygwin-1.6.6.1-1 patches and\nsplit them up according to the discussions in this note and published\nthem as a tag (v1.6.6.1.cygwin-patch) in my fork of the github git/git\nrepo.\n\n git://github.com/jonseymour/git.git\n\nFeel free to clone this and do with it as you will.\n\nIf there any interest, I can post them to the list as patches, but I\nfigured it would probably be less confusing if Eric did that himself,\nshould that be required, especially since he has already started a new\nseries.\n\njon.\n\nOn Fri, Apr 2, 2010 at 7:59 AM, Eric Blake wrote:\n> On 03/31/2010 08:35 PM, Jonathan Nieder wrote:\n>> I just fetched the packaging.\n>>\n>>   http://mirror.mcs.anl.gov/cygwin/release/git/git-1.6.6.1-1-src.tar.bz2\n>>\n>> gitk - work around stderr redirection on Cygwin\n>>\n>>   The description of this patch suggests it is meant to work around\n>>   the old Tcl/Tk version.  In that case, maybe stock gitk should learn\n>>   a workaround.  I don’t think it is supposed to require more recent\n>>   Tcl/Tk than 8.4.\n>>\n>>   Unfortunately, I cannot find a relevant changelog entry.  Maybe\n>>   this is a Windows-specific bug?  http://wiki.tcl.tk/2620\n>>   describes a similar problem.\n>\n> More of a problem of the fact that cygwin tcl is _still_ stuck with a\n> bastardized implementation that is not quite a true cygwin app, and\n> therefore doesn't handle file redirections as gracefully as it could.\n> Mark Levedahl proposed the patch upstream on 14 Jun 2008, but it was\n> never accepted.\n> http://marc.info/?t=121346288300001&r=1&w=2\n>\n>>\n>> gitk - convert gitk-path to Windows if on Cygwin\n>>\n>>   This patch seems reasonable, and it only affects Cygwin.  I think it\n>>   looks reasonable for inclusion in stock gitk, though others might\n>>   disagree.\n>\n> OK, I'll find time to re-submit it upstream.   It was modified from this\n> original post, also by Mark Levedahl:\n> http://marc.info/?l=git&m=121349005001446&w=2\n>\n>>\n>> Documentation/Makefile\n>>\n>>   Adds --unsafe to the asciidoc command line.  Why?\n>\n> Because VPATH builds of the documentation make asciidoc fail otherwise,\n> due to a complaint about an unsafe use of ../ referencing to find the\n> source dir outside of the build dir.\n>\n>>\n>> Makefile\n>>\n>>   Stops disabling so many features, since Cygwin has come a long way.\n>>   This looks worth applying upstream.  The conservative thing to do\n>>   would be to test $(uname -r), but since it is easy to bring a\n>>   Cygwin installation up to date and hard to figure out the appropriate\n>>   versions, it might make sense to make this change unconditionally.\n>\n> Most of those defaults cater to cygwin 1.5, which was released several\n> years ago.  Cygwin 1.7 is the only supported version now, but it was\n> only released late 2009.  I'll try to find time to submit the\n> less-controversial of these upstream.\n>\n>>\n>>   A worrisome one is NO_MMAP.  Was that problem ever understood?  Maybe\n>>   v1.6.3-rc0~133 (MinGW: implement mmap, 2009-03-13) contains some clues\n>>   (just a hope).  The message for v1.5.0-rc1~182 (Set NO_MMAP for Cygwin\n>>   by default, 2006-12-27) indicates that it’s filesystem-specific,\n>\n> No one has ever demonstrated to me why NO_MMAP was needed on cygwin.\n> I'd rather get mmap fixed on cygwin, if it really is a bug (and if it\n> still exists; it is highly likely that the bug was against 1.5 but has\n> been fixed in the meantime).\n>\n>>\n>> Makefile: all:: perl/perl.mak\n>>\n>>   Should be unnecessary. The scripts should pull it in already.\n>\n> It made a difference for me when packaging for cygwin.  But if there's a\n> way to make it work without that line, I'm all ears.\n>\n>>\n>> Makefile: setting INSTALLDIRS=vendor in the perl/perl.mak target\n>>\n>>   Should be unnecessary.  Make passes on variable settings from the\n>>   command line to submakes already.\n>\n> Again, I could never get it to work without this patch; but I'm all ears\n> if there's a better way.\n>\n>>\n>> git-gui/Makefile:\n>>\n>>   Change to Cygwin-specific part.  Probably applicable upstream.\n>\n> OK, I'll try and find time to send an upstream patch submission.\n>\n> Meanwhile, I'm trying to package git 1.7.0.4 for cygwin, so this is a\n> good chance to review all of those patches in the cygwin port.\n>\n> --\n> Eric Blake   eblake@redhat.com    +1-801-349-2682\n> Libvirt virtualization library http://libvirt.org\n>\n>\n"}]}