[Bug 200666] Review Request: theora-exp - Experimental theora decoder

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]