From: Pavel Roskin <proski@gnu.org>
To: Yann Dirson <ydirson@altern.org>
Cc: Catalin Marinas <catalin.marinas@gmail.com>,
git <git@vger.kernel.org>, Charles Lever <cel@citi.umich.edu>
Subject: Re: StGIT: "stg new" vs "stg new --force"
Date: Fri, 20 Jan 2006 01:23:44 -0500 [thread overview]
Message-ID: <1137738224.27911.26.camel@dv> (raw)
In-Reply-To: <20060119213838.GA27397@nowhere.earth>
On Thu, 2006-01-19 at 22:38 +0100, Yann Dirson wrote:
> > I believe StGIT should be primarily designed to be used interactively.
>
> I tend to disagree. 3rd-party apps like qgit should be able to let
> the user do the selection, and tell the command-line tool what they
> want to act upon. It may be useful as well if stg gains
> interactivity, but should not become the only way to work with it
> (unix way of life).
In this particular case, the GUI frontend should be able to supply much
more data than the hunk numbers. It would be much safer.
I didn't mean to say that StGIT shouldn't have any options for
non-interactive processing. I meant that whenever adding a feature, we
should try to make it immediately useful without any frontends or
additional software.
> Maybe we could have a "fold" or "refresh" variant that takes its
> output from the output of "stg diff" (or any arbitrary diff on stdin,
> to be friendly with GUI wrappers) filtered by an arbitrary command.
That would be "fold". "refresh" like is saying - this patch should
produce this file. I actually tend to think now that deep refresh would
be confusing and dangerous if there is another patch closer in the stack
working with the same file, even in a separate part of it. stg would
either report conflict with the higher patches use the file with the
higher patches reverted, and neither sounds good to me.
Deep fold of the local changes would be much easier, since no other
patch should have them.
> That command could be "cat" to get the current "refresh" behaviour, or
> an editor wrapper acting on stdin/out, or a wrapper to filterdiff, or
> whatever clever filter one would want to use.
>
> Does it sound better ?
Yes. The first step would be to fix "stg refresh --edit --showpatch" to
actually respect edits made to the patch.
--
Regards,
Pavel Roskin
next prev parent reply other threads:[~2006-01-20 6:24 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-01-13 9:24 StGIT: "stg new" vs "stg new --force" Pavel Roskin
2006-01-13 9:34 ` Karl Hasselström
2006-01-16 8:18 ` Catalin Marinas
2006-01-17 17:01 ` Pavel Roskin
2006-01-17 21:57 ` Yann Dirson
2006-01-17 23:16 ` Pavel Roskin
2006-01-18 19:37 ` Yann Dirson
2006-01-19 0:49 ` Pavel Roskin
2006-01-19 21:38 ` Yann Dirson
2006-01-20 6:23 ` Pavel Roskin [this message]
2006-01-20 18:22 ` J. Bruce Fields
2006-01-24 5:30 ` Pavel Roskin
2006-01-24 17:54 ` J. Bruce Fields
2006-01-24 18:17 ` Pavel Roskin
2006-01-24 21:23 ` Catalin Marinas
2006-01-21 18:24 ` Catalin Marinas
2006-01-22 5:05 ` Pavel Roskin
2006-01-21 18:20 ` Catalin Marinas
2006-01-21 18:31 ` Catalin Marinas
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=1137738224.27911.26.camel@dv \
--to=proski@gnu.org \
--cc=catalin.marinas@gmail.com \
--cc=cel@citi.umich.edu \
--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).