[Bug 459535] Review Request: backup-manager - A command line backup tool for GNU/Linux

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=459535


Aurelien Bompard <gauret at free.fr> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |gauret at free.fr
         AssignedTo|nobody at fedoraproject.org    |gauret at free.fr
               Flag|                            |fedora-review+




--- Comment #9 from Aurelien Bompard <gauret at free.fr>  2009-01-10 08:34:29 EDT ---
Since there's been no action for over a month, I'm taking over the review :)

Review for release 3.fc10:
* RPM name is OK
* Source backup-manager-0.7.7.tar.gz is the same as upstream
* rpmlint looks OK
* File list looks OK
* Works fine

APPROVED

Two small things though:
- the actual license is GPLv2+ (see the head of the backup-manager file).
Please fix the RPM tag.
- in the comment you add to the conf file, please switch "Specific RPM" to
"RPM-specific", this is better english.
Since those changes are so trivial, no need to resubmit a package here, just do
it right before or after importing.

Thanks Guillaume.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.