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

Bug #593220 reported by Wolfgang Karall-Ahlborn
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
mylvmbackup
Fix Released
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

Revision history for this message
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)
Changed in mylvmbackup:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.