https://bugzilla.rpmfusion.org/show_bug.cgi?id=4993

--- Comment #10 from Nicolas Chauvet <[email protected]> ---
(In reply to Akarshan Biswas from comment #6)


> Anyway, I have added support for widevine(Netflix, amazon video support) and
> bump up base release to 50 so to prevent conflict with chromium package.
This will not prevent conflict. Our policy prevents to obsoletes/provides any
fedora package. Right now the package will replace chromium/freeworld package
from all users either or not they choose to pick the vaapi variant. Please drop
the obsoletes/Provides and keep conflict.

As I understand, the package should be parallel installable. But you said that
because they could use the same profile, it's best to keep either
chromium/chromium-vaapi and not both on the same system.
Maybe at some point the vaapi patches will be merged into chromium, so the
chromium-vaapi could be retired.

There are lot of uneeded spaces in the spec file, please remove them.

Are you sure the libva dependency needs to be explicitly added ?

Please add an arched dependency on libva* (such that the x86_64 version get
picked is only the i686 one is available).
Recommends: libva-intel-driver%{_isa}


@All
Any volunteer for a full review ?
Thx

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.
_______________________________________________
rpmfusion-developers mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to