Re: amraw

https://github.com/zmanda/amanda/pull/92

Op wo 28 nov. 2018 om 20:36 schreef Olivier <[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
> >
> >  --
> >
>
> --
>