Spec URL: http://copr-dist-git.fedorainfracloud.org/cgit/yecheng/mediaextract/mediaextract.git/tree/mediaextract.spec SRPM URL: https://copr-be.cloud.fedoraproject.org/results/yecheng/mediaextract/fedora-rawhide-x86_64/00609486-mediaextract/mediaextract-1.0-1.fc28.src.rpm Description: Extract media files that are embedded within other files. Fedora Account System Username: yecheng
Would be better if you will use link, which can be resolved to the plain files/blobs. Your spec url should be http://copr-dist-git.fedorainfracloud.org/cgit/yecheng/mediaextract/mediaextract.git/plain/mediaextract.spec, instead of its HTML version
Koji reports failed builds: https://koji.fedoraproject.org/koji/taskinfo?taskID=22233771
Thanks. Fixed by removing -m64 and -m32 flags the upstream set in makefile, but I don't know how it will function on alternative architectures. I will test it in qemu.
Not it compiles on koji https://koji.fedoraproject.org/koji/taskinfo?taskID=22361632 Looks good to me. Are you going to remove "NotReady" once you will check it working on other platforms? If you can write some test for that, which can be applied as a patch, you can execute it automatically in the %check section of the spec.
New Spec URL: http://copr-dist-git.fedorainfracloud.org/cgit/yecheng/mediaextract/mediaextract.git/tree/mediaextract.spec New SRPM URL: http://copr-dist-git.fedorainfracloud.org/cgit/yecheng/mediaextract/mediaextract.git/plain/mediaextract.spec Adding man page.
The check section only check whether the built binary is runnable, but not whether it will run correctly. This program probably will not work correctly consistently on architectures that do not support unaligned memory access, according to upstream readme.
Spec URL: http://copr-dist-git.fedorainfracloud.org/cgit/yecheng/mediaextract/mediaextract.git/tree/mediaextract.spec SRPM URL: https://copr-be.cloud.fedoraproject.org/results/yecheng/mediaextract/fedora-rawhide-x86_64/00657935-mediaextract/mediaextract-1.0-1.fc28.src.rpm
> %patch0 > %patch1 https://fedoraproject.org/wiki/Packaging:Guidelines#Patch_Guidelines > %post > if [ $1 = 1 ] ; then > mandb > fi > > %postun > if [ $1 = 0 ] ; then > mandb > fi That's odd. First Fedora package ever I've seen that does. https://fedoraproject.org/wiki/Packaging:Guidelines#Manpages mandb is updated via systemd man-db.timer.
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.
This is an automatic action taken by review-stats script. The ticket submitter failed to clear the NEEDINFO flag in a month. As per https://fedoraproject.org/wiki/Policy_for_stalled_package_reviews we consider this ticket as DEADREVIEW and proceed to close it.