git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: Storing (hidden) per-commit metadata

From
Jelmer Vernooij <jelmer@samba.org>
Date
Feb 22, 2010, 19:13 UTC
Message-ID
<1266865993.11527.49.camel@ganieda>
In-Reply-To
<20100222142013.GA7863@dpotapov.dyndns.org>
On Mon, 2010-02-22 at 17:20 +0300, Dmitry Potapov wrote:
Show 12 quoted lines
> On Mon, Feb 22, 2010 at 02:44:49PM +0100, Jelmer Vernooij wrote:
> > On Mon, 2010-02-22 at 16:08 +0300, Dmitry Potapov wrote:
> > > I am not sure that the commit object is the right place to store that
> > > metadata, but hidding this information is even more problematic. Let's
> > > suppose that someone cherry-pick your Bazaar originated commit. Now when
> > > you try to synchronize with Bazaar, your synchronizer will see that it
> > > has some Bazaar revision ID and branch name, but, in fact, it is new
> > > commit on a completely different branch...
> > I don't see how the fact that the bzr-git/hg-git data is being hidden is
> > the problem in the scenario you mention.
> Because you can easily remove that information manually when you cherry-pick
> some commit. It is more difficult to do when it is hidden.

My point is that if you don't make it part of the user-visible commit message there is no need to remove it at all, it'll just disappear by itself.

Show 8 quoted lines
> > It'd be nice if this sort of information was discarded by "git rebase",
> > but that's another good reason to treat it in a different way from the
> > commit message instead.
> Well, I do not see any other place in the commit object aside the commit
> message where you can easily put information, and I do not think it is a
> good idea for "git rebase" to edit the commit message automatically.
> Maybe, you should look at git-notes. (I don't know enough about them to
> tell whether they are suitable or not).

Some other people have suggested putting e.g. a RFC822-style header in the commit message field and using the headers in that to allow custom revision properties, only displaying the body in "git log", "git show" etc. What do you think about that?

Cheers,
Jelmer
Previous: Dmitry PotapovNext: Jelmer Vernooij
Message 15 of 19 in “Storing (hidden) per-commit metadata”
  1. Jelmer VernooijFeb 19, 2010
  2. Ben GamariFeb 20, 2010
  3. Avery PennarunFeb 20, 2010
  4. Jeff KingFeb 21, 2010
  5. Johannes SchindelinFeb 21, 2010
  6. Jeff KingFeb 21, 2010
  7. Jelmer VernooijFeb 21, 2010
  8. Dmitry PotapovFeb 22, 2010
  9. Jelmer VernooijFeb 22, 2010
  10. Dmitry PotapovFeb 22, 2010
  11. Jelmer VernooijFeb 22, 2010
  12. Dmitry PotapovFeb 22, 2010
  13. Jelmer VernooijFeb 22, 2010
  14. Dmitry PotapovFeb 22, 2010
  15. Jelmer VernooijFeb 22, 2010
  16. Jelmer VernooijFeb 22, 2010
  17. Gabriel FilionFeb 22, 2010
  18. Jelmer VernooijFeb 22, 2010
  19. Alejandro R. SedeñoFeb 22, 2010

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.