From: Jonathan Nieder <jrnieder@gmail.com>
To: Jon Seymour <jon.seymour@gmail.com>
Cc: Git Mailing List <git@vger.kernel.org>,
Ramsay Allan Jones <ramsay@ramsay1.demon.co.uk>,
Dmitry Potapov <dpotapov@gmail.com>,
Eric Blake <eblake@redhat.com>
Subject: Re: Are the patches used to build git on cygwin available in a git repo somewhere?
Date: Wed, 31 Mar 2010 21:35:26 -0500 [thread overview]
Message-ID: <20100401023526.GA20916@progeny.tock> (raw)
In-Reply-To: <o2p2cfc40321003310055i4b26d46l29ac6113be9eb980@mail.gmail.com>
Jon Seymour wrote:
> What I was looking for was a git repo that contains a branch with the
> patches that the cygwin folks mutate the official git release with.
To clarify, I didn’t realize the Cygwin folks needed a patch at all.
Oh well, theory and practice.
I just fetched the packaging.
http://mirror.mcs.anl.gov/cygwin/release/git/git-1.6.6.1-1-src.tar.bz2
gitk - work around stderr redirection on Cygwin
The description of this patch suggests it is meant to work around
the old Tcl/Tk version. In that case, maybe stock gitk should learn
a workaround. I don’t think it is supposed to require more recent
Tcl/Tk than 8.4.
Unfortunately, I cannot find a relevant changelog entry. Maybe
this is a Windows-specific bug? http://wiki.tcl.tk/2620
describes a similar problem.
gitk - convert gitk-path to Windows if on Cygwin
This patch seems reasonable, and it only affects Cygwin. I think it
looks reasonable for inclusion in stock gitk, though others might
disagree.
Documentation/Makefile
Adds --unsafe to the asciidoc command line. Why?
Makefile
Stops disabling so many features, since Cygwin has come a long way.
This looks worth applying upstream. The conservative thing to do
would be to test $(uname -r), but since it is easy to bring a
Cygwin installation up to date and hard to figure out the appropriate
versions, it might make sense to make this change unconditionally.
A worrisome one is NO_MMAP. Was that problem ever understood? Maybe
v1.6.3-rc0~133 (MinGW: implement mmap, 2009-03-13) contains some clues
(just a hope). The message for v1.5.0-rc1~182 (Set NO_MMAP for Cygwin
by default, 2006-12-27) indicates that it’s filesystem-specific,
Makefile: all:: perl/perl.mak
Should be unnecessary. The scripts should pull it in already.
Makefile: setting INSTALLDIRS=vendor in the perl/perl.mak target
Should be unnecessary. Make passes on variable settings from the
command line to submakes already.
git-gui/Makefile:
Change to Cygwin-specific part. Probably applicable upstream.
Thoughts?
Jonathan
next prev parent reply other threads:[~2010-04-01 2:41 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-03-30 16:16 Are the patches used to build git on cygwin available in a git repo somewhere? Jon Seymour
2010-03-31 2:56 ` Jonathan Nieder
2010-03-31 2:58 ` Jonathan Nieder
2010-03-31 7:55 ` Jon Seymour
2010-03-31 11:49 ` Eric Blake
2010-03-31 11:58 ` Jon Seymour
2010-04-01 2:35 ` Jonathan Nieder [this message]
2010-04-01 20:59 ` Eric Blake
2010-04-01 23:41 ` Jon Seymour
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20100401023526.GA20916@progeny.tock \
--to=jrnieder@gmail.com \
--cc=dpotapov@gmail.com \
--cc=eblake@redhat.com \
--cc=git@vger.kernel.org \
--cc=jon.seymour@gmail.com \
--cc=ramsay@ramsay1.demon.co.uk \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.