Bug 2248960

Summary: Review Request: gstreamer1-plugin-openh264 - GStreamer OpenH264 plugin
Product: [Fedora] Fedora Reporter: Kalev Lember <klember>
Component: Package ReviewAssignee: Neal Gompa <ngompa13>
Status: POST --- QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: ngompa13, package-review, yselkowi
Target Milestone: ---Flags: ngompa13: fedora-review+
Target Release: ---   
Hardware: All   
OS: Linux   
URL: https://www.openh264.org/
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Kalev Lember 2023-11-09 21:39:12 UTC
Spec URL: https://kalev.fedorapeople.org/gstreamer1-plugin-openh264.spec
SRPM URL: https://kalev.fedorapeople.org/gstreamer1-plugin-openh264-1.22.5-1.fc40.src.rpm
Description:
GStreamer is a streaming media framework, based on graphs of elements which
operate on media data.

This package contains the OpenH264 plugin.

Fedora Account System Username: kalev

This splits out gstreamer1-plugin-openh264 from openh264 spec file where it was previously built from a single srpm together with the openh264 library (see https://src.fedoraproject.org/rpms/openh264/blob/rawhide/f/openh264.spec). The plan is to link it against the new "noopenh264" stub which we are able to ship in Fedora proper, see https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=2247274 which then gets swapped out with the actual openh264 library package (downloaded from Cisco servers) during first 'dnf update'.

Note that I suspect this package is going to be short lived in Fedora because we'll probably just end up enabling the openh264 plugin in gstreamer1-plugins-bad-free and retiring separate gstreamer1-plugin-openh264 package. There is some value in a new package during transition period but I expect it to be short lived.

The separate gstreamer1-plugin-openh264 package is however going to be needed for EPEL 9 so that we can drop the gstreamer plugin from the openh264 srpm (and Cisco repo) and instead ship the plugin in  EPEL proper. So this is mostly a new EPEL package review request, but I'd like to use it in Fedora as well during the transition period.

Koji scratch build: https://koji.fedoraproject.org/koji/taskinfo?taskID=108818423

Scratch builds are done using 'fedpkg build --scratch --srpm --target f40-build-side-77098' - noopenh264 cannot be merged into rawhide before we have been able to get an openh264 version that obsoletes noopenh264 in place in the Cisco repo, so all builds have to be done using the side tag for now.

Comment 1 Fedora Review Service 2023-11-09 21:42:49 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/6617270
(failed)

Build log:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2248960-gstreamer1-plugin-openh264/fedora-rawhide-x86_64/06617270-gstreamer1-plugin-openh264/builder-live.log.gz

Please make sure the package builds successfully at least for Fedora Rawhide.

- If the build failed for unrelated reasons (e.g. temporary network
  unavailability), please ignore it.
- If the build failed because of missing BuildRequires, please make sure they
  are listed in the "Depends On" field


---
This comment was created by the fedora-review-service
https://github.com/FrostyX/fedora-review-service

If you want to trigger a new Copr build, add a comment containing new
Spec and SRPM URLs or [fedora-review-service-build] string.

Comment 2 Neal Gompa 2023-11-09 21:45:52 UTC
Review notes:

* Package follows Fedora Packaging Guidelines
* Package builds and installs
* Package licensing is correctly handled
* No serious issues from rpmlint

PACKAGE APPROVED.

Comment 3 Yaakov Selkowitz 2023-11-09 21:48:45 UTC
Please DO NOT import this into Fedora branches, but it makes sense for EPEL.

Comment 4 Fedora Admin user for bugzilla script actions 2023-11-09 21:55:15 UTC
The Pagure repository was created at https://src.fedoraproject.org/rpms/gstreamer1-plugin-openh264

Comment 5 Kalev Lember 2023-11-09 21:57:13 UTC
Thanks for the super quick review, Neal!