Re: [PATCH 3/3] [test] Use UTF-8 in test files

Bryce Harrington <[email protected]>
Newsgroups gmane.comp.lib.cairo
Message-ID <[email protected]>
On Tue, Mar 10, 2015 at 12:10:36PM +0100, Andrea Canciani wrote:
> From: Andrea Canciani <[email protected]>
> 
> On MacOSX, the sed utility errors out when parsing non-UTF8
> files. Because of this, the generated cairo-test-constructor only
> contained a few tests and the test suite was thus incomplete.
> ---
>  test/composite-integer-translate-over.c | 2 +-
>  test/degenerate-dash.c                  | 2 +-
>  test/degenerate-path.c                  | 2 +-
>  test/long-dashed-lines.c                | 2 +-
>  test/reflected-stroke.c                 | 2 +-
>  test/user-font-rescale.c                | 4 ++--
>  6 files changed, 7 insertions(+), 7 deletions(-)

Hrm, this patch isn't applying for me.  

error: patch failed: test/user-font-rescale.c:22
Applying patch test/composite-integer-translate-over.c with 1 reject...
Rejected hunk #1.
Applying patch test/degenerate-dash.c with 1 reject...
Rejected hunk #1.
Applying patch test/degenerate-path.c with 1 reject...
Rejected hunk #1.
Applying patch test/long-dashed-lines.c with 1 reject...
Rejected hunk #1.
Applying patch test/reflected-stroke.c with 1 reject...
Rejected hunk #1.
Applying patch test/user-font-rescale.c with 2 rejects...
Rejected hunk #1.
Rejected hunk #2.
Patch failed at 0001 Use UTF-8 in test files

Guessing it to be some sort of platform specific encoding issue.

$ FILES=$(file *.c | grep -v UTF-8 | grep -v ASCII | cut -d: -f1)
$ file $FILES
composite-integer-translate-over.c:        C source, ISO-8859 text
degenerate-dash.c:                         C source, ISO-8859 text
degenerate-path.c:                         C source, ISO-8859 text
long-dashed-lines.c:                       C source, ISO-8859 text
reflected-stroke.c:                        C source, ISO-8859 text
user-font-rescale.c:                       C source, ISO-8859 text

$ for file in $FILES; do iconv -f iso-8859-1 -t utf-8 $file > $file.new; mv $file.new $file; done
$ file $FILES
composite-integer-translate-over.c: C source, UTF-8 Unicode text
degenerate-dash.c:                  C source, UTF-8 Unicode text
degenerate-path.c:                  C source, UTF-8 Unicode text
long-dashed-lines.c:                C source, UTF-8 Unicode text
reflected-stroke.c:                 C source, UTF-8 Unicode text
user-font-rescale.c:                C source, UTF-8 Unicode text

Hopefully that works for you; if not please follow up.

Bryce

> diff --git a/test/composite-integer-translate-over.c b/test/composite-integer-translate-over.c
> index 4f89fa8..1b82a33 100644
> --- a/test/composite-integer-translate-over.c
> +++ b/test/composite-integer-translate-over.c
> @@ -1,5 +1,5 @@
>  /*
> - * Copyright ? 2006 Mozilla Corporation
> + * Copyright © 2006 Mozilla Corporation
>   *
>   * Permission to use, copy, modify, distribute, and sell this software
>   * and its documentation for any purpose is hereby granted without
> diff --git a/test/degenerate-dash.c b/test/degenerate-dash.c
> index 12cd0c7..6e7bf6b 100644
> --- a/test/degenerate-dash.c
> +++ b/test/degenerate-dash.c
> @@ -1,5 +1,5 @@
>  /*
> - * Copyright ? 2008 Chris Wilson
> + * Copyright © 2008 Chris Wilson
>   *
>   * Permission to use, copy, modify, distribute, and sell this software
>   * and its documentation for any purpose is hereby granted without
> diff --git a/test/degenerate-path.c b/test/degenerate-path.c
> index db53cbb..aa5f822 100644
> --- a/test/degenerate-path.c
> +++ b/test/degenerate-path.c
> @@ -1,5 +1,5 @@
>  /*
> - * Copyright ? 2006 Jeff Muizelaar
> + * Copyright © 2006 Jeff Muizelaar
>   *
>   * Permission to use, copy, modify, distribute, and sell this software
>   * and its documentation for any purpose is hereby granted without
> diff --git a/test/long-dashed-lines.c b/test/long-dashed-lines.c
> index 215bd91..4d6ded9 100644
> --- a/test/long-dashed-lines.c
> +++ b/test/long-dashed-lines.c
> @@ -1,5 +1,5 @@
>  /*
> - * Copyright ? 2008 Chris Wilson
> + * Copyright © 2008 Chris Wilson
>   *
>   * Permission to use, copy, modify, distribute, and sell this software
>   * and its documentation for any purpose is hereby granted without
> diff --git a/test/reflected-stroke.c b/test/reflected-stroke.c
> index b11f5a9..895396a 100644
> --- a/test/reflected-stroke.c
> +++ b/test/reflected-stroke.c
> @@ -1,5 +1,5 @@
>  /*
> - * Copyright ? 2008 Chris Wilson
> + * Copyright © 2008 Chris Wilson
>   *
>   * Permission to use, copy, modify, distribute, and sell this software
>   * and its documentation for any purpose is hereby granted without
> diff --git a/test/user-font-rescale.c b/test/user-font-rescale.c
> index 3fe70f6..6f03b8e 100644
> --- a/test/user-font-rescale.c
> +++ b/test/user-font-rescale.c
> @@ -1,5 +1,5 @@
>  /*
> - * Copyright ? 2008 Jeff Muizelaar
> + * Copyright © 2008 Jeff Muizelaar
>   *
>   * Permission to use, copy, modify, distribute, and sell this software
>   * and its documentation for any purpose is hereby granted without
> @@ -22,7 +22,7 @@
>   *
>   * Contributor(s):
>   *	Jeff Muizelaar <[email protected]>
> - *	Kristian H?gsberg <[email protected]>
> + *	Kristian Høgsberg <[email protected]>
>   *	Behdad Esfahbod <[email protected]>
>   */
>  
> -- 
> 1.9.3 (Apple Git-50)
> 

> -- 
> cairo mailing list
> [email protected]
> http://lists.cairographics.org/mailman/listinfo/cairo

-- 
cairo mailing list
[email protected]
http://lists.cairographics.org/mailman/listinfo/cairo
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.