On Wed, Jan 18, 2006 at 07:49:09PM -0500, Pavel Roskin wrote: > > But it would not be a replacement to selecting changes with a > > granularity finer than file-level, which is what I wanted to suggest. > > Why? Maybe you got confused by two meanings of the word "patch"? I > think StGIT should use some other term, e.g. changeset. I meant that > the diff file (e.g. made by "stg diff") could be edited and folded into > one of the StGIT patches (changesets). Unless you want non-interactive > separation of the hunks, using an editor should be a reasonable > approach. > My approach suggests that the user work with the diff from the > beginning, and separates the changes by looking at them. Oh, I think I now see what you meant. I'm not sure it would be a good idea to let the user edit the diff, although running an editor on it is not much different from feeding it through filterdiff, or a even through a graphical equivalent allowing to point at the hunks. > 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). > Your approach looks like a usability disaster to me. The user is > supposed to find numbers of the hunks, although s/he is working on the > whole file (since it's "stg refresh"). That's what I'm currently doing to feed the hunk numbers to filterdiff. Any improvement in the process would be welcomed :) 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 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 ? -- Yann Dirson <ydirson@altern.org> | Debian-related: <dirson@debian.org> | Support Debian GNU/Linux: | Freedom, Power, Stability, Gratis http://ydirson.free.fr/ | Check <http://www.debian.org/> - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.htmlReceived on Fri Jan 20 08:36:28 2006
This archive was generated by hypermail 2.1.8 : 2006-01-20 08:36:36 EST