From mboxrd@z Thu Jan  1 00:00:00 1970
From: Michael Tremer <michael.tremer@ipfire.org>
To: development@lists.ipfire.org
Subject: Re: [PATCH] Fixes for 'backup.pl' (Bug #11816)
Date: Wed, 15 Aug 2018 09:22:22 +0100
Message-ID: <2b3ed62d993cc67c58273e7deae70b0646e0704a.camel@ipfire.org>
In-Reply-To: <20180814193438.1961-1-matthias.fischer@ipfire.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============8143130941136884989=="
List-Id: <development.lists.ipfire.org>

--===============8143130941136884989==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit

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(a)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(a)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`;


--===============8143130941136884989==--