https://bugzilla.redhat.com/show_bug.cgi?id=2244820
Bug ID: 2244820 Summary: Review Request: xevd - Reference MPEG-5 Part 1 (EVC) decoder 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/xevd.spec SRPM URL: https://ngompa.fedorapeople.org/for-review/xevd-0.4.1-1.fc39.src.rpm
Description: The eXtra-fast Essential Video Decoder (XEVD) is an opensource and fast MPEG-5 EVC decoder.
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=2244820
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=2244820
Fedora Review Service fedora-review-bot@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- URL| |https://github.com/mpeg5/xe | |vd
--- Comment #1 from Fedora Review Service fedora-review-bot@fedoraproject.org --- Copr build: https://copr.fedorainfracloud.org/coprs/build/6543577 (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=2244820
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=2244820
Simone Caronni negativo17@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |negativo17@gmail.com Assignee|nobody@fedoraproject.org |negativo17@gmail.com Status|NEW |ASSIGNED
https://bugzilla.redhat.com/show_bug.cgi?id=2244820
--- Comment #3 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=2244820
--- Comment #4 from Simone Caronni negativo17@gmail.com --- @ngompa13@gmail.com There's version 0.5.1 which contains your patches.
https://bugzilla.redhat.com/show_bug.cgi?id=2244820
--- Comment #5 from Neal Gompa ngompa13@gmail.com --- Update to latest version (0.5.0):
Spec URL: https://ngompa.fedorapeople.org/for-review/xevd.spec SRPM URL: https://ngompa.fedorapeople.org/for-review/xevd-0.5.0-1.fc39.src.rpm
https://bugzilla.redhat.com/show_bug.cgi?id=2244820
--- Comment #6 from Neal Gompa ngompa13@gmail.com --- Oops, wrong SRPM URL...
Update to latest version (0.5.0):
Spec URL: https://ngompa.fedorapeople.org/for-review/xevd.spec SRPM URL: https://ngompa.fedorapeople.org/for-review/xevd-0.5.0-1.fc40.src.rpm
https://bugzilla.redhat.com/show_bug.cgi?id=2244820
--- Comment #7 from Fedora Review Service fedora-review-bot@fedoraproject.org --- There seems to be some problem with the following file. SRPM URL: https://ngompa.fedorapeople.org/for-review/xevd-0.5.0-1.fc39.src.rpm Fetching it results in a 404 Not Found error. Please make sure the URL is correct and publicly available.
--- 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=2244820
--- Comment #8 from Fedora Review Service fedora-review-bot@fedoraproject.org --- Created attachment 2049473 --> https://bugzilla.redhat.com/attachment.cgi?id=2049473&action=edit The .spec file difference from Copr build 6543577 to 8091459
https://bugzilla.redhat.com/show_bug.cgi?id=2244820
Fedora Review Service fedora-review-bot@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |AutomationTriaged
--- Comment #9 from Fedora Review Service fedora-review-bot@fedoraproject.org --- Copr build: https://copr.fedorainfracloud.org/coprs/build/8091459 (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=2244820
Petr Pisar ppisar@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags| |fedora-review?
https://bugzilla.redhat.com/show_bug.cgi?id=2244820
--- Comment #10 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=2244820
Neal Gompa ngompa13@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags|fedora-review? | Blocks|182235 (FE-Legal) |
--- Comment #11 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=2244820
--- Comment #12 from Simone Caronni negativo17@gmail.com --- @ngompa13@gmail.com what does it mean removing fedora-review? FE-Legal has blocked importing?
There are a few commits in master after 0.5.0, it seems the package can be compiled as well on ARM; it might be worth investigating.
https://bugzilla.redhat.com/show_bug.cgi?id=2244820
--- Comment #13 from Neal Gompa ngompa13@gmail.com --- (In reply to Simone Caronni from comment #12)
@ngompa13@gmail.com what does it mean removing fedora-review? FE-Legal has blocked importing?
The fedora-review tag was associated with ppisar, not you, so I removed it. You need to set it yourself.
There are a few commits in master after 0.5.0, it seems the package can be compiled as well on ARM; it might be worth investigating.
I'll look into it later.
https://bugzilla.redhat.com/show_bug.cgi?id=2244820
Simone Caronni negativo17@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags| |fedora-review?
https://bugzilla.redhat.com/show_bug.cgi?id=2244820
Simone Caronni negativo17@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags|fedora-review? |fedora-review+
--- Comment #14 from Simone Caronni negativo17@gmail.com --- Package approved.
Please fix the changelog before importing (or switch to autospec):
xevd.x86_64: W: incoherent-version-in-changelog 0.4.1-1 ['0.5.0-1.fc42', '0.5.0-1']
Thanks.
https://bugzilla.redhat.com/show_bug.cgi?id=2244820
Fedora Admin user for bugzilla script actions fedora-admin-xmlrpc@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |RELEASE_PENDING
--- Comment #15 from Fedora Admin user for bugzilla script actions fedora-admin-xmlrpc@fedoraproject.org --- The Pagure repository was created at https://src.fedoraproject.org/rpms/xevd
https://bugzilla.redhat.com/show_bug.cgi?id=2244820
Dominik 'Rathann' Mierzejewski dominik@greysector.net changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RELEASE_PENDING |CLOSED Resolution|--- |RAWHIDE Fixed In Version| |xevd-0.5.0-2.fc43 Last Closed| |2025-05-11 17:01:45
package-review@lists.fedoraproject.org
-
bugzilla＠redhat.com