git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Junio C Hamano <gitster@pobox.com>
To: Jay Soffian <jaysoffian@gmail.com>
Cc: Jeff King <peff@peff.net>,
	Johannes Schindelin <Johannes.Schindelin@gmx.de>,
	Keith Cascio <keith@cs.ucla.edu>,
	git@vger.kernel.org
Subject: Re: backwards compatibility, was Re: [PATCH v1 1/3] Introduce config variable "diff.primer"
Date: Mon, 26 Jan 2009 12:04:20 -0800	[thread overview]
Message-ID: <7vd4e96dh7.fsf@gitster.siamese.dyndns.org> (raw)
In-Reply-To: <76718490901261149xfedc415j8f5dab677b90d693@mail.gmail.com> (Jay Soffian's message of "Mon, 26 Jan 2009 14:49:15 -0500")

Jay Soffian <jaysoffian@gmail.com> writes:

> On Mon, Jan 26, 2009 at 1:48 PM, Jeff King <peff@peff.net> wrote:
>> But then you have that problem that the _user_ is stuck specifying "OK,
>> turn on fancy features." And I don't relish the thought of typing "git
>> diff -J" every time. :)
>
> Well, this issue seems to come up every so often, so the idea would be:
>
> - We're adding a mechanism for scripts to communicate that they need
> plumbing context
> - Start using it in your scripts when calling git if you rely on a
> stable interface
> - In the next major release, git may introduce changes to commands
> which are not clearly plumbing if you haven't adopted the mechanism

Where do all of these nonsense come from?  We are not adding any mechanism
for scripts to say they need plumbing context.  By calling plumbing they
are already asking for stable plumbing behaviour.

The scripts can, if they want to, use newer options updated versions of
the plumbing commands offer, by passing them when they want to.

And the trigger to do so is up to the scripts.  They can get new options
from the end user, or they can peek into user's configuration variables
similar to the diff.primer mentioned earlier in the discussion.

One way could be a new option --screw-me-with=name that can be given to a
plumbing command and tells it pretend as if the command line options
specified by the configuration variable of the given name were given
(e.g. a script runs "git diff-files --screw-me-with=diff.primer").

The important point is that it has to be opt _IN_.

  reply	other threads:[~2009-01-26 20:05 UTC|newest]

Thread overview: 41+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-01-25 17:30 [PATCH v1 0/3] Introduce config variable "diff.primer" Keith Cascio
2009-01-25 17:30 ` [PATCH v1 1/3] " Keith Cascio
2009-01-25 17:30   ` [PATCH v1 2/3] Test functionality of new " Keith Cascio
2009-01-25 17:30     ` [PATCH v1 3/3] git-gui hooks for " Keith Cascio
2009-01-25 18:22       ` Johannes Schindelin
2009-01-25 18:58         ` Keith Cascio
2009-01-25 18:17   ` [PATCH v1 1/3] Introduce " Johannes Schindelin
2009-01-25 18:44     ` Keith Cascio
2009-01-25 19:30       ` Johannes Schindelin
2009-01-25 20:14         ` Keith Cascio
2009-01-25 22:11       ` Jeff King
2009-01-25 22:58         ` Keith Cascio
2009-01-25 23:25           ` Jeff King
2009-01-25 20:34   ` Junio C Hamano
2009-01-26  2:30     ` Junio C Hamano
2009-01-26  2:37       ` Keith Cascio
2009-01-26  3:18       ` Jeff King
2009-01-26  3:38         ` Junio C Hamano
2009-01-26  2:40     ` Keith Cascio
2009-01-26  3:12       ` Jeff King
2009-01-26  3:42         ` Junio C Hamano
2009-01-26  3:45           ` Jeff King
2009-01-26 10:54         ` Johannes Schindelin
2009-01-26 11:06           ` Jeff King
2009-01-26 10:59         ` backwards compatibility, was " Johannes Schindelin
2009-01-26 11:16           ` Jeff King
2009-01-26 11:28             ` Johannes Schindelin
2009-01-26 11:59               ` Jeff King
2009-01-27  3:01                 ` Keith Cascio
2009-01-26 15:29             ` Jay Soffian
2009-01-26 18:48               ` Jeff King
2009-01-26 19:49                 ` Jay Soffian
2009-01-26 20:04                   ` Junio C Hamano [this message]
2009-01-26 20:32                     ` Jay Soffian
2009-01-26 20:35                     ` Jeff King
2009-01-26  3:36       ` Junio C Hamano
2009-01-25 20:35 ` [PATCH v1 0/3] " Junio C Hamano
2009-01-25 20:41   ` Keith Cascio
2009-01-25 22:07     ` Jeff King
2009-01-27  1:47       ` Keith Cascio
2009-01-27  4:54         ` Jeff King

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=7vd4e96dh7.fsf@gitster.siamese.dyndns.org \
    --to=gitster@pobox.com \
    --cc=Johannes.Schindelin@gmx.de \
    --cc=git@vger.kernel.org \
    --cc=jaysoffian@gmail.com \
    --cc=keith@cs.ucla.edu \
    --cc=peff@peff.net \
    /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 NNTP newsgroup(s).