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=737972 --- Comment #3 from Miroslav Suchý <msuchy@xxxxxxxxxx> 2012-03-26 04:06:39 EDT --- Updated: http://miroslav.suchy.cz/fedora/spacewalk-admin/spacewalk-admin-1.8.1-1.fc16.src.rpm http://miroslav.suchy.cz/fedora/spacewalk-admin/spacewalk-admin.spec Please note that I'm member of upstream. Therefore I created new release (as I put those changes directly to upstream) and not put it in spec as patch. > - for a fedora rpm, BuildRoot is no longer needed, same goes for %defattr, and %clean, and cleaning of %install We build the package for EPEL and mainly EPEL5 as well. And there there it is needed. It is easier to maintain the spec file when we can leave it there (and it makes no harm of Fedora). But if you insinst on that I can make separate spec for Fedora and for EPEL. > you should consider using packages rather than filedesps fixed > you should use %global, not %define for %rhnroot fixed > you should also ( IMHO ) place requires on 1 line each, as it permit to have easier to read diff when sending patch That is only two perl packages. Somebody in past decided that it would be nice to have perl packages together. I do not think it is necessary, but if you see it as blocker, I can change it. > the file spacewalk-service look like a init script, and recent fedora use > systemd, so the file should be converted > http://fedoraproject.org/wiki/Packaging:Systemd It looks like init script, but it is not. Spacewalk server is layered application and is build on top of several services (jabberd, httpd, tomcat6, osad-dispatcher, postgresql...). And since usually Spacewalk server is run on dedicated machine and these services are run just because Spacewalk server - if sysadmin want to restart Spacewalk, he would have to write: service tomcat6 stop service httpd stop ... service httpd start service tomcat6 start which means: he has to enumerate all services which Spacewalk requires, not forget single one and do it in correct order. To make it easy for sysadmin, we provide spacewalk-service script, which can do that. But it does *not* start those service during boot. And neither stop those services during shutdown. Each of those services still has its own initd/systemd script. -- 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