public inbox for cygwin@cygwin.com
 help / color / mirror / Atom feed
From: cyg Simple <cygsimple@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 14:28:00 -0000	[thread overview]
Message-ID: <d9761d0b-5495-a4f2-e072-8c308a0cb833@gmail.com> (raw)
In-Reply-To: <5971ed2f.c6efca0a.c764c.a02b@mx.google.com>

On 7/21/2017 8:01 AM, Steven Penny wrote:
> 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.

Perl is a requirement for development work using autoconf and friends
anyway so is it that much of a struggle to require it for git?

-- 
cyg Simple

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

  reply	other threads:[~2017-07-21 12:58 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
2017-07-21 14:28       ` cyg Simple [this message]
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=d9761d0b-5495-a4f2-e072-8c308a0cb833@gmail.com \
    --to=cygsimple@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).