https://bugzilla.redhat.com/show_bug.cgi?id=2244819
Bug ID: 2244819 Summary: Review Request: xeve - Reference MPEG-5 Part 1 (EVC) encoder Product: Fedora Version: rawhide Hardware: All OS: Linux Status: NEW Component: Package Review Severity: medium Priority: medium Assignee: nobody@fedoraproject.org Reporter: ngompa13@gmail.com QA Contact: extras-qa@fedoraproject.org CC: package-review@lists.fedoraproject.org Target Milestone: --- Classification: Fedora
Spec URL: https://ngompa.fedorapeople.org/for-review/xeve.spec SRPM URL: https://ngompa.fedorapeople.org/for-review/xeve-0.4.3-1.fc39.src.rpm
Description: The eXtra-fast Essential Video Encoder (XEVE) is an opensource and fast MPEG-5 EVC encoder.
MPEG-5 Essential Video Coding (EVC) is a video compression standard of ISO/IEC Moving Picture Experts Grop (MPEG). The main goal of the EVC is to provide a significantly improved compression capability over existing video coding standards with timely publication of terms.
EVC defines two profiles, including "Baseline Profile" and "Main Profile".
The "Baseline profile" contains only technologies that are older than 20 years or otherwise freely available for use in the standard. In addition, the "Main profile" adds a small number of additional tools, each of which can be either cleanly disabled or switched to the corresponding baseline tool on an individual basis.
This package only includes the "Baseline profile".
Fedora Account System Username: ngompa
https://bugzilla.redhat.com/show_bug.cgi?id=2244819
Neal Gompa ngompa13@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Doc Type|--- |If docs needed, set a value Blocks| |2218117 (MultimediaSIG)
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=2218117 [Bug 2218117] Tracker for multimedia-sig
https://bugzilla.redhat.com/show_bug.cgi?id=2244819
Fedora Review Service fedora-review-bot@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- URL| |https://github.com/mpeg5/xe | |ve
--- Comment #1 from Fedora Review Service fedora-review-bot@fedoraproject.org --- Copr build: https://copr.fedorainfracloud.org/coprs/build/6543576 (succeeded)
Review template: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-rev...
Please take a look if any issues were found.
--- 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.
https://bugzilla.redhat.com/show_bug.cgi?id=2244819
Neal Gompa ngompa13@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks| |182235 (FE-Legal)
--- Comment #2 from Neal Gompa ngompa13@gmail.com --- Requesting FE-Legal to review whether the stripping is sufficient.
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=182235 [Bug 182235] Fedora Legal Tracker
https://bugzilla.redhat.com/show_bug.cgi?id=2244819
Aoife Moloney amoloney@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags| |needinfo?(ngompa13@gmail.co | |m) CC| |amoloney@redhat.com
--- Comment #3 from Aoife Moloney amoloney@redhat.com --- Hi Neal,
From discussing this with the internal team, the feedback is that the video codecs need to be released 25+ years in order to be included in Fedora and/or RHEL, and the bz's (including rhbz#2244820) and/or github files associated with them are not clear that they meet this criteria. Is there a way you can verify that they do in fact meet this requirement?
Thanks, Aoife
https://bugzilla.redhat.com/show_bug.cgi?id=2244819
Neal Gompa ngompa13@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags|needinfo?(ngompa13@gmail.co | |m) |
--- Comment #4 from Neal Gompa ngompa13@gmail.com --- (In reply to Aoife Moloney from comment #3)
Hi Neal,
From discussing this with the internal team, the feedback is that the video codecs need to be released 25+ years in order to be included in Fedora and/or RHEL, and the bz's (including rhbz#2244820) and/or github files associated with them are not clear that they meet this criteria. Is there a way you can verify that they do in fact meet this requirement?
The rule I've been told is 20 years rather than 25 years, since the former is the length of time of a patent.
That being said, the whitepaper published by MPEG about the new codec explicitly notes this:
The Baseline profile contains technologies assessed to be over 20 years to maximally approach to a royalty free codec. The Baseline profile builds a video codec using only conventional coding technologies which consist of traditional methods from the early 1980s to the end of the 1990s.
From: https://www.mpeg.org/wp-content/uploads/mpeg_meetings/136_OnLine/w21036.zip
From: https://www.mpeg.org/standards/MPEG-5/1/
From that perspective, it should fulfill the necessary requirements.
https://bugzilla.redhat.com/show_bug.cgi?id=2244819
--- Comment #5 from Aoife Moloney amoloney@redhat.com --- Thank you for the quick reply! Let me take this back to the team and see if this satisfies the requirements to unblock this bug and the associated bug too.
https://bugzilla.redhat.com/show_bug.cgi?id=2244819
Simone Caronni negativo17@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED Assignee|nobody@fedoraproject.org |negativo17@gmail.com CC| |negativo17@gmail.com
https://bugzilla.redhat.com/show_bug.cgi?id=2244819
--- Comment #6 from Simone Caronni negativo17@gmail.com --- The line "Requires: %{name}%{?_isa} = %{version}-%{release}" at the top can be removed. It requires itself, and if the original intention was requiring the libraries, those are satisfied anyway by the automatic dependencies.
https://bugzilla.redhat.com/show_bug.cgi?id=2244819
Simone Caronni negativo17@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags| |needinfo?(amoloney@redhat.c | |om)
--- Comment #7 from Simone Caronni negativo17@gmail.com --- @amoloney@redhat.com any news on the legal review?
@ngompa13@gmail.com There's version 0.5.0 out which contains your patches.
https://bugzilla.redhat.com/show_bug.cgi?id=2244819
Simone Caronni negativo17@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags| |needinfo?(amoloney@redhat.c | |om)
https://bugzilla.redhat.com/show_bug.cgi?id=2244819
--- Comment #8 from Neal Gompa ngompa13@gmail.com --- Update to latest version (0.5.1)
Spec URL: https://ngompa.fedorapeople.org/for-review/xeve.spec SRPM URL: https://ngompa.fedorapeople.org/for-review/xeve-0.5.1-1.fc40.src.rpm
https://bugzilla.redhat.com/show_bug.cgi?id=2244819
--- Comment #9 from Fedora Review Service fedora-review-bot@fedoraproject.org --- Created attachment 2049474 --> https://bugzilla.redhat.com/attachment.cgi?id=2049474&action=edit The .spec file difference from Copr build 6543576 to 8091458
https://bugzilla.redhat.com/show_bug.cgi?id=2244819
Fedora Review Service fedora-review-bot@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |AutomationTriaged
--- Comment #10 from Fedora Review Service fedora-review-bot@fedoraproject.org --- Copr build: https://copr.fedorainfracloud.org/coprs/build/8091458 (succeeded)
Review template: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-rev...
Please take a look if any issues were found.
--- 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.
https://bugzilla.redhat.com/show_bug.cgi?id=2244819
Petr Pisar ppisar@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags| |fedora-review?
https://bugzilla.redhat.com/show_bug.cgi?id=2244819
--- Comment #11 from Simone Caronni negativo17@gmail.com --- Any update from FE-Legal?... it's almost 3 months now.
https://bugzilla.redhat.com/show_bug.cgi?id=2244819
Neal Gompa ngompa13@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks|182235 (FE-Legal) | Flags|needinfo?(amoloney@redhat.c | |om) | |needinfo?(amoloney@redhat.c | |om) |
--- Comment #12 from Neal Gompa ngompa13@gmail.com --- Per discussion with mattdm, lifting FE-Legal.
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=182235 [Bug 182235] Fedora Legal Tracker
https://bugzilla.redhat.com/show_bug.cgi?id=2244819
Simone Caronni negativo17@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags|fedora-review? |fedora-review+
--- Comment #13 from Simone Caronni negativo17@gmail.com --- Package approved.
Please fix the changelog before importing or switch to autospec:
xeve.x86_64: W: incoherent-version-in-changelog 0.4.3-1 ['0.5.1-1.fc42', '0.5.1-1']
https://bugzilla.redhat.com/show_bug.cgi?id=2244819
Fedora Admin user for bugzilla script actions fedora-admin-xmlrpc@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |RELEASE_PENDING
--- Comment #14 from Fedora Admin user for bugzilla script actions fedora-admin-xmlrpc@fedoraproject.org --- The Pagure repository was created at https://src.fedoraproject.org/rpms/xeve
https://bugzilla.redhat.com/show_bug.cgi?id=2244819
Dominik 'Rathann' Mierzejewski dominik@greysector.net changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed In Version| |xeve-0.5.1-2.fc43 Resolution|--- |RAWHIDE Status|RELEASE_PENDING |CLOSED Last Closed| |2025-05-13 15:41:12
package-review@lists.fedoraproject.org
-
bugzilla＠redhat.com