[PATCH STABLE] test-cache-abuse: correct for different hunk headers between Solaris and GNU

Augie Fackler raf at durin42.com
Sat May 28 17:48:16 EDT 2016


On Fri, May 27, 2016 at 11:27:55AM -0700, danek.duvall at oracle.com wrote:
> # HG changeset patch
> # User Danek Duvall <danek.duvall at oracle.com>
> # Date 1464372869 25200
> #      Fri May 27 11:14:29 2016 -0700
> # Branch stable
> # Node ID b1ffda6b55ce3f920faec1d553c9fafafb4d8c9c
> # Parent  89bba2beb03ea62e7fc8bcf3272e3cda1065ad89
> test-cache-abuse: correct for different hunk headers between Solaris and GNU
>
> When diffing against an empty file, Solaris diff uses 1 to designate the
> first line of the empty file (either -1,0 on the left or +1,0 on the right)
> while GNU diff uses 0 (-0,0 and +0,0).  We use a glob here to make sure the
> test passes with either toolchain.
>
> I've not added tests to check-code because there are scads of places in the
> tests where the GNU format is used due to that being the format that "hg
> diff" and "hg export" use, and changing those to use globs seems wrong.

Sounds reasonable. Maybe coordinate with Kevin about getting a Solaris
buildbot slave set up so we can notice these proactively?

>
> diff --git a/tests/test-cache-abuse.t b/tests/test-cache-abuse.t
> --- a/tests/test-cache-abuse.t
> +++ b/tests/test-cache-abuse.t
> @@ -88,7 +88,7 @@ We currently can't detect an rbc cache w
>    $ damage "log -qr branch(b2)" rbc-names-v1
>    --- before	* (glob)
>    +++ after	* (glob)
> -  @@ -1,8 +0,0 @@
> +  @@ -1,8 +?,0 @@ (glob)
>    -2:5fb7d38b9dc4
>    -3:60b597ffdafa
>    -4:b1174d11b69e
> _______________________________________________
> Mercurial-devel mailing list
> Mercurial-devel at mercurial-scm.org
> https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel


More information about the Mercurial-devel mailing list