From: Scott Weaver <scwea...@redhat.com>

DO NOT MERGE: test

Signed-off-by: Scott Weaver <scwea...@redhat.com>

diff --git a/redhat/scripts/bundle_bindgen.sh b/redhat/scripts/bundle_bindgen.sh
index blahblah..blahblah 100755
--- a/redhat/scripts/bundle_bindgen.sh
+++ b/redhat/scripts/bundle_bindgen.sh
@@ -7,20 +7,21 @@
 # currently only packaged in Fedora/ELN. In order to build kernel-ark
 # in brew we will temporarily need to build bindgen in the the kernel
 # build until the bindgen package is added.
-
+set -x
 SOURCES=$1
+SRC_TMP=$(mktemp -d)
 
 BINDGEN_CLI=bindgen-cli
 BINDGEN_CLI_VERSION="0.71.1"
 BINDGEN_CLI_CRATE=bindgen-cli.crate
 
CRATESIO_API_ENDPOINT=https://crates.io/api/v1/crates/bindgen-cli/${BINDGEN_CLI_VERSION}/download
 
-curl -sL $CRATESIO_API_ENDPOINT -o "$SOURCES/$BINDGEN_CLI_CRATE"
-tar -xf "$SOURCES/$BINDGEN_CLI_CRATE" -C "$SOURCES"
-mv "$SOURCES/$BINDGEN_CLI-$BINDGEN_CLI_VERSION" "$SOURCES/$BINDGEN_CLI"
+curl -sL $CRATESIO_API_ENDPOINT -o "$SRC_TMP/$BINDGEN_CLI_CRATE"
+tar -xf "$SRC_TMP/$BINDGEN_CLI_CRATE" -C "$SRC_TMP"
+mv "$SRC_TMP/$BINDGEN_CLI-$BINDGEN_CLI_VERSION" "$SRC_TMP/$BINDGEN_CLI"
 
 # vendor bindgen-cli
-cd "$SOURCES/$BINDGEN_CLI" || exit 1
+cd "$SRC_TMP/$BINDGEN_CLI" || exit 1
 mkdir .cargo 
 cat > .cargo/config.toml <<EOF
 [source.crates-io]
@@ -34,7 +35,4 @@ cargo vendor --locked --quiet
 
 cd ..
 tar czf $BINDGEN_CLI.tar.gz $BINDGEN_CLI
-
-# clean up
-rm -f "$SOURCES/$BINDGEN_CLI_CRATE"
-rm -rf "${SOURCES:?}/$BINDGEN_CLI"
+mv $SRC_TMP/$BINDGEN_CLI.tar.gz $SOURCES

--
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/3880

-- 
_______________________________________________
kernel mailing list -- kernel@lists.fedoraproject.org
To unsubscribe send an email to kernel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org
Do not reply to spam, report it: 
https://pagure.io/fedora-infrastructure/new_issue

Reply via email to