Re: amraw

Olivier <[email protected]> Thu, 29 Nov 2018 08:36:35 +0700
Newsgroups gmane.comp.archivers.amanda.devel
Message-ID <[email protected]>
"Dustin J. Mitchell" <[email protected]> writes:

> Sounds like a bug.. Do you want to make a PR?

Dustin,

Given my complete failure the last time I tried making a PR on Github, I
prefer someone in the knowhow does it.

Best regards,

Olivier


>
> Dustin
>
> Op wo 28 nov. 2018 om 02:19 schreef Olivier <[email protected]>:
>
>  Hello,
>
>  I am using amraw as a skeleton to develop a MySQl plugin (and hopefully
>  a VMware one later).
>
>  Reading the code for amraw, I saw the following, repeated in several
>  places in the code:
>
>  if ($#{$self->{include_list}} >= 0) {
>  $self->print_to_server("include-list not supported for backup",
>  $Amanda::Script_App::ERROR);
>  }
>  if ($#{$self->{include_list}} >= 0) {
>  $self->print_to_server("exclude-list not supported for backup",
>  $Amanda::Script_App::ERROR);
>  }
>
>  While the message is for "include" and "exclude", the test is always on
>  "include".
>
>  Best regards,
>
>  Olivier
>
>  -- 
>

--