From: Mark Levedahl <mlevedahl@gmail.com>
To: Johannes Schindelin <Johannes.Schindelin@gmx.de>
Cc: Adam Brewster <adambrewster@gmail.com>,
git@vger.kernel.org, Junio C Hamano <gitster@pobox.com>,
Jakub Narebski <jnareb@gmail.com>
Subject: Re: [PATCH/v2] git-basis, a script to manage bases for git-bundle
Date: Fri, 04 Jul 2008 12:47:09 -0400 [thread overview]
Message-ID: <486E540D.8000008@gmail.com> (raw)
In-Reply-To: <alpine.DEB.1.00.0807040237580.2849@eeepc-johanness>
Johannes Schindelin wrote:
> I am not convinced. This sort of feature belongs into git-bundle. It
> certainly does not deserve being blessed by yet-another git-* command,
> when we are constantly being bashed for having _way_ too many _already_.
>
> Ciao,
> Dscho
>
>
Actually, I would like to see "normal" interface for git-bundle handled
by git-push, git-fetch, and git-remote. We fixed the retrieve side to
use git-fetch before first integration, but didn't understand the
semantics for creation well-enough to put into git-push. Right now, we
can do "git remote add bundle-nick <path-to-bundle>" and set up the remote.
We should have "git push bundle-nick" create the new bundle, updating
the basis refs kept somewhere in refs/* (possibly refs/remotes, possibly
refs/bundles?).
However, we need two helpers to maintain the basis refs, both I believe
should be sub-commands of git-remote:
- a "rewind" function to roll the refs back to a previous state because
the bundle didn't get applied, whatever. This is well supported by
reflogs, is "expire anything *newer* than time", and for convenience
should apply to all refs for the given remote so the user doesn't have
to invoke per branch on the remote. e.g., "git remote rewind bundle-nick
3.days.ago".
- a "prune" function to remove any branch for the remote that is not
known to the local refs/heads/* hierarchy. This is needed to support
cleaning up pruned topic branches. Could be a special behavior of "git
remote prune" triggered by the remote being a bundle, but that might be
confusing a perhaps need a new sub-command name. Perhaps, "git remote
prune-non-local bundle-nick"
If we did the above, then git-bundle can be relegated to plumbing and
bundles become better integrated to the porcelain.
Mark
next prev parent reply other threads:[~2008-07-04 16:48 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <1214272713-7808-1-git-send-email-adambrewster@gmail.com>
2008-06-30 22:49 ` [PATCH/v2] git-basis, a script to manage bases for git-bundle Adam Brewster
2008-07-01 9:51 ` Jeff King
2008-07-02 1:36 ` Adam Brewster
2008-07-02 2:10 ` Jay Soffian
2008-07-02 2:16 ` Adam Brewster
2008-07-02 2:21 ` Jay Soffian
2008-07-02 3:21 ` Jeff King
2008-07-02 9:44 ` Jakub Narebski
2008-07-03 19:59 ` Jeff King
2008-07-03 23:38 ` Adam Brewster
2008-07-04 0:44 ` Johannes Schindelin
2008-07-04 2:04 ` Adam Brewster
2008-07-04 16:47 ` Mark Levedahl [this message]
2008-07-04 20:55 ` Jakub Narebski
2008-07-04 19:51 ` Jeff King
2008-07-01 23:55 ` Junio C Hamano
2008-07-02 0:16 ` Mark Levedahl
2008-07-03 23:13 ` Adam Brewster
2008-07-04 13:14 ` Mark Levedahl
2008-07-04 13:22 ` Johannes Schindelin
2008-07-04 13:49 ` Mark Levedahl
2008-07-02 2:12 ` Adam Brewster
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=486E540D.8000008@gmail.com \
--to=mlevedahl@gmail.com \
--cc=Johannes.Schindelin@gmx.de \
--cc=adambrewster@gmail.com \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=jnareb@gmail.com \
/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).