git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Catalin Marinas" <catalin.marinas@gmail.com>
To: "Yann Dirson" <ydirson@altern.org>
Cc: git@vger.kernel.org
Subject: Re: StGIT rebasing safeguard
Date: Tue, 19 Jun 2007 23:12:30 +0100	[thread overview]
Message-ID: <b0943d9e0706191512l5f19b80v5b5be9151be13025@mail.gmail.com> (raw)
In-Reply-To: <20070614213032.GR6992@nan92-1-81-57-214-146.fbx.proxad.net>

On 14/06/07, Yann Dirson <ydirson@altern.org> wrote:
> When the parent branch is a rewinding one (eg. an stgit stack), then
> the old version of the patch will be turned to unreachable by
> pull/rebase, and we probably have even no way of telling stgit that it
> is indeed expected, since the parent stack is a local one.  My own
> workflow on StGIT is affected by the issue, since my "bugs" stack is
> forked off my "master" stack (but hopefully an hydra will help me ;).

If I understand correctly, is this the case where you do a 'stg
commit'? This command is meant for branches that are never rebased
(i.e. my master stgit branch). For this branch one wouldn't have a
remote branch configured and hence git fetch shouldn't do anything.

> That makes me suspecting the reachability approach is a dead-end, and
> we should either get back to the approach of recording old-base, or
> find another solution.

Maybe defaulting to 'git pull' and only use 'git fetch' if explicitly
asked. A git-pull would still keep the old patch accessible from HEAD.

-- 
Catalin

  reply	other threads:[~2007-06-19 22:12 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-06-11  0:33 [StGIT PATCH 0/4] Cleanups of the Series object Yann Dirson
2007-06-11  0:33 ` [StGIT PATCH 1/4] Use get/set_name for a stack's name Yann Dirson
2007-06-11  0:33 ` [StGIT PATCH 2/4] Abstract a PatchSet object out of Series Yann Dirson
2007-06-12 22:09   ` Catalin Marinas
2007-06-13 20:38     ` Yann Dirson
2007-06-14 21:30       ` StGIT rebasing safeguard Yann Dirson
2007-06-19 22:12         ` Catalin Marinas [this message]
2007-06-19 22:18           ` Catalin Marinas
2007-06-19 22:37             ` Yann Dirson
2007-06-14 22:56       ` [StGIT PATCH 2/4] Abstract a PatchSet object out of Series Catalin Marinas
2007-06-15 20:14         ` Yann Dirson
2007-06-19 22:32           ` Catalin Marinas
2007-06-11  0:33 ` [StGIT PATCH 3/4] Move stack format-version checks into Stack class Yann Dirson
2007-06-11  0:33 ` [StGIT PATCH 4/4] Rename branch.*.stgitformatversion to branch.*.stgit.stackformatversion Yann Dirson

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=b0943d9e0706191512l5f19b80v5b5be9151be13025@mail.gmail.com \
    --to=catalin.marinas@gmail.com \
    --cc=git@vger.kernel.org \
    --cc=ydirson@altern.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).