From: Jeff King <peff@peff.net>
To: Avery Pennarun <apenwarr@gmail.com>
Cc: Ben Gamari <bgamari@gmail.com>, git <git@vger.kernel.org>
Subject: Re: Storing (hidden) per-commit metadata
Date: Sun, 21 Feb 2010 01:34:33 -0500 [thread overview]
Message-ID: <20100221063433.GA2840@coredump.intra.peff.net> (raw)
In-Reply-To: <32541b131002201057t31fc8a6aydb0942171fe1b8c8@mail.gmail.com>
On Sat, Feb 20, 2010 at 01:57:31PM -0500, Avery Pennarun wrote:
> As for git-notes, they sound like they would be useful for this sort
> of thing. I haven't tried them yet, but my understanding is that
> notes anywhere other than the "default" notes ref are not shown in
> commit messages, so you can use them for whatever you want.
I would want to hear more about the actual data being stored. The
strength of notes is that you can _change_ them after the commit has
been created. And the price you pay is that they are more annoying to
move around, because they are in a totally different ref.
If this is data that is being generated at the time the commit is
created and then set in stone, then it probably should be part of the
commit object.
If the only problem is that the data is ugly in "git show", then perhaps
we need a "suppress these pseudo-headers" feature for showing logs. It
keeps them easily available for inspection or for --grep, but most of
the time you would not see them.
-Peff
next prev parent reply other threads:[~2010-02-21 6:34 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-02-19 17:11 Storing (hidden) per-commit metadata Jelmer Vernooij
2010-02-20 17:41 ` Ben Gamari
2010-02-20 18:57 ` Avery Pennarun
2010-02-21 6:34 ` Jeff King [this message]
2010-02-21 8:49 ` Johannes Schindelin
2010-02-21 8:52 ` Jeff King
2010-02-21 12:17 ` Jelmer Vernooij
2010-02-22 5:17 ` Dmitry Potapov
2010-02-22 9:56 ` Jelmer Vernooij
2010-02-22 11:28 ` Dmitry Potapov
2010-02-22 11:59 ` Jelmer Vernooij
2010-02-22 13:08 ` Dmitry Potapov
2010-02-22 13:44 ` Jelmer Vernooij
2010-02-22 14:20 ` Dmitry Potapov
2010-02-22 19:13 ` Jelmer Vernooij
2010-02-22 14:57 ` Jelmer Vernooij
2010-02-22 5:11 ` Gabriel Filion
2010-02-22 9:49 ` Jelmer Vernooij
2010-02-22 22:13 ` "Alejandro R. Sedeño"
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=20100221063433.GA2840@coredump.intra.peff.net \
--to=peff@peff.net \
--cc=apenwarr@gmail.com \
--cc=bgamari@gmail.com \
--cc=git@vger.kernel.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).