On Thursday 21 January 2010, Junio C Hamano wrote: > [...] I just want to note that I've read the whole thread (up to here), and I agree with pretty much everything that's been said so far: - We should be more conservative about showing notes, especially in contexts that may be used by scripts. Disabling notes by default when --pretty/-- format is in use, sounds like a good idea. So does adding a --show-notes option for overriding the default. - The format-patch bug is grave and unexcusable and must be fixed. Michael: Thanks for discovering. - I'd still like to keep notes as part of the default output from git log and friends (when NOT using --pretty/--format). Only notes from a single notes ref (typically the default "refs/notes/commits") should be shown. - Re. Peff's worry that "git log" will fill up with random bisection cruft: Any notes that are related to bisection (or any other special use case for notes) should live on its own notes ref (typically "refs/notes/bisect" for bisection cruft) that is not used by "git log" (unless you explicitly say so through $GIT_NOTES_REF or core.notesRef). - Re. Junio's worry that he will become the janitor for these patches. Please don't. As long as the patch series is in 'pu', it is MY responsibility to address issues and organize any additional patches on top of the series. Feel free to ignore all additional patches, and wait for an updated series from my end. - Yes, there should be more tests verifying that there is no negative impact on git log and friends. Docs must be updated as well, where needed. Unfortunately I don't have the time to work on this right now, but I'll do my best to get around to it as soon as possible (at least by the end of the coming weekend). Again, thanks for your involvement. It is really appreciated. ...Johan -- Johan Herland, <johan@xxxxxxxxxxx> www.herland.net -- 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