search

ngageoint / hootenanny-rpms

RPMs needed for a Hootenanny install
GNU General Public License v3.0
5 stars 4 forks source link

RPM_VERSION hotfix #222

Closed jbronn closed 6 years ago

jbronn commented 6 years ago

During testing of #221 I disabled slack notifications and missed that I didn't include the setting of $RPM_VERSION. I'm bringing it back here, otherwise the script will fail after sync and there'll be no slack message (due to set -u). In addition, I used find instead of ls because shellcheck will fail otherwise with:

SC2012: Use find instead of ls to better handle non-alphanumeric filenames.

jbronn commented 6 years ago

The latest develop RPM was uploaded, however the Slack message wasn't sent and the overall build failed. See CircleCI Build 407.