public inbox for cygwin@cygwin.com
 help / color / mirror / Atom feed
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

  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).