[Bug 1992877] Review Request: flintqs - William Hart's quadratic sieve

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1992877



--- Comment #2 from Jerry James <loganjerry@xxxxxxxxx> ---
Several of the patches touch the same bits of code, and github's support for
dependent pull requests is not great, so I had hoped that upstream would merge
the memory leak fix first, then I would submit the rest.  However, as you
noted, upstream is not active.  I have interacted with the primary committer
before, so I will attempt to contact him via email.

In the meantime, I submitted pull request 2 with all of the warning fixes
packed together.  The final one, falling back to /tmp if TMPDIR is not set, is
problematic.  It touches code that is changed by *both* of the existing pull
requests.  Rather than tear my hair out, I'm going to see if I can get some
movement on the other 2 pull requests first, then submit that one.

Mock switched from generating fc35 to fc36 rpms while I was working on that, so
here are new URLs:

Spec URL: https://jjames.fedorapeople.org/flintqs/flintqs.spec
SRPM URL: https://jjames.fedorapeople.org/flintqs/flintqs-1.0-1.fc36.src.rpm


-- 
You are receiving this mail because:
You are always notified about changes to this product and component
You are on the CC list for the bug.
_______________________________________________
package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/package-review@xxxxxxxxxxxxxxxxxxxxxxx
Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux