Git development
 help / color / mirror / Atom feed
From: "Philip Oakley" <philipoakley@iee.org>
To: "Junio C Hamano" <gitster@pobox.com>
Cc: "GitList" <git@vger.kernel.org>
Subject: Re: [RFC/PATCH 0/2] Test the Git version string
Date: Tue, 16 Apr 2013 22:22:57 +0100	[thread overview]
Message-ID: <2FE4D2171A03425AA9C8FF10140AC030@PhilipOakley> (raw)
In-Reply-To: 7v8v4imjji.fsf@alter.siamese.dyndns.org

From: "Junio C Hamano" <gitster@pobox.com>
Sent: Tuesday, April 16, 2013 7:12 PM
> "Philip Oakley" <philipoakley@iee.org> writes:
>
>>> What kind of benefit are you envisioning out of this?
>>
>> The purpose of tests is to detect mistakes and spot regressions.
>>
>> A change to the 'git version X.Y.z' string would be a regression, as 
>> I
>> spotted earlier, as it conflicts with expectations of standard
>> programmes such as git-gui.
>
> Sorry, but I do not follow.
>
> A released version says "git version 1.8.2.1".  In a month or so,
> I'll have another one that says "git version 1.8.3".  Or I may
> decide to bump in preparation for 2.0 and it may identify itself as
> "git version 1.9".
>
> Neither of which no existing "program such as git-gui" has ever
> seen.
>
> In what way is that a regression?

But they both pass the test and test suite, yes? And even if you use 
git-gui with them, git-gui will not barf on start up, which it would if 
the version string fails my test.

Passing the test suite should be a reasonble guarantee that co-tools 
will work, including those that check for version. This is a check for 
that version string.

However if someone[1] creates "My Special Git Version 1-9-3 (Index 
V7b)", and here I'm suggesting they may have other special changes, then 
the version check will tell them that even when they have fixed their 
special changes to pass the other parts of the test suite, other 
co-tools could barf.

Its about pushing the piece of string from the users end ;-) It also 
stops others trying to change 'git' to 'Git' within this message, just 
as I did.

Philip
>
[1] my first draft had 'you', but that is not a reasonable starting 
position. It's more about *others* attempting to create release 
versions, which announce their name, that we expect to be compatible 
with say git-gui (via the rest of the test suite), and need to check 
that announcement. 

      parent reply	other threads:[~2013-04-16 21:22 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-04-14 14:27 [RFC/PATCH 0/2] Test the Git version string Philip Oakley
2013-04-14 14:27 ` [RFC/PATCH 1/2] test git " Philip Oakley
2013-04-14 14:27 ` [RFC/PATCH 2/2] Doc: State the exact " Philip Oakley
2013-04-14 19:22 ` [RFC/PATCH 0/2] Test the Git " Junio C Hamano
2013-04-14 21:26   ` Philip Oakley
2013-04-15  1:39     ` Junio C Hamano
2013-04-16  7:25       ` Philip Oakley
2013-04-16 18:12         ` Junio C Hamano
2013-04-16 18:24           ` David Aguilar
2013-04-16 19:02             ` Junio C Hamano
2013-04-16 21:22           ` Philip Oakley [this message]

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=2FE4D2171A03425AA9C8FF10140AC030@PhilipOakley \
    --to=philipoakley@iee.org \
    --cc=git@vger.kernel.org \
    --cc=gitster@pobox.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