From mboxrd@z Thu Jan  1 00:00:00 1970
From: Andrew Cooper <andrew.cooper3@citrix.com>
Subject: Re: [PATCH V2] .gitignore: Add vim swap files
Date: Fri, 10 May 2013 19:15:30 +0100
Message-ID: <518D3942.4060208@citrix.com>
References: <1368116223-31218-1-git-send-email-julien.grall@linaro.org>
	<518BCDCA.7030902@citrix.com>
	<20877.14289.660009.193592@mariner.uk.xensource.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
In-Reply-To: <20877.14289.660009.193592@mariner.uk.xensource.com>
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: Ian Jackson <Ian.Jackson@eu.citrix.com>
Cc: Julien Grall <julien.grall@linaro.org>, "xen-devel@lists.xensource.com" <xen-devel@lists.xensource.com>, "patches@linaro.org" <patches@linaro.org>
List-Id: xen-devel@lists.xenproject.org

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@linaro.org>
>>>
>>> Changes in v2:
>>>     - update .hgignore
>> Much appreciated.
>>
>> Reviewed-by: Andrew Cooper <andrew.cooper3@citrix.com>
> Nacked-by: Ian Jackson <ian.jackson@eu.citrix.com>
>
>>> 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

http://www.selenic.com/mercurial/hgignore.5.html

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

~Andrew

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