Please (re)move "# ie. nothing" from example config file

Bug #593220 reported by Wolfgang Karall on 2010-06-13
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
mylvmbackup
Medium
Lenz Grimmer

Bug Description

Hello,

the example config file has the following lines included:

# for cat:
#compressarg= # ie. nothing

Since there is config/script syntax that allows for comments at the end of normal, active lines, when I tried to do an uncompressed tar I just removed the first # from the line:

compressarg= # ie. nothing

which resulted in my Cronjob trying to send me a rather big uncompressed tar file instead of writing one to the backup directory. :)

I'd suggest to move the notice to the line before, so it could look like

# for cat, nothing
#compressarg=

Sorry for providing no patch, don't have bzr at hand.

Probably it would also be a good idea to check for unescaped # in command lines before running them. :)

Regards
Wolfgang Karall

Related branches

Lenz Grimmer (lenzgr) wrote :

Thanks for the suggestion! Good point, now commited to the trunk.

Changed in mylvmbackup:
assignee: nobody → LenZ (lenzgr)
importance: Undecided → Medium
milestone: none → 0.14
status: New → Fix Committed
Lenz Grimmer (lenzgr) on 2013-06-24
Changed in mylvmbackup:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers