Fixes for 'backup.pl' (Bug #11816)

Message ID 20180814193438.1961-1-matthias.fischer@ipfire.org
State Accepted
Commit 84578512f52e62bc994b944f88f1785c73b4d96a
Headers
Series Fixes for 'backup.pl' (Bug #11816) |

Commit Message

Matthias Fischer Aug. 15, 2018, 5:34 a.m. UTC
  Hi,

Fixes #11816
(https://bugzilla.ipfire.org/show_bug.cgi?id=11816 and
https://bugzilla.ipfire.org/attachment.cgi?id=608):

"[root@ipfire ~]# backupctrl exclude
...
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: --exclude-from '/var/ipfire/backup/exclude.user' has no effect
tar: Exiting with failure status due to previous errors"

Please test - I got no errors anymore.

Best,
Matthias

Signed-off-by: Matthias Fischer <matthias.fischer@ipfire.org>
---
 config/backup/backup.pl | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
  

Comments

Michael Tremer Aug. 15, 2018, 6:22 p.m. UTC | #1
Thanks. That was quick! Merged.

-Michael

On Tue, 2018-08-14 at 21:34 +0200, Matthias Fischer wrote:
> Hi,
> 
> Fixes #11816
> (https://bugzilla.ipfire.org/show_bug.cgi?id=11816 and
> https://bugzilla.ipfire.org/attachment.cgi?id=608):
> 
> "[root@ipfire ~]# backupctrl exclude
> ...
> 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: --exclude-from '/var/ipfire/backup/exclude.user' has no effect
> tar: Exiting with failure status due to previous errors"
> 
> Please test - I got no errors anymore.
> 
> Best,
> Matthias
> 
> Signed-off-by: Matthias Fischer <matthias.fischer@ipfire.org>
> ---
>  config/backup/backup.pl | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/config/backup/backup.pl b/config/backup/backup.pl
> index d430f3467..ce16e7f42 100644
> --- a/config/backup/backup.pl
> +++ b/config/backup/backup.pl
> @@ -46,7 +46,7 @@ if (($ARGV[0] eq 'include') || ($ARGV[0] eq 'iso')) {
>    print DATEI @include;
>    print "/var/log/messages";
>    close(DATEI);
> -  system("tar -cvzf /var/ipfire/backup/$Jahr$Monat$Monatstag-
> $Stunden$Minuten.ipf --files-from='/tmp/include' --exclude-
> from='/var/ipfire/backup/exclude' --files-
> from='/var/ipfire/backup/include.user' --exclude-
> from='/var/ipfire/backup/exclude.user'");
> +  system("tar -cvzf /var/ipfire/backup/$Jahr$Monat$Monatstag-
> $Stunden$Minuten.ipf --exclude-from='/var/ipfire/backup/exclude' --exclude-
> from='/var/ipfire/backup/exclude.user' --files-from='/tmp/include' --files-
> from='/var/ipfire/backup/include.user'");
>    system("rm /tmp/include");
>    if ($ARGV[0] eq 'iso') {
>    	system("/usr/local/bin/backupiso $Jahr$Monat$Monatstag-$Stunden$Minuten
> &");
> @@ -57,7 +57,7 @@ elsif ($ARGV[0] eq 'exclude') {
>    open(DATEI, ">/tmp/include") || die "Could not save temp include file";
>    print DATEI @include;
>    close(DATEI);
> -  system("tar -cvzf /var/ipfire/backup/$Jahr$Monat$Monatstag-
> $Stunden$Minuten.ipf --files-from='/tmp/include' --exclude-
> from='/var/ipfire/backup/exclude' --files-
> from='/var/ipfire/backup/include.user' --exclude-
> from='/var/ipfire/backup/exclude.user'");
> +  system("tar -cvzf /var/ipfire/backup/$Jahr$Monat$Monatstag-
> $Stunden$Minuten.ipf --exclude-from='/var/ipfire/backup/exclude' --exclude-
> from='/var/ipfire/backup/exclude.user' --files-from='/tmp/include' --files-
> from='/var/ipfire/backup/include.user'");
>    system("rm /tmp/include");
>  }
>  elsif ($ARGV[0] eq 'restore') {
> @@ -158,7 +158,7 @@ sub createinclude(){
>    open(DATEI, "<${General::swroot}/backup/include") || die "Can not open
> include file";
>    my @Zeilen = <DATEI>;
>    close(DATEI);
> -  
> +
>    foreach (@Zeilen){
>      chomp($_);
>      my @files = `find $_ -maxdepth 0 2>/dev/null`;
  

Patch

diff --git a/config/backup/backup.pl b/config/backup/backup.pl
index d430f3467..ce16e7f42 100644
--- a/config/backup/backup.pl
+++ b/config/backup/backup.pl
@@ -46,7 +46,7 @@  if (($ARGV[0] eq 'include') || ($ARGV[0] eq 'iso')) {
   print DATEI @include;
   print "/var/log/messages";
   close(DATEI);
-  system("tar -cvzf /var/ipfire/backup/$Jahr$Monat$Monatstag-$Stunden$Minuten.ipf --files-from='/tmp/include' --exclude-from='/var/ipfire/backup/exclude' --files-from='/var/ipfire/backup/include.user' --exclude-from='/var/ipfire/backup/exclude.user'");
+  system("tar -cvzf /var/ipfire/backup/$Jahr$Monat$Monatstag-$Stunden$Minuten.ipf --exclude-from='/var/ipfire/backup/exclude' --exclude-from='/var/ipfire/backup/exclude.user' --files-from='/tmp/include' --files-from='/var/ipfire/backup/include.user'");
   system("rm /tmp/include");
   if ($ARGV[0] eq 'iso') {
   	system("/usr/local/bin/backupiso $Jahr$Monat$Monatstag-$Stunden$Minuten &");
@@ -57,7 +57,7 @@  elsif ($ARGV[0] eq 'exclude') {
   open(DATEI, ">/tmp/include") || die "Could not save temp include file";
   print DATEI @include;
   close(DATEI);
-  system("tar -cvzf /var/ipfire/backup/$Jahr$Monat$Monatstag-$Stunden$Minuten.ipf --files-from='/tmp/include' --exclude-from='/var/ipfire/backup/exclude' --files-from='/var/ipfire/backup/include.user' --exclude-from='/var/ipfire/backup/exclude.user'");
+  system("tar -cvzf /var/ipfire/backup/$Jahr$Monat$Monatstag-$Stunden$Minuten.ipf --exclude-from='/var/ipfire/backup/exclude' --exclude-from='/var/ipfire/backup/exclude.user' --files-from='/tmp/include' --files-from='/var/ipfire/backup/include.user'");
   system("rm /tmp/include");
 }
 elsif ($ARGV[0] eq 'restore') {
@@ -158,7 +158,7 @@  sub createinclude(){
   open(DATEI, "<${General::swroot}/backup/include") || die "Can not open include file";
   my @Zeilen = <DATEI>;
   close(DATEI);
-  
+
   foreach (@Zeilen){
     chomp($_);
     my @files = `find $_ -maxdepth 0 2>/dev/null`;