From: Junio C Hamano <gitster@pobox.com>
To: Michael J Gruber <git@drmicha.warpmail.net>
Cc: git@vger.kernel.org, Linus Torvalds <torvalds@linux-foundation.org>
Subject: Re: [RFC] add test cases for the --repo option to git push
Date: Fri, 27 Feb 2009 09:34:07 -0800 [thread overview]
Message-ID: <7vy6vrkcmo.fsf@gitster.siamese.dyndns.org> (raw)
In-Reply-To: 49A7C3A7.6060202@drmicha.warpmail.net
Michael J Gruber <git@drmicha.warpmail.net> writes:
> Junio C Hamano venit, vidit, dixit 26.02.2009 18:09:
>> Michael J Gruber <git@drmicha.warpmail.net> writes:
>>
>>> First of all: I define good/bad as matching the documentation.
>>
>> Ok, I was primarily working from this:
>>
>> commit bcc785f611dc6084be75999a3b6bafcc950e21d6
>> Author: Linus Torvalds <torvalds@osdl.org>
>> Date: Mon Oct 30 08:28:59 2006 -0800
>
>
> [snip]
>> You will see that:
>>
>> (1) bf07cc5 (i.e. J6t's documentation) passes your tests;
>
> Hmm, I don't see that, we must be doing something differently, see below.
No, I didn't even know if the commit passed your tests. Before the [snip]
I think I had something like:
If your claim that this is a bug is true, it might have happened
this way: Linus originally wrote it, behaviour changed, J6t
documented the updated behaviour, behaviour changed again.
and after (1)/(2)/..., I have:
>> if the above conjecture is true, and we may want to fix that regression to
>> match the documentation.
So if you don't see that, it merely proves that the conjecture is false.
The updated documentation didn't describe correctly what should happen and
there is no bug other than in the documentation.
It's a different matter if we want to update the semantics, but I am not
sure if people agree with "the documented version is more natural". I'm
neutral right now. I may have more to say after I re-read the detailed
analysis part of your message.
next prev parent reply other threads:[~2009-02-27 17:35 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-02-20 9:16 git push usage Jay Soffian
2009-02-21 9:32 ` Jay Soffian
2009-02-24 17:40 ` [RFC] add test cases for the --repo option to git push Michael J Gruber
2009-02-25 21:58 ` Junio C Hamano
2009-02-26 9:26 ` Michael J Gruber
2009-02-26 17:09 ` Junio C Hamano
2009-02-26 17:48 ` Michael J Gruber
2009-02-26 22:11 ` Jay Soffian
2009-02-27 10:42 ` Michael J Gruber
2009-02-27 17:34 ` Junio C Hamano [this message]
2009-02-27 20:48 ` Jay Soffian
2009-02-27 21:00 ` Linus Torvalds
2009-02-27 21:21 ` Jay Soffian
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=7vy6vrkcmo.fsf@gitster.siamese.dyndns.org \
--to=gitster@pobox.com \
--cc=git@drmicha.warpmail.net \
--cc=git@vger.kernel.org \
--cc=torvalds@linux-foundation.org \
/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).