Git development
 help / color / mirror / Atom feed
From: Johannes Schindelin <Johannes.Schindelin@gmx.de>
To: Jeff King <peff@peff.net>
Cc: Keith Cascio <keith@CS.UCLA.EDU>,
	Junio C Hamano <gitster@pobox.com>,
	git@vger.kernel.org
Subject: backwards compatibility, was Re: [PATCH v1 1/3] Introduce config variable "diff.primer"
Date: Mon, 26 Jan 2009 11:59:46 +0100 (CET)	[thread overview]
Message-ID: <alpine.DEB.1.00.0901261154330.14855@racer> (raw)
In-Reply-To: <20090126031206.GB14277@sigill.intra.peff.net>

Hi,

On Sun, 25 Jan 2009, Jeff King wrote:

>   1. Sometimes we blur the line of plumbing and porcelain, where
>      functionality is available only through plumbing. For example,
>      gitweb until recently called "git diff" because there is no other
>      way to diff two arbitrary blobs. But the solution there is, I
>      think, to make that functionality available through plumbing. Not
>      to disallow enhancements to porcelain.

Just a reminder: we are very conservative when it comes to breaking 
backwards compatibility.  For example, people running (but not upgrading) 
gitweb who want to upgrade Git may rightfully expect their setups not to 
be broken for a long time, if ever.

So your mentioning gitweb using "git diff" precludes all kind of cute 
games, methinks.

And please no "anybody who would do this and that would be nuts" excuses: 
if you want to change something fundamental like this, _you_ have to 
defend it.

It is not acceptable to just shout out what you want and expect those 
affected negatively to do the impact analysis for you.

Ciao,
Dscho

  parent reply	other threads:[~2009-01-26 11:00 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         ` Johannes Schindelin [this message]
2009-01-26 11:16           ` backwards compatibility, was " 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
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=alpine.DEB.1.00.0901261154330.14855@racer \
    --to=johannes.schindelin@gmx.de \
    --cc=git@vger.kernel.org \
    --cc=gitster@pobox.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