Spec URL: https://download.copr.fedorainfracloud.org/results/nielsenb/firejail/fedora-rawhide-x86_64/08232246-firetools/firetools.spec SRPM URL: https://download.copr.fedorainfracloud.org/results/nielsenb/firejail/fedora-rawhide-x86_64/08232246-firetools/firetools-0.9.72-3.fc42.src.rpm Description: A simple Qt5 based interface for the Firejail security sandbox. Fedora Account System Username: nielsenb Note that I have a long stalled review / sponsorship request open[0], not sure if that is relevant or not. The patch in the specfile has been submitted upstream[1]. [0] - https://bugzilla.redhat.com/show_bug.cgi?id=1350884 [1] - https://github.com/netblue30/firetools/pull/76
lgt5m. but you stated it is a gui application. pluse install desktop-file-utils and generate a .desktop file [0], to gether with a metainfo.xml file[1] [0] https://docs.fedoraproject.org/en-US/packaging-guidelines/#_desktop_files [1] https://docs.fedoraproject.org/en-US/packaging-guidelines/AppData/
Copr build: https://copr.fedorainfracloud.org/coprs/build/8371525 (succeeded) Review template: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2331188-firetools/fedora-rawhide-x86_64/08371525-firetools/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.
(In reply to solomoncyj from comment #1) > lgt5m. but you stated it is a gui application. pluse install > desktop-file-utils and generate a .desktop file [0], to gether with a > metainfo.xml file[1] > > [0] https://docs.fedoraproject.org/en-US/packaging-guidelines/#_desktop_files > [1] https://docs.fedoraproject.org/en-US/packaging-guidelines/AppData/ Spec URL: https://download.copr.fedorainfracloud.org/results/nielsenb/firejail/fedora-rawhide-x86_64/08373257-firetools/firetools.spec SRPM URL: https://download.copr.fedorainfracloud.org/results/nielsenb/firejail/fedora-rawhide-x86_64/08373257-firetools/firetools-0.9.72-4.fc42.src.rpm Added the desktop file check, as well as appdata checks. Changes to make both checks happy have been submitted upstream[0][1]. [0] - https://github.com/netblue30/firetools/pull/77 [1] - https://github.com/netblue30/firetools/pull/78
(In reply to Fedora Review Service from comment #2) > Copr build: > https://copr.fedorainfracloud.org/coprs/build/8371525 > (succeeded) > > Review template: > https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora- > review-2331188-firetools/fedora-rawhide-x86_64/08371525-firetools/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. Regarding "Sources are verified with gpgverify first in %prep if upstream publishes signatures. Note: gpgverify is not used." I would love to add a gpgverify step[0] as upstream does provide a key[1] and signed hashes. But the published asc[2] file isn't a detached signature so the macro doesn't understand it. I could feasibly "unarmor" the asc file and then run sha256 sum. Given the security nature of the package, it might be an avenue worth pursuing. [0] - https://docs.fedoraproject.org/en-US/packaging-guidelines/#_verifying_signatures [1] - https://firejailtools.wordpress.com/downloads/ [2] - https://sourceforge.net/projects/firejail/files/firetools/firetools-0.9.72.asc/download
Copr build: https://copr.fedorainfracloud.org/coprs/build/8373539 (succeeded) Review template: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2331188-firetools/fedora-rawhide-x86_64/08373539-firetools/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.
Copr build: https://copr.fedorainfracloud.org/coprs/build/8373555 (succeeded) Review template: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2331188-firetools/fedora-rawhide-x86_64/08373555-firetools/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.
This is an automatic check from review-stats script. This review request ticket hasn't been updated for some time. We're sorry it is taking so long. If you're still interested in packaging this software into Fedora repositories, please respond to this comment clearing the NEEDINFO flag. You may want to update the specfile and the src.rpm to the latest version available and to propose a review swap on Fedora devel mailing list to increase chances to have your package reviewed. If this is your first package and you need a sponsor, you may want to post some informal reviews. Read more at https://fedoraproject.org/wiki/How_to_get_sponsored_into_the_packager_group. Without any reply, this request will shortly be considered abandoned and will be closed. Thank you for your patience.
0.9.72 is the latest release of Firetools.