Skip to content

Commit

Permalink
Merge pull request #323 from Will-Cross1/303-add-linting-to-shell-script
Browse files Browse the repository at this point in the history
Added linting to new shell build script
  • Loading branch information
tofu-rocketry authored Apr 17, 2024
2 parents d3a9364 + d5e8811 commit d03c230
Showing 1 changed file with 16 additions and 1 deletion.
17 changes: 16 additions & 1 deletion scripts/ssm-build.sh
Original file line number Diff line number Diff line change
Expand Up @@ -5,10 +5,13 @@
# @Author: Nicholas Whyatt ([email protected])

# Script runs well with FPM 1.14.2 on ruby 2.7.1, setuptools 51.3.3 on RHEL and Deb platforms

# Download ruby (if you're locked to 2.5, use RVM) and then run:
# sudo gem install fpm -v 1.14.2
# (may need to be run without the 'sudo')

# for RPM builds, you will also need:
# sudo yum install rpm-build | sudo apt-get install rpm
# sudo yum install rpm-build rpmlint | sudo apt-get install rpm lintian
# ./ssm-build.sh (deb | rpm) <version> <iteration> <python_root_dir>
# e.g.
# ./ssm-build.sh deb 3.4.0 1 /usr/lib/python3.6
Expand Down Expand Up @@ -197,3 +200,15 @@ fpm -s pleaserun -t "$PACK_TYPE" \
--depends apel-ssm \
--package "$BUILD_DIR" \
/usr/bin/ssmreceive

echo "Possible Issues to Fix:"
if [ "$OS_EXTENSION" == "_all" ]
then
# Check the resultant debs for 'lint'
TAG="$VERSION-$ITERATION"
lintian "$BUILD_DIR"/apel-ssm_"${TAG}"_all.deb
lintian "$BUILD_DIR"/apel-ssm-service_"${TAG}"_all.deb
else
# Check for errors in SPEC and built packages
rpmlint ~/rpmbuild
fi

0 comments on commit d03c230

Please sign in to comment.