Re: [PATCH] fix many comment typos

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



René Scharfe <rene.scharfe@xxxxxxxxxxxxxx> writes:

> Am 03.06.2012 09:11, schrieb Jim Meyering:
>> Perhaps a more palatable change, here and in the other two places:
>> s/seeked/cg-seek'd/, i.e.,
>>
>> - * "git bisect" showed mysterious "won't bisect on seeked tree" error message.
>> + * "git bisect" showed mysterious "won't bisect on cg-seek'd tree" error message.
>>
>> Then, if someone does this again, it will be more obvious that
>> it is not a typo.
>
> This change is only valid if the command "git bisect" at some point
> printed "won't bisect on cg-seek'd tree" instead of "won't bisect on
> seeked tree".

Yeah, that is how I should have spelled my original message; thanks
for clarifying.

> And even then, it doesn't make now sense to change
> already published release notes (Documentation/RelNotes/1.5.4.4.txt),
> after the fact.

If it makes it easier to touch older release notes at the same time
for a mechanical typocorrection procedure, whose primary purpose is
to fix the tip of the current tree, I wouldn't mind typofixes to
older release notes.

But we would need to be careful to avoid a mistake like this one.
Fixing typo in the words I used to describe the changes is perfectly
fine and welcomed; changing the exact output string that was copied
from the implementation to show historical fact is not.

I have found Jim's patches always carefully crafted and thoughtfully
done in the past, especially with the disclosure of the mechanical
procedure that was used so that the result can easily be verified
independently; the first hunk that showed a change that was totally
against the care the procedure claimed to have taken stood up like a
sore thumb and that was the only reason I noticed it.

Thanks.

--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]