Description of problem:
backup-manager installs the following script as cron job

/etc/cron.daily/backup-manager.cron

which is executed daily. This script evaluates the variable BM_DAILY_CRON from /etc/backup-manager.conf

If /etc/backup-manager.conf contains multiple lines that contain the string 'BM_DAILY_CRON' (eg. you created a copy of a configuration line and commented it out by preceding it with '#') the grep command in the above script will pickup both matching lines and when it tries to evaluate the variable from the output of the grep command it fails. This causes backup-manager not to run daily as expected.


Version-Release number of selected component (if applicable):
0.7.10 13.el7

How reproducible:
always

Steps to Reproduce:
1. install backup-manager from epel.
2. open the file /etc/backup-manager.conf with your favourite editor and change the lines

   # RPM-specific : Enabled daily cron
   export BM_DAILY_CRON="false"

to

   # RPM-specific : Enabled daily cron
   #export BM_DAILY_CRON="false"
   export BM_DAILY_CRON="true"

Your backup will never run because the evaluation of the variable BM_DAILY_CRON will fail due to grep is finding 2 matching lines.

Actual results:
/etc/cron.daily/backup-manager.cron: line 10: [: =: unary operator expected
[backup-manager] Daily cron is disabled

Expected results:
  ${BM_PATH} -v
should be executed which is usually
  /usr/sbin/backup-manager -v

Additional info:
The problem can be solved by changing the line

  eval $(grep BM_DAILY_CRON /etc/backup-manager.conf)

in /etc/cron.daily/backup-manager.cron

to

  eval $(grep BM_DAILY_CRON /etc/backup-manager.conf | grep -v ^#)

or use a regex that ignores comment lines to evaluate the variable.