git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Shawn Pearce <spearce@spearce.org>
To: Johannes Schindelin <Johannes.Schindelin@gmx.de>
Cc: Junio C Hamano <junkio@cox.net>, git@vger.kernel.org
Subject: Re: [RFC] git reflog show
Date: Sun, 24 Dec 2006 01:11:51 -0500	[thread overview]
Message-ID: <20061224061151.GE7443@spearce.org> (raw)
In-Reply-To: <Pine.LNX.4.63.0612231552140.19693@wbgn013.biozentrum.uni-wuerzburg.de>

Johannes Schindelin <Johannes.Schindelin@gmx.de> wrote:
> I wonder if it would make sense to teach the revision walking machinery 
> about reflogs. A commit could be marked as coming from a reflog entry, and 
> in that case the parents could be determined by the reflog rather than the 
> commit itself.

The revision machinery already knows about reflogs with --reflog,
used by git-pack-objects via git-repack.  But here its really only
useful to seed the list of commits to be walked as part of a pack
generation, to make sure the things referenced by the reflog stay
around after a repacking.  And it implies --all.

Rewriting the commits in memory to appear to have parents based
on their order of appearence in the reflog would nicely generate
a single strand of perls, but it makes it difficult to then access
the same commit's real parents, doesn't it?  So that may make the
revision machinary somewhat limited in some applications.

Besides we want the reflog message entry and not the commit message
when we perform pretty output, etc.  So really we are then talking
about generating synthetic commit objects for the reflog data.

-- 
Shawn.

  reply	other threads:[~2006-12-24  6:12 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-12-23 10:19 [RFC] git reflog show Shawn Pearce
2006-12-23 10:42 ` Junio C Hamano
2006-12-23 14:53   ` Johannes Schindelin
2006-12-24  6:11     ` Shawn Pearce [this message]
2006-12-24 12:49       ` Johannes Schindelin
2006-12-24 21:47         ` Junio C Hamano
2006-12-25 14:01           ` Johannes Schindelin
2006-12-25 18:07             ` Junio C Hamano
2006-12-26  5:48               ` Shawn Pearce
2006-12-26  6:14                 ` Junio C Hamano
2006-12-26 16:14               ` Johannes Schindelin
2006-12-26 19:03                 ` Junio C Hamano
2006-12-26 23:20                   ` Johannes Schindelin
2006-12-26 23:50                     ` Junio C Hamano

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=20061224061151.GE7443@spearce.org \
    --to=spearce@spearce.org \
    --cc=Johannes.Schindelin@gmx.de \
    --cc=git@vger.kernel.org \
    --cc=junkio@cox.net \
    /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).