[Skiboot] [PATCH] make check: Use mktemp instead of tempfile

Patrick Williams patrick at stwcx.xyz
Tue Jul 19 02:41:03 AEST 2016


On Sat, Jul 16, 2016 at 11:49:30PM +0530, Vasant Hegde wrote:
> Commit 2ceb8b8c (make check: make test runs less noisy) used tempfile
> command to create temporary files. Looks like distros like Fedora
> doesn't have tempfile command.
> 
> Looks like mktemp is supported on all distros (at least Fedora and
> Ubuntu supports this command). Hence replace tempfile with mktemp.
> 
> Output without patch:
>   [ CLEANUP ]   gard-test-clean
>   ./test/run.sh: line 3: tempfile: command not found
>   skiboot/external/gard/test/Makefile.check:18: recipe for target 'gard-test-clean' failed
> 	make: *** [gard-test-clean] Error 1
> 
> Signed-off-by: Vasant Hegde <hegdevasant at linux.vnet.ibm.com>
> CC: Stewart Smith <stewart at linux.vnet.ibm.com>
> ---
>  test/run.sh | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/test/run.sh b/test/run.sh
> index 5185f5a..e159bd3 100755
> --- a/test/run.sh
> +++ b/test/run.sh
> @@ -1,6 +1,6 @@
>  #!/bin/bash
>  
> -t=$(tempfile) || exit 1
> +t=$(mktemp) || exit 1

Confirmed RHEL6 also has 'mktemp'.

Reviewed-by: Patrick Williams <stwcx.xyz>

>  
>  trap "rm -f -- '$t'" EXIT
>  
> -- 
> 2.5.5
> 
> _______________________________________________
> Skiboot mailing list
> Skiboot at lists.ozlabs.org
> https://lists.ozlabs.org/listinfo/skiboot

-- 
Patrick Williams
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.ozlabs.org/pipermail/skiboot/attachments/20160718/59c314e1/attachment.sig>


More information about the Skiboot mailing list