"Philippe Bruhat (BooK)" <book@xxxxxxxx> writes: > On Thu, Feb 28, 2008 at 12:07:46PM -0800, Junio C Hamano wrote: > >> I'll take a look at them later, when I have enough time to fish >> for messages and discussions from earlier round in the list >> archive in order to process this. > > You can ignore my previous messages, then. These four patches were > rebased on the top of master, and correct the problems that were > previously pointed to me. I do not work that way. I am a trust-but-verify kind of person. So I dug up the old ones and the discussion. The series looks fine. cvsimport: have default merge regex allow for dashes in the branch name You can extend the default with -M anyway, but I guess the default pattern can be loosened like this without increasing the risk of false hits, so probably it is Ok. cvsimport: allow for multiple -M options Ok. cvsimport: document that -M can be used multiple times Ok, except that "can be used seveval times" should probably be "can be used more than once". cvsimport: configure Getopt::Long to bundle options Ok. Why isn't bundling the default, I have to wonder... -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
