Bug 2248960 - Review Request: gstreamer1-plugin-openh264 - GStreamer OpenH264 plugin [NEEDINFO]
Summary: Review Request: gstreamer1-plugin-openh264 - GStreamer OpenH264 plugin
Keywords:
Status: POST
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Neal Gompa
QA Contact: Fedora Extras Quality Assurance
URL: https://www.openh264.org/
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2023-11-09 21:39 UTC by Kalev Lember
Modified: 2024-11-09 08:31 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed:
Type: ---
Embargoed:
ngompa13: fedora-review+
package-review: needinfo? (klember)


Attachments (Terms of Use)

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!

Comment 6 Package Review 2024-11-09 08:31:32 UTC
This package was never imported in the repository. Are you still interested in finalizing this?


Note You need to log in before you can comment on or make changes to this bug.