Fixing tar command order

gkiefer / backup2l

backup2l - A Low-Maintenance Backup/Restore Tool

https://gkiefer.github.io/backup2l

GNU General Public License v2.0

47 stars 16 forks source link

Fixing tar command order #16

Closed boppy closed 5 years ago

boppy commented 5 years ago

Fixes Error

tar: The following options were used after any non-optional arguments in archive create or update mode.  These options are positional and affect only arguments that follow them.  Please, rearrange them properly.
tar: --no-recursion has no effect
tar: Exiting with failure status due to previous errors

wturrell commented 5 years ago

(Thanks for this.)

Doesn't this need to be updated in first-time.conf too? Also - what about DRIVER_TAR_GZ_SPLIT (which is included in first-time.conf but not the source)?

gkiefer commented 5 years ago

Example drivers in 'first-time.conf' are donated by users, provided "as is", and are not regularly updated and tested by myself.

Anyway: I agree. DRIVER_TAR_GZ_SPLIT and several others should be updated and tested(!) accordingly. Without testing I prefer to leave them in their original state.

Pull requests (or patches) are welcome.