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=514833 Andrew Colin Kissa <andrew@xxxxxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |andrew@xxxxxxxxxxxxx Flag| |fedora-review? --- Comment #1 from Andrew Colin Kissa <andrew@xxxxxxxxxxxxx> 2009-08-03 13:47:19 EDT --- A few issues to begin with * The package contains searchd which is a server daemon (http://sphinxsearch.com/docs/current.html#ref-searchd) so the package should contain an init script for managing the service. * The *.conf.dist files installed in /etc/sphinx could be installed as doc's as they are not really required for the program to function. * The sample sql file should not be placed in /etc, better installed as part of the doc's * Consider packaging libsphinxclient as a sub package as we will need this to package the PHP PEAR package for sphinx http://pecl.php.net/package/sphinx * Need to create a data directory where the index is stored, since this is data that changes it cannot be stored in /etc where the default configuration is configured to store it. -- 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. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review