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=592670 --- Comment #2 from Rafael Aquini <aquini@xxxxxxxxx> 2010-05-17 10:15:32 EDT --- Spec URL: http://aquini.tchesoft.com/RPMS/mongoose/mongoose.spec SRPM URL: http://aquini.tchesoft.com/RPMS/mongoose/mongoose-2.8-2.fc12.src.rpm MD5: fa58e9038709d389b2fc7c6cffc9c068 mongoose-2.8-2.fc12.src.rpm 3ffb7344a8f8cb8056cfa2c8e220649c mongoose.spec Chen, Thanks for reviewing the spec. I did make some of the modifications you suggested, as follows: [2] -devel package was stripped off the spec file; [3] The misusing of %doc was fixed; [4] Group changed to Applications/System -- IMHO, I do not think it could fit in anyone else in System Environment class; [5] Now the man page compression is done by rpmbuild itself. Unfortunately, there was some suggestions of yours I could not deliver as promptly as you wish: [1] I do not see a real necessity to ship a SysV initscrip, since mongoose is a regular user tool not intended to work as a daemon. The purpose of mongoose is to give flexibility to an ordinary user (not administrator) when he/she needs to set up quickly an web server to share files in a specific directory, or just do some tests. Please, let me know if you observation [1] is a real blocker to this package. [6] As you may notice, there is no install section on Makefile delivered by upstream, so I have the real need of being using %{__install} macros. IMHO, it would be preferable using this macros than the commands itself, wouldn' be? Please, let me know if you observation [6] is also a real blocker to this package. Once again, thanks a lot for your attention on this matter. -- 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. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review