From: Johannes Sixt <J.Sixt@eudaptics.com>
To: Junio C Hamano <junkio@cox.net>
Cc: git@vger.kernel.org
Subject: Re: grafts+repack+prune = history at danger
Date: Fri, 26 Jan 2007 10:48:06 +0100 [thread overview]
Message-ID: <45B9CE56.D16DFC81@eudaptics.com> (raw)
In-Reply-To: 7vzm86yw0q.fsf@assigned-by-dhcp.cox.net
Junio C Hamano wrote:
>
> Johannes Sixt <J.Sixt@eudaptics.com> writes:
>
> > Here's my stance on it. Grafts should be a local matter. And they alter
> > the world view, with a pronounciation on *view*. That's why I proposed
> > that only log familiy of commands obey them[*]. And probably rev-list so
> > that gitk et.al. have a way to obey them. And also the ref parser (so
> > that master~20 is what it looks it is). Everything else should disregard
> > grafts: repack, prune, fetch, <transfer>-pack, push etc. No nasty side
> > effects anymore.
>
> I said you are not agreeing, but I should have said you are not
> understanding.
Oh, I think I understand very well. It may just be that I cannot express
myself that well ;)
I propose that grafts are only about *view*, not database integrity.
There are no tools that manipulate grafts, that would stop the user to
make some blunder; the user has to edit the file *manually*. It is
wrong, wrong, wrong to let such a file dictate database integrity.
> grafts can bring otherwise disconnected commits into the
> altered history, so if you want your log to honor grafts, your
> prune and repack need to be aware of them lest you would not
> lose them.
Sure, if I connect my linux repo with a graft to the historical BK tree,
then toss the ref that pointed to the historical tree, then git prune:
- then currently it won't prune the historical tree
- but under my proposal it would. Silly me. Why did I remove that ref?
-- Hannes
next prev parent reply other threads:[~2007-01-26 9:46 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-01-25 17:17 grafts+repack+prune = history at danger Johannes Sixt
2007-01-25 23:07 ` Junio C Hamano
2007-01-26 8:13 ` Johannes Sixt
2007-01-26 8:54 ` Junio C Hamano
2007-01-26 9:21 ` Johannes Sixt
2007-01-26 9:31 ` Junio C Hamano
2007-01-26 9:48 ` Johannes Sixt [this message]
2007-01-26 10:15 ` Junio C Hamano
2007-01-26 10:41 ` Johannes Sixt
2007-01-26 11:29 ` Junio C Hamano
2007-01-26 13:08 ` Jakub Narebski
2007-01-26 15:55 ` Linus Torvalds
2007-01-26 23:46 ` Junio C Hamano
2007-01-27 0:56 ` Linus Torvalds
2007-01-26 9:15 ` Mark Wooding
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=45B9CE56.D16DFC81@eudaptics.com \
--to=j.sixt@eudaptics.com \
--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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.