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