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=492396 --- Comment #6 from Xavier Bachelot <xavier@xxxxxxxxxxxx> 2009-04-14 13:02:12 EDT --- (In reply to comment #5) > ok great. Let's see if we can move this forward. > > As noted on bug 492395, it seems that you can just require php-common because > it requires php. > Or the other way round, just require php. That's what I've done on my local copy. > Then, I am not sure if there are guidelines against it, but installing the file > in {datadir}/php is probably not a wise idea. What about putting it in a > subdirectory (for example feedcreator would do). In that case, please remember > to let the package own the directory. Not sure what's best here. As this is only one file, I think it's ok to let it at the root of the php include dir. And anyway, looking at the pear include dir, I see that's what is done : /usr/share/pear/someclass.php, then a /usr/share/pear/someclass subdir with everything else. That's what I've done for php-geshi (https://bugzilla.redhat.com/show_bug.cgi?id=492395). -- 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