Re: [PATCH] git-notes.txt: clarify -C vs. copy and -F

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

 



Junio C Hamano <gitster@xxxxxxxxx> writes:

> Michael J Gruber <git@xxxxxxxxxxxxxxxxxxxx> writes:
>
>> The current description of '-C' together with the analogy to 'git commit
>> -C' can lead to the wrong conclusion that '-C' copies notes between
>> objects. Make this clearer by rewording and pointing to 'copy'.
>>
>> The example for attaching binary notes with 'git hash-object' followed
>> by 'git notes add -C' immediately raises the question: "Why not use 'git
>> notes add -F'?". Answer it (the latter is not binary-safe).
>>
>> Signed-off-by: Michael J Gruber <git@xxxxxxxxxxxxxxxxxxxx>
>> ---
>> In fact, the long name '--reuse-message' is really misleading, but I've been
>> around long enough to refrain from trying to change it ;)
>
> Yeah, it utterly is broken.  Why not fix it before people start making
> serious use of notes?

Actually I take it back and throw it again after doubling it.  Not just
the long name, but using -C/-c is already utterly broken.  These are meant
to reuse (meta)data associated with an existing object, not using some
data that happens to be stored in a random loose blob.  I don't think of
any similar option anywhere in git.

Instead of mucking with the documentation, why not fix the behaviour to
match what -C/-c/--reuse usually means, which is what the documentation
describes?
--
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]