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

Re: [PATCH] git-cvsimport: strip question-mark characters in tags

From: Junio C Hamano <hidden>
Date: 2016-06-15 22:48:37

Ed Santiago [off-list ref] writes:
Question mark character appears to be valid in a CVS tag,
but not a git one.  Remove it.  Leave open the possibility that there may
be more such characters; and comment (FIXME) that we may want to replace
those instead of removing them.
I agree that people may want to optionally replace them to avoid mapping
two originally different tags into the same one.

Does your regexp chain check other invalid refname sequences, such as ".name",
"na..me", etc.?
Also: if git tag command fails, do not include $! in our
error message: it is not useful after system(), and will
only serve as a red herring.
---
Sign-off?
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help