[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH V2] .gitignore: Add vim swap files

On 10/05/13 19:09, Ian Jackson wrote:
> Andrew Cooper writes ("Re: [PATCH V2] .gitignore: Add vim swap files"):
>> On 09/05/13 17:17, Julien Grall wrote:
>>> Signed-off-by: Julien Grall <julien.grall@xxxxxxxxxx>
>>> Changes in v2:
>>>     - update .hgignore
>> Much appreciated.
>> Reviewed-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
> Nacked-by: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>
>>> diff --git a/.hgignore b/.hgignore
>>> index 6b432f7..4bd24fc 100644
>>> --- a/.hgignore
>>> +++ b/.hgignore
>>> @@ -11,6 +11,7 @@
>>>  .*\.tar\.bz2$
>>>  .*\.tar\.gz$
>>>  .*~$
>>> +.swp$
> AIUI the format of .hgignore files is lines containing
> implicitly-^-anchored regexps.

Nope - the documentation states otherwise


"Neither glob nor regexp patterns are rooted. A glob-syntax pattern of
the form *.c will match a file ending in .c in any directory, and a
regexp pattern of the form \.c$ will do the same. To root a regexp
pattern, start it with ^."

> So this is wrong.  Please copy the (hideous) syntax of the other
> entries.

It is wrong because it failes to escape the literal . (which admittedly
I missed when reviewing it)

Just because the rest of the .hgignore is wrong doesn't mean new
additions shouldn't be.


> Furthermore, while it's good to update both ignore files I think it's
> fine to update only one of them if that's what's annoying you.
> Ian.

Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.