Thread (7 messages) flat view 7 messages, 3 authors, 2016-06-15

Re: [PATCH] Handle branch names with slashes

From: Karl Hasselström <hidden>
Date: 2016-06-15 22:42:20

On 2006-02-17 05:21:08 +0100, Karl Hasselström wrote:
On 2006-02-17 16:01:10 +1300, Sam Vilain wrote:
quoted
Karl Hasselström wrote:
quoted
Let StGIT grok branch names with slashes in them. It used to
fall flat on its face when confronted with them.

I think I've covered all, or at least most cases, but there are
probably some bugs left if you look hard enough.
Does `stgit -r patchname/bottom` still work?
Yes (if you mean 'stg diff -r ... ' :-). It's just branches that can
have slashes in their names, not patches.
There was a bug here after all: I just tried "stg pick
multi@kha/patches" (to pick a patch named "multi" from the branch
"kha/patches"), and StGIT tried to pick the patch from branch "kha".

Looking closer, I realized that the complete patch specification
syntax is "patchname@branchname/bottom", not
"patchname/bottom@branchname" as I had assumed. This is obviously hard
to reconcile with branch names containing /.

Thoughts?

-- 
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