[apparmor] [patch] fix utils/test/runtests-py*.sh exitcode

Seth Arnold seth.arnold at canonical.com
Thu Oct 2 22:36:22 UTC 2014


On Thu, Oct 02, 2014 at 10:48:50PM +0200, Christian Boltz wrote:
> Hello,
> 
> utils/test/runtests-py*.sh always exits with $? = 1 even if there is no 
> error. This is caused by the last executed command, test -n
> 
> This patch changes it to test -z so that we'll get $? = 0 if all tests 
> succeed.

Acked-by: Seth Arnold <seth.arnold at canonical.com>

Thanks

> 
> === modified file 'utils/test/runtests-py3.sh'
> --- utils/test/runtests-py3.sh  2014-07-22 19:25:25 +0000
> +++ utils/test/runtests-py3.sh  2014-10-02 20:37:28 +0000
> @@ -24,7 +24,7 @@
>         echo
>  done
>  
> -test -n "$failed" && {
> +test -z "$failed" || {
>         echo
>         echo "*** The following tests failed:"
>         echo "***   $failed"
> 
> 
> 
> 
> 
> Regards,
> 
> Christian Boltz
> -- 
> I've not had any luck reading the manpage and making SWAGs
> (Scientific Wild Ass Guesses - Trademark -- Amdahl Corporation).
> [Sid Boyce in opensuse-factory]
> 
> 
> -- 
> AppArmor mailing list
> AppArmor at lists.ubuntu.com
> Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/apparmor
> 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: Digital signature
URL: <https://lists.ubuntu.com/archives/apparmor/attachments/20141002/9b744b0b/attachment-0001.pgp>


More information about the AppArmor mailing list