Thread (51 messages) flat view 51 messages, 5 authors, 2016-06-15

Re: [StGIT PATCH] Don't use patches/<branch>/current

From: Karl Hasselström <hidden>
Date: 2016-06-15 22:43:10

On 2007-05-15 16:56:33 +0100, Catalin Marinas wrote:
On 06/05/07, Karl Hasselström [off-list ref] wrote:
quoted
The name of the current patch, if any, is always the last line of
patches/<branch>/applied (and there is no current patch if and
only if the "applied" file is empty). So use that instead, and
stop having to worry about keeping the redundant "current" file
up-to-date.
I applied this patch. Could you also send me a patch for the
bash-completion script as it uses this file?
I realized this myself yesterday or so, and patched it to not need the
current, applied, and unapplied files. Are you OK with that patch, or
would you like one that keeps using {,un}applied?
I think the self.__current_file (same for the base file removed in a
different patch) should still be available in the Series object and
removed when deleting a branch, otherwise you get a "Series
directory ... is not empty" exception.
Ah, very true. I'll whip up a fix.

Same question there: are you OK with a single fix for base, current,
applied, and unapplied, or do you want them separate?

-- 
Karl Hasselström, kha@treskal.com
      www.treskal.com/kalle
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help