fix: daily.sh log_dir fails when install_dir and log_dir are not set #4992

Merged
merged 1 commit into from Nov 15, 2016

Projects

None yet

4 participants

@murrant
Contributor
murrant commented Nov 14, 2016

Please note

Please read this information carefully. You can run ./scripts/pre-commit.php to check your code before submitting.

@murrant murrant fix: daily.sh log_dir failed when install_dir and log_dir were not set
a0e3378
@murrant murrant added the Bug label Nov 14, 2016
@murrant murrant changed the title from fix: daily.sh log_dir failed when install_dir and log_dir were not set to fix: daily.sh log_dir failed when install_dir and log_dir are not set Nov 14, 2016
@murrant murrant changed the title from fix: daily.sh log_dir failed when install_dir and log_dir are not set to fix: daily.sh log_dir fails when install_dir and log_dir are not set Nov 14, 2016
@scrutinizer-notifier

The inspection completed: No new issues

@laf
Member
laf commented Nov 15, 2016

Weird, I have neither set and didn't see an issue before hand. Either way, this works for me still as well.

@laf laf merged commit e409178 into librenms:master Nov 15, 2016

2 checks passed

Auto-Deploy Build finished.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@murrant
Contributor
murrant commented Nov 15, 2016

Shrug, I got errors saying /logs not found...

@murrant murrant deleted the murrant:fix-daily-log branch Nov 15, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment