diff mbox series

Use -c instead of check for sha512sum

Message ID 20190309114807.30954-1-remyabel@gmail.com
State New
Headers show
Series Use -c instead of check for sha512sum | expand

Commit Message

Tommy Nguyen March 9, 2019, 11:48 a.m. UTC
Busybox doesn't support "check", which is an alias for "-c" in
coreutils. Tested on Alpine Edge and Fedora 29.

2019-03-09  Tommy Nguyen  <remyabel@gmail.com>

    PR contrib/82704
    * contrib/download_prerequisites: Use -c instead of check for
    sha512sum
---
 contrib/download_prerequisites | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Segher Boessenkool March 10, 2019, 12:51 p.m. UTC | #1
Hi Tommy,

On Sat, Mar 09, 2019 at 06:48:07AM -0500, Tommy Nguyen wrote:
> Busybox doesn't support "check", which is an alias for "-c" in
> coreutils. Tested on Alpine Edge and Fedora 29.
> 
> 2019-03-09  Tommy Nguyen  <remyabel@gmail.com>
> 
>     PR contrib/82704
>     * contrib/download_prerequisites: Use -c instead of check for
>     sha512sum

I've committed this for you (as trivial and obviously correct) (after
fixing the changelog a bit).  Thanks for the contribution!


Segher
diff mbox series

Patch

diff --git a/contrib/download_prerequisites b/contrib/download_prerequisites
index b50f47cda79..72976c46c92 100755
--- a/contrib/download_prerequisites
+++ b/contrib/download_prerequisites
@@ -51,7 +51,7 @@  case $OS in
     chksum='shasum -a 512 --check'
   ;;
   *)
-    chksum='sha512sum --check'
+    chksum='sha512sum -c'
   ;;
 esac