From: Steven Penny <svnpenn@gmail.com>
To: cygwin@cygwin.com
Subject: Re: Removing Git dependency on Perl [Was: [ANNOUNCEMENT] Updated: Git v2.13.2-2]
Date: Fri, 21 Jul 2017 12:20:00 -0000 [thread overview]
Message-ID: <5971ed2f.c6efca0a.c764c.a02b@mx.google.com> (raw)
In-Reply-To: <20170721101141.GB2185@dinwoodie.org>
On Fri, 21 Jul 2017 11:11:41, Adam Dinwoodie wrote:
> Hmm. I'm struggling to work out what's appropriate here. I'd be
> entirely happy splitting off the `git archimport` command to a separate
> package if there were something to gain there, but I'm very wary of
> splitting `git add -p` and friends off: I think that's liable to cause
> considerable confusion for people who use those commands and who would
> probably not expect to need a separate package for them.
>
> I'm experimenting with what this would look like now, but if you can
> find and point me at some other platform's Git distribution that also
> separates out `git-add--interactive`, so I can see how they've handled
> this situation, that would potentially be very useful.
Good points. We might need to just leave it alone for the time being:
1. One person has already spoken up in this thread against it:
http://cygwin.com/ml/cygwin/2017-07/msg00287.html
2. Debian Git still requires it:
http://packages.debian.org/experimental/git
3. git-add--interactive.perl is still being worked on, 14 commits this year:
http://github.com/git/git/commits/master/git-add--interactive.perl
So until they rewrite it in C, we might need to just keep things the way they
are. I dont use it with my workflow, so its a shame to pull in Perl for a single
file that I do not use, but I get the arguments against.
--
Problem reports: http://cygwin.com/problems.html
FAQ: http://cygwin.com/faq/
Documentation: http://cygwin.com/docs.html
Unsubscribe info: http://cygwin.com/ml/#unsubscribe-simple
next prev parent reply other threads:[~2017-07-21 12:01 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-07-13 14:37 [ANNOUNCEMENT] Updated: Git v2.13.2-2 Adam Dinwoodie
2017-07-14 2:28 ` Steven Penny
2017-07-21 10:18 ` Removing Git dependency on Perl [Was: [ANNOUNCEMENT] Updated: Git v2.13.2-2] Adam Dinwoodie
2017-07-21 12:01 ` James Darnley
2017-07-21 12:20 ` Steven Penny [this message]
2017-07-21 14:28 ` cyg Simple
2017-07-21 20:02 ` Steven Penny
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=5971ed2f.c6efca0a.c764c.a02b@mx.google.com \
--to=svnpenn@gmail.com \
--cc=cygwin@cygwin.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).