Spec URL: https://jonathanspw.fedorapeople.org/qsort.spec SRPM URL: https://jonathanspw.fedorapeople.org/qsort-0.2-1.fc38.src.rpm Description: This is a traditional Quicksort implementation which for the most part follows Robert Sedgewick's 1978 paper. It is implemented as a C macro, which means that comparisons can be inlined. A distinctive feature of this implementation is that it works entirely on array indices, while actual access to the array elements is abstracted out with the `less` and `swap` primitives provided by the caller. Fedora Account System Username: jonathanspw This is in relation to https://bugzilla.redhat.com/show_bug.cgi?id=2118835 to avoid bundling qsort.h.
Possible update to add a functionality test: https://copr-dist-git.fedorainfracloud.org/cgit/fed500/qsort/qsort.git/commit/?id=35632d7593c23f2767e68a11a85ac53c59fc0f59
More complete results: https://download.copr.fedorainfracloud.org/results/fed500/qsort/fedora-36-x86_64/04752816-qsort/
Added bench for x86_64. Maintaining the patches to run it on all arches is more than I'm comfortable maintaining. Spec URL: https://jonathanspw.fedorapeople.org/qsort.spec SRPM URL: https://jonathanspw.fedorapeople.org/qsort-0.2-1.fc38.src.rpm
Happy to be a co-maintainer to enable the patches to work, which probably requires someone else to review.
https://github.com/svpv/qsort/pull/2
I'd be happy to have you co-maintain the package. I don't think there's any guideline preventing you from doing the review even if the intent is for you to co-maintain it?
* Checking the guidelines for header libraries, the devel package must not be marked as no-arch [0] * You should add a comment above the patch to describe the fedora specific change [1] (mjt qsort is not packaged in fedora or something) [0] https://docs.fedoraproject.org/en-US/packaging-guidelines/#_use_noarch_only_in_subpackages [1] https://docs.fedoraproject.org/en-US/packaging-guidelines/#_fedora_specific_or_rejected_upstream_patches
This is an automatic check from review-stats script. This review request ticket hasn't been updated for some time, but it seems that the review is still being working out by you. If this is right, please respond to this comment clearing the NEEDINFO flag and try to reach out the submitter to proceed with the review. If you're not interested in reviewing this ticket anymore, please clear the fedora-review flag and reset the assignee, so that a new reviewer can take this ticket. Without any reply, this request will shortly be resetted.
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.