Spec URL: https://slaanesh.fedorapeople.org/oneVPL-intel-gpu.spec SRPM URL: https://slaanesh.fedorapeople.org/oneVPL-intel-gpu-22.2.0-1.fc35.src.rpm Description: Intel® oneVPL GPU Runtime is a Runtime implementation of oneVPL API for Intel Gen GPUs. Runtime provides access to hardware-accelerated video decode, encode and filtering. Fedora Account System Username: slaanesh
Taking this review.
> Intel® oneVPL GPU Runtime We don't use (R)/(C)/(TM) or their unicode equivalents: https://docs.fedoraproject.org/en-US/packaging-guidelines/#_trademarks_in_summary_or_description Please drop them on import. Other than that... Review notes: * Packaging complies with the guidelines * Package builds and installs * No serious issues from rpmlint * Licensing is correct and license files are correctly installed PACKAGE APPROVED.
(In reply to Neal Gompa from comment #2) > > Intel® oneVPL GPU Runtime > > We don't use (R)/(C)/(TM) or their unicode equivalents: > https://docs.fedoraproject.org/en-US/packaging-guidelines/ > #_trademarks_in_summary_or_description > > Please drop them on import. Ah! I missed that. I will do. Thanks!
(fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/oneVPL-intel-gpu
Did this have any legal review? there is definitely some possible patented stuff that could be linked to this.
Nope, I was not aware sorry. How to trigger a legal review after the package is already available?
I'm reopening this for legal review, based on it seems to enable APIs for h264/5 decoding from what I understand of the package.
Reverting review state back to "fedora-review?".
(In reply to Dave Airlie from comment #7) > I'm reopening this for legal review, based on it seems to enable APIs for > h264/5 decoding from what I understand of the package. As far as I know this thing can't do anything without a functioning driver component, which is being reviewed in bug 1942132 with the encumbered codecs stripped.
For some additional protection, we could also add "-DMFX_ENABLE_H264_VIDEO_DECODE=OFF -DMFX_ENABLE_H265_VIDEO_DECODE=OFF -DMFX_ENABLE_VC1_VIDEO_DECODE=OFF -DMFX_ENABLE_H264_VIDEO_ENCODE=OFF -DMFX_ENABLE_H265_VIDEO_ENCODE=OFF" to the cmake option list to forcibly disable those runtime paths entirely.
Lifting FE-Legal.