[Bug 1098480] Review Request: fts-rest - The REST interface for FTS (File Transfer Service V3)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

Alejandro Alvarez <alejandro.alvarez.ayllon@xxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |alejandro.alvarez.ayllon@ce
                   |                            |rn.ch



--- Comment #5 from Alejandro Alvarez <alejandro.alvarez.ayllon@xxxxxxx> ---
Hi,

Please, can you check the rpmlint output from Adrien? Those errors are still
appearing.

You are probably missing for each %files section

%defattr(-,root,root,-)

Also, from the guidelines
http://fedoraproject.org/wiki/Packaging:Python

python2-devel should be required

Also, the rpm assumes /etc/fts3 exists, which is own by fts-server, so that
should be a Require

Since it packages Apache configuration files, maybe it would be a good idea to
add a condrestart in the %post section

No manpages for the CLI tools. Not strictly required, but should eventually be
provided.

Other than that, it looks OK to me.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review





[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]