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
Copr build: https://copr.fedorainfracloud.org/coprs/build/6543577 (succeeded) Review template: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2244820-xevd/fedora-rawhide-x86_64/06543577-xevd/fedora-review/review.txt 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.
Requesting FE-Legal to review whether the stripping is sufficient.
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.
@ngompa13 There's version 0.5.1 which contains your patches.
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
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
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.
Created attachment 2049473 [details] The .spec file difference from Copr build 6543577 to 8091459
Copr build: https://copr.fedorainfracloud.org/coprs/build/8091459 (succeeded) Review template: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2244820-xevd/fedora-rawhide-x86_64/08091459-xevd/fedora-review/review.txt 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.
Any update from FE-Legal?... it's almost 3 months now.
Per discussion with mattdm, lifting FE-Legal.
@ngompa13 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.
(In reply to Simone Caronni from comment #12) > @ngompa13 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.
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.
The Pagure repository was created at https://src.fedoraproject.org/rpms/xevd