Re: [platform-testers] new snapshot available: coreutils-8.30.79-efa3e.tar.xz

[Eric Blake]

On 3/6/19 11:09 PM, Pádraig Brady wrote:

> * tests/id/zero.sh: sed on OSX will output a \n even
> if the input doesn't have a \n on the last "line".
> So ensure we always have a trailing '\n' to avoid the disparity.
> ---
>  tests/id/zero.sh | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/tests/id/zero.sh b/tests/id/zero.sh
> index 085ea7c..471962c 100755
> --- a/tests/id/zero.sh
> +++ b/tests/id/zero.sh
> @@ -89,13 +89,15 @@ for o in G Gr ; do
>    for n in '' n ; do
>      id -${o}${n}  $users >> gtmp1 ||
>        { test $? -ne 1 || test -z "$n" && fail=1; }
> +    printf '\n' >> gtmp1 || framework_failure_

Worth spelling it "echo" instead of the more verbose "printf '\n'"?

-- 
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3226
Virtualization:  qemu.org | libvirt.org

signature.asc
Description: OpenPGP digital signature