Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: theora-exp - Experimental theora decoder https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=200666 ------- Additional Comments From j.w.r.degoede@xxxxxx 2006-11-10 10:34 EST ------- (In reply to comment #11) > I think it would be better to not own /usr/include/theora/ dir by this package. > First, in hope that libtheora-devel will fix this issue soon (and after the > fixing no any changes for theora-exp-devel's %files section will be required). > Second, let's consider such Core's package behaviour as a precedent, which > allows us to do the same (at least for a while). Otherwise this issue is a > blocker, which prevents the including of theora-exp until libtheora-devel will > be fixed, which might require unpredictable amount of time. > I don't see how this is a blocker, independent of libtheora-devel being fixed this package must still own /usr/include/theora, because quoting from the review guidelines: "MUST: A package must own all directories that it creates. If it does not create a directory that it uses, then it should require a package which does create that directory." And since this package does not and will not Require libtheora-devel, because it simply doesn't need it, it thus MUST own that directory. And having multiple owners for directories, although undesirable is not a blocker. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug, or are watching the QA contact. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review