Skip to content

Commit

Permalink
Update moderncppkafka.sh
Browse files Browse the repository at this point in the history
  • Loading branch information
Barthelemy authored Jul 4, 2024
1 parent 0399f64 commit 70fec67
Showing 1 changed file with 5 additions and 10 deletions.
15 changes: 5 additions & 10 deletions moderncppkafka.sh
Original file line number Diff line number Diff line change
Expand Up @@ -5,20 +5,15 @@ requires:
- librdkafka
source: https://github.com/morganstanley/modern-cpp-kafka.git
---
#!/bin/bash -e
#!/bin/bash -e

Check notice on line 8 in moderncppkafka.sh

View workflow job for this annotation

GitHub Actions / alidistlint

Invalid script shebang. Use exactly "#!/bin/bash -e" to match aliBuild environment. You may see spurious errors until you fix the shebang. [ali:bad-shebang]

# this is header only library, so we can just copy it
# this is header only library, so we can just copy it
mkdir -p "$INSTALLROOT"
cp -r "$SOURCEDIR/include" "$INSTALLROOT/include"
rm "$INSTALLROOT/include/CMakeLists.txt" # for some reason it is there

# Modulefile
MODULEDIR="$INSTALLROOT/etc/modulefiles"
MODULEFILE="$MODULEDIR/$PKGNAME"

# Modulefile
mkdir -p "etc/modulefiles"
alibuild-generate-module --lib > "etc/modulefiles/$PKGNAME"

cat << EOF >> etc/modulefiles/$PKGNAME
prepend-path ROOT_INCLUDE_PATH \$PKG_ROOT/include
EOF
mkdir -p $INSTALLROOT/etc/modulefiles && rsync -a --delete etc/modulefiles/ $INSTALLROOT/etc/modulefiles

Check notice on line 18 in moderncppkafka.sh

View workflow job for this annotation

GitHub Actions / alidistlint

"mkdir && rsync" ignores errors if "mkdir" fails; prefer writing the commands on separate lines [ali:masked-exitcode]

Check notice on line 18 in moderncppkafka.sh

View workflow job for this annotation

GitHub Actions / alidistlint

Double quote to prevent globbing and word splitting. [SC2086]

Check notice on line 18 in moderncppkafka.sh

View workflow job for this annotation

GitHub Actions / alidistlint

Double quote to prevent globbing and word splitting. [SC2086]

0 comments on commit 70fec67

Please sign in to comment.