Re: [PATCH 1/3] pathspec: catch prepending :(prefix) on pathspec with short magic

2013-09-05 Thread Junio C Hamano
Nguyễn Thái Ngọc Duy writes: > :(prefix) is in the long form. Suppose people pass :!foo with '!' > being the short form of magic 'bar', the code will happily turn it to > :(prefix..)!foo, which makes '!' part of the path and no longer a magic. > > The correct form must be ':(prefix..,bar)foo', b

[PATCH 1/3] pathspec: catch prepending :(prefix) on pathspec with short magic

2013-09-04 Thread Nguyễn Thái Ngọc Duy
:(prefix) is in the long form. Suppose people pass :!foo with '!' being the short form of magic 'bar', the code will happily turn it to :(prefix..)!foo, which makes '!' part of the path and no longer a magic. The correct form must be ':(prefix..,bar)foo', but as so far we haven't had any magic in