diff --git a/Makefile b/Makefile index b1763c3..1aab6f7 100644 --- a/Makefile +++ b/Makefile @@ -35,6 +35,7 @@ help: @echo "Available targets:" @echo " install - Install the script to $(LIBEXECDIR), and systemd files" @echo " uninstall - Remove installed files" + @echo " check - Check for syntax errors in $(SCRIPT_SRC)" @echo " package - Build both RPM and DEB packages" @echo " rpm - Build RPM package" @echo " deb - Build DEB package" @@ -67,6 +68,10 @@ uninstall: rm -rf $(DESTDIR)$(DOCDIR) @echo "Uninstall complete" +check: + @echo "Running syntax test for $(SCRIPT_SRC)" + bash -n $(SCRIPT_SRC) + # Package building targets package: rpm deb deb-source diff --git a/debian/rules b/debian/rules index cd8b79f..e25195b 100755 --- a/debian/rules +++ b/debian/rules @@ -13,7 +13,7 @@ override_dh_auto_build: # Nothing to build - this is a shell script package override_dh_auto_test: - # Skip tests for now + $(MAKE) check override_dh_install: dh_install diff --git a/prometheus-cvmfs-exporter.spec b/prometheus-cvmfs-exporter.spec index 8f1c7c9..8a69045 100644 --- a/prometheus-cvmfs-exporter.spec +++ b/prometheus-cvmfs-exporter.spec @@ -51,6 +51,9 @@ make install-systemd DESTDIR=%{buildroot} # Remove duplicate LICENSE file from doc directory since %license handles it rm -f %{buildroot}%{_docdir}/%{name}/LICENSE +%check +make check + %post %systemd_post cvmfs-client-prometheus@.service %systemd_post cvmfs-client-prometheus.socket