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@hhs.nl 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.