Re: [PATCH] "not uptodate" changed to "has local changes"

From: Johannes Schindelin <johannes.schindelin@gmx.de>
Date: May 17, 2008, 10:04 UTC
Message-ID: <alpine.DEB.1.00.0805171102480.30431@racer>
In-Reply-To: <b8bf37780805162030m4c961505nabd72e8f5bd08404@mail.gmail.com>

Hi,

On Sat, 17 May 2008, André Goddard Rosa wrote:

Show 23 quoted lines

> On Fri, May 16, 2008 at 2:12 PM, Kevin Ballard <kevin@sb.org> wrote:
> > On May 16, 2008, at 6:25 AM, Johannes Schindelin wrote:
> >
> >> On Thu, 15 May 2008, André Goddard Rosa wrote:
> >>
> >>>>> This patch will make git a little more human friendly, reporting
> >>>>> "file.txt: has local changes".
> >>>>
> >>>> Documentation/git-checkout.txt should also change in this case,
> >>>> otherwise users will see different output to that described and
> >>>> possibly get confused if following along with the examples.
> >>>>
> >>>
> >>> I like the idea too.
> >>
> >> No comment on the concern that it might break people's scripts?  None?
> >
> >
> > How about an ugly hack? Look to see if stdout is a tty, if so spit out the
> > more human-readable version, otherwise spit out the old version >:-)
> 
> Is this user interface set on stone? I think we should reserve the right 
> to improve always.

Umm.

As has been mentioned, this is not a "user interface". The message you are seeing comes from a _plumbing_ program, i.e. something _not_ meant for human consumption.

I still think that it might be better to add a command line option with a custom message, because that would _not_ break backwards-compatibility.

Thankyouverymuch, Dscho