Use -r (--no-run-if-empty) to xargs
authorFredrik Fornwall <fredrik@fornwall.net>
Thu, 11 Aug 2016 13:32:56 +0000 (09:32 -0400)
committerFredrik Fornwall <fredrik@fornwall.net>
Thu, 11 Aug 2016 13:32:56 +0000 (09:32 -0400)
This avoids false error messages when building packages without
ELF files.

Fixes #373.

build-package.sh

index 626fbec..bed59bb 100755 (executable)
@@ -561,7 +561,7 @@ termux_step_massage () {
         find . -exec chmod u+w,o-rwx \{\} \;
        # .. strip binaries (setting them as writeable first)
        if [ "$TERMUX_DEBUG" = "" ]; then
-                find . -type f | xargs file | grep -E "(executable|shared object)" | grep ELF | cut -f 1 -d : | xargs $STRIP --strip-unneeded --preserve-dates -R '.gnu.version*'
+                find . -type f | xargs -r file | grep -E "(executable|shared object)" | grep ELF | cut -f 1 -d : | xargs -r $STRIP --strip-unneeded --preserve-dates -R '.gnu.version*'
        fi
         # Fix shebang paths:
         for file in `find -L . -type f`; do
@@ -570,7 +570,7 @@ termux_step_massage () {
        set -e -o pipefail
         # Remove DT_ entries which the android 5.1 linker warns about:
        if [ "$TERMUX_DEBUG" = "" ]; then
-               find . -type f -print0 | xargs -0 $TERMUX_ELF_CLEANER
+               find . -type f -print0 | xargs -r -0 $TERMUX_ELF_CLEANER
        fi
 
        test ! -z "$TERMUX_PKG_RM_AFTER_INSTALL" && rm -Rf $TERMUX_PKG_RM_AFTER_INSTALL