[Bug 1868850] Review Request: fcitx5-chinese-addons - Chinese related addon for fcitx5
https://bugzilla.redhat.com/show_bug.cgi?id=1868850 --- Comment #6 from Qiyu Yan --- Also, I will investigate into if this test error can be safely ignored (not easy, since I don't have a s390x testing environment.) And this problem occurs in debian builds to, maybe they can provide some help. Similar to plans before, if we can't ignore this failure, I will keep the ExcludeArch and ask upstream for help, if otherwise, I will ask FPC for an exception. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1857584] Review Request: python-serpent - Serialization based on ast.literal_eval
https://bugzilla.redhat.com/show_bug.cgi?id=1857584 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #4 from Fedora Update System --- FEDORA-2020-2387058a2d has been pushed to the Fedora 32 testing repository. In short time you'll be able to install the update with the following command: `sudo dnf install --enablerepo=updates-testing --advisory=FEDORA-2020-2387058a2d \*` You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2020-2387058a2d See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1868850] Review Request: fcitx5-chinese-addons - Chinese related addon for fcitx5
https://bugzilla.redhat.com/show_bug.cgi?id=1868850 --- Comment #5 from Qiyu Yan --- This is harder to fix, I am contacting upstream for help. And I added a ExcludeArch, and would file a build-fail ticket then -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1868850] Review Request: fcitx5-chinese-addons - Chinese related addon for fcitx5
https://bugzilla.redhat.com/show_bug.cgi?id=1868850 Qiyu Yan changed: What|Removed |Added Comment|0 |updated --- Comment #0 has been edited --- Spec URL: https://download.copr.fedorainfracloud.org/results/yanqiyu/fcitx5/fedora-33-x86_64/01628036-fcitx5-chinese-addons/fcitx5-chinese-addons.spec SRPM URL: https://download.copr.fedorainfracloud.org/results/yanqiyu/fcitx5/fedora-33-x86_64/01628036-fcitx5-chinese-addons/fcitx5-chinese-addons-0-0.2.20200812gitef9beb7.fc33.src.rpm Description: Chinese related addon for fcitx5 Fedora Account System Username: yanqiyu -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1858639] Review Request: qt-avif-image-plugin - Qt plug-in to read/write AVIF images
https://bugzilla.redhat.com/show_bug.cgi?id=1858639 --- Comment #6 from Michel Alexandre Salim --- Ack, re-reviewing -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1868992] Review Request: jakarta-server-pages - Jakarta Server Pages (JSP)
https://bugzilla.redhat.com/show_bug.cgi?id=1868992 --- Comment #2 from Mat Booth --- Issues: 1) Hmm, just like jakarta-servlet package, half the source files have "ASL 2.0" headers instead of the "EPL-2.0 or GPLv2", so I think the license field would be more accurate as this: ASL 2.0 and (EPL-2.0 or GPLv2 with exceptions) I read the upstream bug you filed for servlet-api ( https://github.com/eclipse-ee4j/servlet-api/issues/347 ) and it seemed to take them by surprise! So probably worth filing a bug upstream for this one too. 2) For some boring technical reasons, Eclipse requires the JDTJavaCompiler adapter implementation, which in the old package was enabled by this patch: https://src.fedoraproject.org/rpms/glassfish-jsp/blob/84ff3bc904d196045a9ec2d5140391847dd3/f/0001-Enable-Eclipse-JDT-compiler-support-for-runtime-JSP-.patch This does not appear to have made the transition. The new package instead deletes JDTJavaCompiler and removes the dep on ecj. Any chance this can be restored? I know it's a deviation from the upstream's distribution available from maven central, but the Eclipse Foundation ships it's own build with this addition to allow JSP to use Eclipse's built-in ECJ Java compiler for compiling JSP pages at runtime. 3) The "compat namespace" (cp -pr && sed s/javax/jakarta/) causes compilation errors in Eclipse. The problem is due to mixed usage of javax.* and jakarta.* classes when referencing classes directly from the implementation jar. Eclipse does stuff like this example: import javax.servlet.Servlet; // An API class import org.apache.jasper.servlet.JspServlet; // An impl class Servlet s = new JspServlet; HOWEVER, JspServlet extends "jakarta.servlet.Servlet" and NOT "javax.servlet.Servlet" which makes JspServlet unassignable to Servlet in this context (type mismatch, compilation error) So although this compat "hack" works fine when only the API classes are used (javax.* classes) it fails when we try to use classes from the implementation jar without fully migrating to jakarta.* APIs. I need a bit of time to think about how best to solve this. It looks like we only reference JSP implementation classes a couple times in Eclipse code, so perhaps I will be able to come up with a concise patch there. Or do something cleverer in this package and jakarta-servlet package. Hmmm For completion, the full review tool output follows below. FTR, obsoletes/provides look fine to me, I don't know why the review tool does not enumerate the obsoletes as it does with the provides... Package Review == Legend: [x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated = MUST items = Generic: [x]: Package is licensed with an open-source compatible license and meets other legal requirements as defined in the legal section of Packaging Guidelines. [!]: License field in the package spec file matches the actual license. Note: Checking patched sources after %prep for licenses. Licenses found: "Unknown or generated", "*No copyright* Eclipse Public License 2.0", "Eclipse Public License 2.0", "Apache License 2.0". 45 files have unknown license. Detailed output of licensecheck in /home/mbooth/fedora/1868992-jakarta-server-pages/licensecheck.txt [x]: License file installed when any subpackage combination is installed. [!]: If the package is under multiple licenses, the licensing breakdown must be documented in the spec. [x]: Package contains no bundled libraries without FPC exception. [x]: Changelog in prescribed format. [x]: Sources contain only permissible code or content. [-]: Package contains desktop file if it is a GUI application. [-]: Development files must be in a -devel package [x]: Package uses nothing in %doc for runtime. [x]: Package consistently uses macros (instead of hard-coded directory names). [x]: Package is named according to the Package Naming Guidelines. [x]: Package does not generate any conflict. [x]: Package obeys FHS, except libexecdir and /usr/target. [x]: If the package is a rename of another package, proper Obsoletes and Provides are present. [x]: Requires correct, justified where necessary. [x]: Spec file is legible and written in American English. [-]: Package contains systemd file(s) if in need. [x]: Package is not known to require an ExcludeArch tag. [-]: Large documentation must go in a -doc subpackage. Large could be size (~1MB) or number of files. Note: Documentation size is 10240 bytes in 1 files. [x]: Package complies to the Packaging Guidelines [x]: Package successfully compiles and builds into binary rpms on at least one supported primary architecture. [x]: Package installs properly. [x]: Rpmlint is run on all rpms the build produces. Note: There are rpmlint messages (see attachment). [x]: If (and only if) the source package includes the text of the license(s) in its own file, then that file, containing the text of the license(s) for the
[Bug 1858639] Review Request: qt-avif-image-plugin - Qt plug-in to read/write AVIF images
https://bugzilla.redhat.com/show_bug.cgi?id=1858639 --- Comment #5 from Robert-André Mauchin --- New Spec URL: https://eclipseo.fedorapeople.org/for-review/qt-avif-image-plugin.spec New SRPM URL: https://eclipseo.fedorapeople.org/for-review/qt-avif-image-plugin-0.3.1-1.fc34.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1871908] Review Request: python-aiohomekit - Python HomeKit client
https://bugzilla.redhat.com/show_bug.cgi?id=1871908 --- Comment #3 from Fabian Affolter --- %changelog * Mon Aug 24 2020 Fabian Affolter - 0.2.50-1 - LICENSE file is now present (rhbz#1871908) Updated files: Spec URL: https://fab.fedorapeople.org/packages/SRPMS/python-aiohomekit.spec SRPM URL: https://fab.fedorapeople.org/packages/SRPMS/python-aiohomekit-0.2.50-1.fc32.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1871908] Review Request: python-aiohomekit - Python HomeKit client
https://bugzilla.redhat.com/show_bug.cgi?id=1871908 --- Comment #2 from Fabian Affolter --- PR was merged and upstream released 0.2.50 with the LICENSE file that has content. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1871997] Review Request: (unretiring) menulibre - MenuLibre is an advanced FreeDesktop.org compliant menu editor
https://bugzilla.redhat.com/show_bug.cgi?id=1871997 Spec URL: https://lyessaadi.fedorapeople.org/menulibre/menulibre.spec SOURCE1: https://lyessaadi.fedorapeople.org/menulibre/menulibre.appdata.xml Upstream Bug: https://github.com/bluesabre/menulibre/issues/61 SRPM URL: https://lyessaadi.fedorapeople.org/menulibre/menulibre-2.2.1-3.fc32.src.rpm Koji Build: https://koji.fedoraproject.org/koji/taskinfo?taskID=50086730 Description: MenuLibre is an advanced FreeDesktop.org compliant menu editor. All fields specified in the FreeDesktop.org Desktop Entry and Menu specifications are available to quickly update. Additionally, MenuLibre provides an editor for the launcher actions used by applications such as Unity and Plank. Features: - A beautiful interface powered by the latest version of GTK+. - Create new launchers, or modify existing ones with complete control over common settings and access to advanced settings. - Add, remove, and adjust desktop actions: powerful shortcuts available used by Unity, Xfce, and Pantheon. - Easily rearrange menu items to suit your needs. Fedora Account System Username: lyessaadi Note: I am unretiring this package (I made some changes to the original spec file). -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1842225] Review Request: python-authlib - Build OAuth and OpenID Connect servers
https://bugzilla.redhat.com/show_bug.cgi?id=1842225 Robert-André Mauchin changed: What|Removed |Added Blocks|177841 (FE-NEEDSPONSOR) | Flags|fedora-review+ |fedora-review? Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=177841 [Bug 177841] Tracker: Review requests from new Fedora packagers who need a sponsor -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1842225] Review Request: python-authlib - Build OAuth and OpenID Connect servers
https://bugzilla.redhat.com/show_bug.cgi?id=1842225 Robert-André Mauchin changed: What|Removed |Added Status|ASSIGNED|POST Flags|fedora-review? |fedora-review+ --- Comment #2 from Robert-André Mauchin --- Refreshed flag + sponsored. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1871997] New: Review Request: (unretiring) menulibre - MenuLibre is an advanced FreeDesktop.org compliant menu editor
https://bugzilla.redhat.com/show_bug.cgi?id=1871997 Bug ID: 1871997 Summary: Review Request: (unretiring) menulibre - MenuLibre is an advanced FreeDesktop.org compliant menu editor Product: Fedora Version: rawhide Hardware: All OS: Linux Status: NEW Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: fed...@lyes.eu QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Target Milestone: --- Classification: Fedora -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1871908] Review Request: python-aiohomekit - Python HomeKit client
https://bugzilla.redhat.com/show_bug.cgi?id=1871908 --- Comment #1 from Andy Mender --- All is good. We just need that license file. Package Review == Legend: [x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated [ ] = Manual review needed Issues: === - Package installs properly. Note: Installation errors (see attachment) See: https://docs.fedoraproject.org/en-US/packaging-guidelines/ Review: Problems with mock. Ignore this warning. = MUST items = Generic: [x]: Package successfully compiles and builds into binary rpms on at least one supported primary architecture. Note: Using prebuilt packages Review: Tested in Koji by the submitter. [x]: Package is licensed with an open-source compatible license and meets other legal requirements as defined in the legal section of Packaging Guidelines. [ ]: If (and only if) the source package includes the text of the license(s) in its own file, then that file, containing the text of the license(s) for the package is included in %license. Review: Leaving this one empty, since a ticket with upstream was opened. [x]: License field in the package spec file matches the actual license. Note: Checking patched sources after %prep for licenses. Licenses found: "Unknown or generated", "Apache License 2.0". 8 files have unknown license. Detailed output of licensecheck in /home/amender/rpmbuild/SPECS/python-aiohomekit/python- aiohomekit/licensecheck.txt Review: Yes, but see above point. [x]: Package contains no bundled libraries without FPC exception. [x]: Changelog in prescribed format. [x]: Sources contain only permissible code or content. [-]: Package contains desktop file if it is a GUI application. [-]: Development files must be in a -devel package [?]: Package uses nothing in %doc for runtime. [x]: Package consistently uses macros (instead of hard-coded directory names). [x]: Package is named according to the Package Naming Guidelines. [x]: Package does not generate any conflict. [x]: Package obeys FHS, except libexecdir and /usr/target. [-]: If the package is a rename of another package, proper Obsoletes and Provides are present. [x]: Requires correct, justified where necessary. [x]: Spec file is legible and written in American English. [-]: Package contains systemd file(s) if in need. [x]: Package is not known to require an ExcludeArch tag. Review: It's a Python noarch package. [x]: Large documentation must go in a -doc subpackage. Large could be size (~1MB) or number of files. Note: Documentation size is 10240 bytes in 1 files. [x]: Package complies to the Packaging Guidelines [x]: Rpmlint is run on all rpms the build produces. Note: There are rpmlint messages (see attachment). [x]: Package requires other packages for directories it uses. [x]: Package must own all directories that it creates. [x]: Package does not own files or directories owned by other packages. [x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT [x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the beginning of %install. [x]: Macros in Summary, %description expandable at SRPM build time. [x]: Dist tag is present. [x]: Package does not contain duplicates in %files. [x]: Permissions on files are set properly. [x]: Package must not depend on deprecated() packages. [x]: Package use %makeinstall only when make install DESTDIR=... doesn't work. [x]: Package is named using only allowed ASCII characters. [x]: Package does not use a name that already exists. [x]: Package is not relocatable. [x]: Sources used to build the package match the upstream source, as provided in the spec URL. [x]: Spec file name must match the spec package %{name}, in the format %{name}.spec. [x]: File names are valid UTF-8. [x]: Packages must not store files under /srv, /opt or /usr/local Python: [x]: Python eggs must not download any dependencies during the build process. [x]: A package which is used by another package via an egg interface should provide egg info. [x]: Package meets the Packaging Guidelines::Python [x]: Package contains BR: python2-devel or python3-devel [x]: Packages MUST NOT have dependencies (either build-time or runtime) on packages named with the unversioned python- prefix unless no properly versioned package exists. Dependencies on Python packages instead MUST use names beginning with python2- or python3- as appropriate. [x]: Python packages must not contain %{pythonX_site(lib|arch)}/* in %files [x]: Binary eggs must be removed in %prep = SHOULD items = Generic: [x]: Reviewer should test that the package builds in mock. [x]: If the source package does not include license text(s) as a separate file from upstream, the packager SHOULD query upstream to include it. Review: ticket with upstream https://github.com/Jc2k/aiohomekit/pull/19 [x]: Final provides and requires are sane
[Bug 1871908] Review Request: python-aiohomekit - Python HomeKit client
https://bugzilla.redhat.com/show_bug.cgi?id=1871908 Andy Mender changed: What|Removed |Added Status|NEW |ASSIGNED CC||andymenderu...@gmail.com Assignee|nob...@fedoraproject.org|andymenderu...@gmail.com Flags||fedora-review? -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1868850] Review Request: fcitx5-chinese-addons - Chinese related addon for fcitx5
https://bugzilla.redhat.com/show_bug.cgi?id=1868850 --- Comment #4 from Andy Mender --- > - removed ExcludeArch > - switched to pkgconfig(curl) > - added -data subpackage > - more specific on /usr/share/* Nice! Koji build based on latest changes: https://koji.fedoraproject.org/koji/taskinfo?taskID=50083236 I triggered it again just in case, but unfortunately 1 test now fails on the s390x architecture: https://koji.fedoraproject.org/koji/taskinfo?taskID=50083448 Here's the error from the tail of the log: F2020-08-24 18:50:41.880776 testpinyin.cpp:23] pinyin failed 67% tests passed, 1 tests failed out of 3 Total Test time (real) = 53.79 sec The following tests FAILED: 3 - testpinyin (Child aborted) Errors while running CTest error: Bad exit status from /var/tmp/rpm-tmp.o9qgTj (%check) RPM build errors: Bad exit status from /var/tmp/rpm-tmp.o9qgTj (%check) Child return code was: 1 EXCEPTION: [Error()] -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1868853] Review Request: fcitx5-configtool - Configuration tools used by fcitx5
https://bugzilla.redhat.com/show_bug.cgi?id=1868853 Andy Mender changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #5 from Andy Mender --- Koji build based on latest changes: https://koji.fedoraproject.org/koji/taskinfo?taskID=50082338 > they doesn't provide pkgconfig(foo) but have > cmake(foo), while cmake(foo) was introduced recently, some package didn't > build with that. On f33+, mostly yes, on f32 some don't have tags like this. We can do without it in this case. > Just in case upstream add content to readme, I won't forget to include that. Fair point. Let's keep it then. > added > - Requires: kf5-kcmutils Verified: [x]: Package requires other packages for directories it uses. [x]: Package must own all directories that it creates. Package approved! -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1861839] Review Request: php-openpsa-universalfeedcreator - RSS and Atom feed generator
https://bugzilla.redhat.com/show_bug.cgi?id=1861839 --- Comment #10 from Gwyn Ciesla --- (fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/php-openpsa-universalfeedcreator -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1871908] New: Review Request: python-aiohomekit - Python HomeKit client
https://bugzilla.redhat.com/show_bug.cgi?id=1871908 Bug ID: 1871908 Summary: Review Request: python-aiohomekit - Python HomeKit client Product: Fedora Version: rawhide Hardware: All OS: Linux Status: NEW Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: m...@fabian-affolter.ch QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Target Milestone: --- Classification: Fedora Spec URL: https://fab.fedorapeople.org/packages/SRPMS/python-aiohomekit.spec SRPM URL: https://fab.fedorapeople.org/packages/SRPMS/python-aiohomekit-0.2.49-1.fc32.src.rpm Project URL: https://github.com/Jc2k/aiohomekit Description: This library implements the HomeKit protocol for controlling Homekit accessories. Koji scratch build: https://koji.fedoraproject.org/koji/taskinfo?taskID=50070619 rpmlint output: $ rpmlint python-aiohomekit-0.2.49-1.fc32.src.rpm python-aiohomekit.src:39: W: macro-in-comment %license 1 packages and 0 specfiles checked; 0 errors, 1 warnings. $ rpmlint python3-aiohomekit-0.2.49-1.fc32.noarch.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. Fedora Account System Username: fab -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1871908] Review Request: python-aiohomekit - Python HomeKit client
https://bugzilla.redhat.com/show_bug.cgi?id=1871908 Fabian Affolter changed: What|Removed |Added Blocks||1269538 (IoT) Alias||python-aiohomekit Doc Type|--- |If docs needed, set a value Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1269538 [Bug 1269538] Tracker for IoT on Fedora -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1867999] Review Request: perl-GeoIP2 - Perl API for MaxMind's GeoIP2 web services and databases
https://bugzilla.redhat.com/show_bug.cgi?id=1867999 Petr Pisar changed: What|Removed |Added Status|ASSIGNED|CLOSED Fixed In Version||perl-GeoIP2-2.006002-1.fc34 Resolution|--- |RAWHIDE Last Closed||2020-08-24 14:10:55 --- Comment #4 from Petr Pisar --- Thank you for the review and the repository. I fixed both the issues. Although I'm curious how your build end up with /usr/bin/env shebang if rpmbuild rewrites on build for some time. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1861839] Review Request: php-openpsa-universalfeedcreator - RSS and Atom feed generator
https://bugzilla.redhat.com/show_bug.cgi?id=1861839 --- Comment #9 from Remi Collet --- (In reply to Artur Iwicki from comment #8) > Thanks for the review. Please assign the ticket to yourself (required per > "fedpkg request-repo" procedures). Sorry about this, done. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1861839] Review Request: php-openpsa-universalfeedcreator - RSS and Atom feed generator
https://bugzilla.redhat.com/show_bug.cgi?id=1861839 Remi Collet changed: What|Removed |Added Assignee|nob...@fedoraproject.org|fed...@famillecollet.com -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1861839] Review Request: php-openpsa-universalfeedcreator - RSS and Atom feed generator
https://bugzilla.redhat.com/show_bug.cgi?id=1861839 --- Comment #8 from Artur Iwicki --- Thanks for the review. Please assign the ticket to yourself (required per "fedpkg request-repo" procedures). -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1871825] Review Request: python-ratelimiter - Python module providing rate limiting
https://bugzilla.redhat.com/show_bug.cgi?id=1871825 Nasir Hussain changed: What|Removed |Added CC||nasirhussain...@gmail.com --- Comment #1 from Nasir Hussain --- Looks like a simple Py3 library, LGTM. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1867999] Review Request: perl-GeoIP2 - Perl API for MaxMind's GeoIP2 web services and databases
https://bugzilla.redhat.com/show_bug.cgi?id=1867999 --- Comment #3 from Gwyn Ciesla --- (fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/perl-GeoIP2 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1867999] Review Request: perl-GeoIP2 - Perl API for MaxMind's GeoIP2 web services and databases
https://bugzilla.redhat.com/show_bug.cgi?id=1867999 --- Comment #2 from Petr Pisar --- https://pagure.io/releng/fedora-scm-requests/issue/27854 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1871837] Review Request: golang-github-ubccr-kerby - Go wrapper for Kerberos GSSAPI
https://bugzilla.redhat.com/show_bug.cgi?id=1871837 obu...@redhat.com changed: What|Removed |Added Comment|0 |updated --- Comment #0 has been edited --- Spec URL: https://download.copr.fedorainfracloud.org/results/obudai/kerby/fedora-rawhide-x86_64/01629888-golang-github-ubccr-kerby/golang-github-ubccr-kerby.spec SRPM URL: golang-github-ubccr-kerby-0-0.1.20200824git201a958.fc34.src.rpm Description: Go wrapper for Kerberos GSSAPI Fedora Account System Username: obudai Koji scratch build: https://koji.fedoraproject.org/koji/taskinfo?taskID=50042441 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1867999] Review Request: perl-GeoIP2 - Perl API for MaxMind's GeoIP2 web services and databases
https://bugzilla.redhat.com/show_bug.cgi?id=1867999 Jitka Plesnikova changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #1 from Jitka Plesnikova --- Source file is ok Summary is ok License is ok Description is ok URL and Source0 are ok All tests passed BuildRequires are almost ok FIX: Please add perl(lib) - GeoIP2-2.006002/t/GeoIP2/Model/City.t:4 $ rpm -qp --requires perl-GeoIP2-2.006002-1.fc34.noarch.rpm | sort | uniq -c | grep -v rpmlib 1 perl(B) 1 perl(Data::Dumper) 1 perl(Data::Validate::IP) >= 0.25 1 perl(Exporter) 1 perl(GeoIP2::Error::Generic) 1 perl(GeoIP2::Error::HTTP) 1 perl(GeoIP2::Error::IPAddressNotFound) 1 perl(GeoIP2::Error::Type) 1 perl(GeoIP2::Error::WebService) 1 perl(GeoIP2::Model::AnonymousIP) 1 perl(GeoIP2::Model::ASN) 1 perl(GeoIP2::Model::City) 1 perl(GeoIP2::Model::ConnectionType) 1 perl(GeoIP2::Model::Country) 1 perl(GeoIP2::Model::Domain) 1 perl(GeoIP2::Model::Enterprise) 1 perl(GeoIP2::Model::Insights) 1 perl(GeoIP2::Model::ISP) 1 perl(GeoIP2::Record::City) 1 perl(GeoIP2::Record::Continent) 1 perl(GeoIP2::Record::Country) 1 perl(GeoIP2::Record::Location) 1 perl(GeoIP2::Record::MaxMind) 1 perl(GeoIP2::Record::Postal) 1 perl(GeoIP2::Record::RepresentedCountry) 1 perl(GeoIP2::Record::Subdivision) 1 perl(GeoIP2::Record::Traits) 1 perl(GeoIP2::Types) 1 perl(GeoIP2::WebService::Client) 1 perl(Getopt::Long) 1 perl(HTTP::Headers) 1 perl(HTTP::Request) 1 perl(JSON::MaybeXS) 1 perl(List::SomeUtils) 1 perl(List::Util) 1 perl(LWP::Protocol::https) 1 perl(LWP::UserAgent) 1 perl(MaxMind::DB::Reader) >= 1.00 1 perl(MIME::Base64) 1 perl(:MODULE_COMPAT_5.32.0) 1 perl(Moo) 1 perl(Moo::Role) 1 perl(namespace::clean) 1 perl(Params::Validate) 1 perl(Scalar::Util) 1 perl(strict) 1 perl(Sub::Quote) 1 perl(Throwable::Error) 1 perl(Try::Tiny) 1 perl(URI) 1 perl(:VERSION) >= 5.8.0 1 perl(warnings) 1 /usr/bin/env Binary requires are Ok. $ rpm -qp --provides perl-GeoIP2-2.006002-1.fc34.noarch.rpm | sort | uniq -c 1 perl(GeoIP2) = 2.006002 1 perl-GeoIP2 = 2.006002-1.fc34 1 perl(GeoIP2::Database::Reader) = 2.006002 1 perl(GeoIP2::Error::Generic) = 2.006002 1 perl(GeoIP2::Error::HTTP) = 2.006002 1 perl(GeoIP2::Error::IPAddressNotFound) = 2.006002 1 perl(GeoIP2::Error::Type) = 2.006002 1 perl(GeoIP2::Error::WebService) = 2.006002 1 perl(GeoIP2::Model::AnonymousIP) = 2.006002 1 perl(GeoIP2::Model::ASN) = 2.006002 1 perl(GeoIP2::Model::City) = 2.006002 1 perl(GeoIP2::Model::ConnectionType) = 2.006002 1 perl(GeoIP2::Model::Country) = 2.006002 1 perl(GeoIP2::Model::Domain) = 2.006002 1 perl(GeoIP2::Model::Enterprise) = 2.006002 1 perl(GeoIP2::Model::Insights) = 2.006002 1 perl(GeoIP2::Model::ISP) = 2.006002 1 perl(GeoIP2::Record::City) = 2.006002 1 perl(GeoIP2::Record::Continent) = 2.006002 1 perl(GeoIP2::Record::Country) = 2.006002 1 perl(GeoIP2::Record::Location) = 2.006002 1 perl(GeoIP2::Record::MaxMind) = 2.006002 1 perl(GeoIP2::Record::Postal) = 2.006002 1 perl(GeoIP2::Record::RepresentedCountry) = 2.006002 1 perl(GeoIP2::Record::Subdivision) = 2.006002 1 perl(GeoIP2::Record::Traits) = 2.006002 1 perl(GeoIP2::Role::Error::HTTP) = 2.006002 1 perl(GeoIP2::Role::HasIPAddress) = 2.006002 1 perl(GeoIP2::Role::HasLocales) = 2.006002 1 perl(GeoIP2::Role::Model) = 2.006002 1 perl(GeoIP2::Role::Model::Flat) = 2.006002 1 perl(GeoIP2::Role::Model::HasSubdivisions) = 2.006002 1 perl(GeoIP2::Role::Model::Location) = 2.006002 1 perl(GeoIP2::Role::Record::Country) = 2.006002 1 perl(GeoIP2::Role::Record::HasNames) = 2.006002 1 perl(GeoIP2::Types) = 2.006002 Binary provides are Ok. $ rpmlint ./perl-GeoIP2* perl-GeoIP2.noarch: E: env-script-interpreter /usr/bin/geoip2-web-service-request /usr/bin/env perl perl-GeoIP2.noarch: W: no-manual-page-for-binary geoip2-web-service-request 2 packages and 1 specfiles checked; 1 errors, 1 warnings. FIX: Please update the script interpreter Please correct all 'FIX' issues. Otherwise the package looks good. Resolution: Approved -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org Fedora Code of Conduct:
[Bug 1871837] New: Review Request: golang-github-ubccr-kerby - Go wrapper for Kerberos GSSAPI
https://bugzilla.redhat.com/show_bug.cgi?id=1871837 Bug ID: 1871837 Summary: Review Request: golang-github-ubccr-kerby - Go wrapper for Kerberos GSSAPI Product: Fedora Version: rawhide Hardware: All OS: Linux Status: NEW Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: obu...@redhat.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Target Milestone: --- Classification: Fedora Spec URL: https://download.copr.fedorainfracloud.org/results/obudai/kerby/fedora-rawhide-x86_64/01629888-golang-github-ubccr-kerby/golang-github-ubccr-kerby.spec SRPM URL: golang-github-ubccr-kerby-0-0.1.20200824git201a958.fc34.src.rpm Description: Go wrapper for Kerberos GSSAPI Fedora Account System Username: obudai Koji scratch build: https://koji.fedoraproject.org/koji/taskinfo?taskID=50042441 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1869719] Review Request: rubygem-image_processing - High-level wrapper for processing images for the web with ImageMagick or libvips
https://bugzilla.redhat.com/show_bug.cgi?id=1869719 --- Comment #8 from Vít Ondruch --- (In reply to Pavel Valena from comment #6) > > * `skip` vs `return` > > To me, the benefit of `skip` is that I see "S" in the test output, > so I know the tests are skipped actually. `return` would just hide the > issue, right? That is nice of course, but if the test case contains more asserts, the `skip` won't execute the following asserts, while the `return` does. So what are pros/cons? :) -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1821189] Package Review: Snakemake - Workflow management system to create reproducible and scalable data analyses
https://bugzilla.redhat.com/show_bug.cgi?id=1821189 Aniket Pradhan changed: What|Removed |Added Depends On||1871824, 1871825 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1871824 [Bug 1871824] Review Request: python-toposort - Implements a topological sort algorithm https://bugzilla.redhat.com/show_bug.cgi?id=1871825 [Bug 1871825] Review Request: python-ratelimiter - Python module providing rate limiting -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1871824] Review Request: python-toposort - Implements a topological sort algorithm
https://bugzilla.redhat.com/show_bug.cgi?id=1871824 Aniket Pradhan changed: What|Removed |Added Blocks||1821189 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1821189 [Bug 1821189] Package Review: Snakemake - Workflow management system to create reproducible and scalable data analyses -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1871825] Review Request: python-ratelimiter - Python module providing rate limiting
https://bugzilla.redhat.com/show_bug.cgi?id=1871825 Aniket Pradhan changed: What|Removed |Added Blocks||1821189 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1821189 [Bug 1821189] Package Review: Snakemake - Workflow management system to create reproducible and scalable data analyses -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1871824] Review Request: python-toposort - Implements a topological sort algorithm
https://bugzilla.redhat.com/show_bug.cgi?id=1871824 Aniket Pradhan changed: What|Removed |Added Blocks||1276941 (fedora-neuro) Doc Type|--- |If docs needed, set a value Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1276941 [Bug 1276941] Fedora NeuroImaging and NeuroScience tracking bug -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1871825] Review Request: python-ratelimiter - Python module providing rate limiting
https://bugzilla.redhat.com/show_bug.cgi?id=1871825 Aniket Pradhan changed: What|Removed |Added Blocks||1276941 (fedora-neuro) Doc Type|--- |If docs needed, set a value Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1276941 [Bug 1276941] Fedora NeuroImaging and NeuroScience tracking bug -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1871825] New: Review Request: python-ratelimiter - Python module providing rate limiting
https://bugzilla.redhat.com/show_bug.cgi?id=1871825 Bug ID: 1871825 Summary: Review Request: python-ratelimiter - Python module providing rate limiting Product: Fedora Version: rawhide Hardware: All OS: Linux Status: NEW Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: aniketpradhan1...@gmail.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Target Milestone: --- Classification: Fedora Spec URL: https://major.fedorapeople.org/python-ratelimiter/python-ratelimiter.spec SRPM URL: https://major.fedorapeople.org/python-ratelimiter/python3-ratelimiter-1.2.0.post0-1.fc32.src.rpm Description: This package provides the ratelimiter module, which ensures that an operation will not be executed more than a given number of times on a given period. This can prove useful when working with third parties APIs which require for example a maximum of 10 requests per second. Fedora Account System Username: major Koji scratch build: https://koji.fedoraproject.org/koji/taskinfo?taskID=50051239 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1871824] New: Review Request: python-toposort - Implements a topological sort algorithm
https://bugzilla.redhat.com/show_bug.cgi?id=1871824 Bug ID: 1871824 Summary: Review Request: python-toposort - Implements a topological sort algorithm Product: Fedora Version: rawhide Hardware: All OS: Linux Status: NEW Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: aniketpradhan1...@gmail.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Target Milestone: --- Classification: Fedora Spec URL: https://major.fedorapeople.org/python-toposort/python-toposort.spec SRPM URL: https://major.fedorapeople.org/python-toposort/python3-toposort-1.5-1.fc32.src.rpm Description: In computer science, a topological sort (sometimes abbreviated topsort or toposort) or topological ordering of a directed graph is a linear ordering of its vertices such that for every directed edge uv from vertex u to vertex v, u comes before v in the ordering. Fedora Account System Username: major Koji scratch build: https://koji.fedoraproject.org/koji/taskinfo?taskID=50051232 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1824156] Review Request: exfatprogs - Userspace utilities for exFAT filesystems
https://bugzilla.redhat.com/show_bug.cgi?id=1824156 --- Comment #35 from Simone Caronni --- (In reply to Peter Robinson from comment #34) > See https://pagure.io/Fedora-Council/tickets/issue/311 for details. > Everything is still in motion, handled currently by Matthew and Ben I get a 404 on that URL, but good to hear. Thanks! -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1871813] New: Review Request: python-aioesphomeapi - Library to interact with devices flashed with esphome
https://bugzilla.redhat.com/show_bug.cgi?id=1871813 Bug ID: 1871813 Summary: Review Request: python-aioesphomeapi - Library to interact with devices flashed with esphome Product: Fedora Version: rawhide Hardware: All OS: Linux Status: NEW Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: m...@fabian-affolter.ch QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Target Milestone: --- Classification: Fedora Spec URL: https://fab.fedorapeople.org/packages/SRPMS/python-aioesphomeapi.spec SRPM URL: https://fab.fedorapeople.org/packages/SRPMS/python-aioesphomeapi-2.6.2-1.fc32.src.rpm Project URL: https://esphome.io/ Description: aioesphomeapi allows you to interact with devices flashed with esphome. Koji scratch build: https://koji.fedoraproject.org/koji/taskinfo?taskID=50049193 rpmlint output: $ rpmlint python-aioesphomeapi-2.6.2-1.fc32.src.rpm python-aioesphomeapi.src: W: spelling-error Summary(en_US) esphome -> esp home, esp-home, homespun python-aioesphomeapi.src: W: spelling-error %description -l en_US esphome -> esp home, esp-home, homespun 1 packages and 0 specfiles checked; 0 errors, 2 warnings. $ rpmlint python3-aioesphomeapi-2.6.2-1.fc32.noarch.rpm python3-aioesphomeapi.noarch: W: spelling-error Summary(en_US) esphome -> esp home, esp-home, homespun python3-aioesphomeapi.noarch: W: spelling-error %description -l en_US esphome -> esp home, esp-home, homespun 1 packages and 0 specfiles checked; 0 errors, 2 warnings. Fedora Account System Username: fab -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1824156] Review Request: exfatprogs - Userspace utilities for exFAT filesystems
https://bugzilla.redhat.com/show_bug.cgi?id=1824156 --- Comment #34 from Peter Robinson --- (In reply to Simone Caronni from comment #33) > Just saw that Tom is no longer working for Red Hat.. does this change > anything regarding the legal approval? He has been removed from CC in this > ticket. > > I would set a needinfo to some address but I wouldn't know who to... See https://pagure.io/Fedora-Council/tickets/issue/311 for details. Everything is still in motion, handled currently by Matthew and Ben -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1809405] Review Request: python-sumatra - Tool for managing and tracking projects based on numerical simulation and/or analysis, with the aim of supporting reproducible research
https://bugzilla.redhat.com/show_bug.cgi?id=1809405 --- Comment #24 from Ankur Sinha (FranciscoD) --- Thanks! I'll have a look this week. On my list now. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1856005] Review Request: dmtcp - Checkpoint/Restart functionality for Linux processes
https://bugzilla.redhat.com/show_bug.cgi?id=1856005 --- Comment #14 from Dave Love --- I don't know about Orion, wouldn't worry about a build time of half an hour with tests. I'm more worried about not being able to get the tests to pass if I make sure they're not skipped (which they still are in the build). One or more fail in mock on el7 and f33, as well as with a straight build from the tarball on centos7 and debian 10. I don't have time to try to debug them. Can you say what's going wrong and how significant it is? Koji examples: https://koji.fedoraproject.org/koji/taskinfo?taskID=49885693 https://koji.fedoraproject.org/koji/taskinfo?taskID=49893986 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1871345] Review Request: jakarta-saaj - SOAP with Attachments API for Java
https://bugzilla.redhat.com/show_bug.cgi?id=1871345 Fabio Valentini changed: What|Removed |Added Status|POST|CLOSED Fixed In Version||jakarta-saaj-1.4.2-1.fc33 Resolution|--- |NEXTRELEASE Last Closed||2020-08-24 08:41:37 --- Comment #5 from Fabio Valentini --- Built for rawhide: https://koji.fedoraproject.org/koji/buildinfo?buildID=1599021 And for fedora 33: https://koji.fedoraproject.org/koji/buildinfo?buildID=1599022 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1871350] Review Request: jakarta-xml-rpc - Jakarta XML RPC API
https://bugzilla.redhat.com/show_bug.cgi?id=1871350 Bug 1871350 depends on bug 1871345, which changed state. Bug 1871345 Summary: Review Request: jakarta-saaj - SOAP with Attachments API for Java https://bugzilla.redhat.com/show_bug.cgi?id=1871345 What|Removed |Added Status|POST|CLOSED Resolution|--- |NEXTRELEASE -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1871765] Review Request: thoth-toolbox - This is a toolbox container image for Thoth tools
https://bugzilla.redhat.com/show_bug.cgi?id=1871765 Christoph Görn changed: What|Removed |Added Comment|0 |updated --- Comment #0 has been edited --- Spec URL: n/a, its just a container image SRPM URL: n/a, its just a container image Description: This is a toolbox container image aggregating some of Thoth tools, including micropipenv and thamos Fedora Account System Username: goern -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1871765] New: Review Request: thoth-toolbox - This is a toolbox container image for Thoth tools
https://bugzilla.redhat.com/show_bug.cgi?id=1871765 Bug ID: 1871765 Summary: Review Request: thoth-toolbox - This is a toolbox container image for Thoth tools Product: Fedora Version: rawhide Hardware: All OS: Linux Status: NEW Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: cgo...@redhat.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Target Milestone: --- Classification: Fedora Spec URL: n/a, its just a container image SRPM URL: n/a, its just a container image Description: This is a toolbox container image aggregating some of Thoth tools, including micropipenv and thamos Fedora Account System Username: goern -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1824156] Review Request: exfatprogs - Userspace utilities for exFAT filesystems
https://bugzilla.redhat.com/show_bug.cgi?id=1824156 --- Comment #33 from Simone Caronni --- Just saw that Tom is no longer working for Red Hat.. does this change anything regarding the legal approval? He has been removed from CC in this ticket. I would set a needinfo to some address but I wouldn't know who to... -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1867617] Review Request: perl-MaxMind-DB-Reader - Read MaxMind database files and look up IP addresses
https://bugzilla.redhat.com/show_bug.cgi?id=1867617 Bug 1867617 depends on bug 1867162, which changed state. Bug 1867162 Summary: Review Request: perl-Math-Int128 - Manipulate 128 bits integers in Perl https://bugzilla.redhat.com/show_bug.cgi?id=1867162 What|Removed |Added Status|MODIFIED|CLOSED Resolution|--- |RAWHIDE -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1867967] Review Request: perl-MaxMind-DB-Reader-XS - Fast XS implementation of MaxMind DB reader
https://bugzilla.redhat.com/show_bug.cgi?id=1867967 Bug 1867967 depends on bug 1867162, which changed state. Bug 1867162 Summary: Review Request: perl-Math-Int128 - Manipulate 128 bits integers in Perl https://bugzilla.redhat.com/show_bug.cgi?id=1867162 What|Removed |Added Status|MODIFIED|CLOSED Resolution|--- |RAWHIDE -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1867611] Review Request: perl-Net-Works - API for IP addresses and networks
https://bugzilla.redhat.com/show_bug.cgi?id=1867611 Bug 1867611 depends on bug 1867162, which changed state. Bug 1867162 Summary: Review Request: perl-Math-Int128 - Manipulate 128 bits integers in Perl https://bugzilla.redhat.com/show_bug.cgi?id=1867162 What|Removed |Added Status|MODIFIED|CLOSED Resolution|--- |RAWHIDE -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1867162] Review Request: perl-Math-Int128 - Manipulate 128 bits integers in Perl
https://bugzilla.redhat.com/show_bug.cgi?id=1867162 Petr Pisar changed: What|Removed |Added Status|MODIFIED|CLOSED Resolution|--- |RAWHIDE Last Closed||2020-08-24 07:26:26 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1867611] Review Request: perl-Net-Works - API for IP addresses and networks
https://bugzilla.redhat.com/show_bug.cgi?id=1867611 Petr Pisar changed: What|Removed |Added Status|MODIFIED|CLOSED Resolution|--- |RAWHIDE Last Closed||2020-08-24 07:26:07 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1867617] Review Request: perl-MaxMind-DB-Reader - Read MaxMind database files and look up IP addresses
https://bugzilla.redhat.com/show_bug.cgi?id=1867617 Bug 1867617 depends on bug 1867611, which changed state. Bug 1867611 Summary: Review Request: perl-Net-Works - API for IP addresses and networks https://bugzilla.redhat.com/show_bug.cgi?id=1867611 What|Removed |Added Status|MODIFIED|CLOSED Resolution|--- |RAWHIDE -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1857584] Review Request: python-serpent - Serialization based on ast.literal_eval
https://bugzilla.redhat.com/show_bug.cgi?id=1857584 Fedora Update System changed: What|Removed |Added Status|POST|MODIFIED --- Comment #3 from Fedora Update System --- FEDORA-2020-2387058a2d has been submitted as an update to Fedora 32. https://bodhi.fedoraproject.org/updates/FEDORA-2020-2387058a2d -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1867967] Review Request: perl-MaxMind-DB-Reader-XS - Fast XS implementation of MaxMind DB reader
https://bugzilla.redhat.com/show_bug.cgi?id=1867967 Petr Pisar changed: What|Removed |Added Status|ASSIGNED|CLOSED Fixed In Version||perl-MaxMind-DB-Reader-XS-1 ||.08-1.fc34 Resolution|--- |RAWHIDE Last Closed||2020-08-24 07:09:43 --- Comment #4 from Petr Pisar --- Thank you for the review and the repository. The /usr/lib64/perl5/vendor_perl/auto/MaxMind/DB/Reader/XS/XS.so permissions should be corrected with %{_fixperms} call. Moreover it's a fresh file generated with a compiler. It should not inherit any bad mode. Also I cannot reproduce it. I suspect you build the package with Mock tool that recently suffered from a glibc bug meddling with mode checks everywhere. This transitional bug should be fixed now. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1861839] Review Request: php-openpsa-universalfeedcreator - RSS and Atom feed generator
https://bugzilla.redhat.com/show_bug.cgi?id=1861839 Remi Collet changed: What|Removed |Added Status|ASSIGNED|POST Flags|fedora-review? |fedora-review+ --- Comment #7 from Remi Collet --- [x]: Package complies to the Packaging Guidelines See minor TIPS above, but no blocker ** APPROVED ** -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1861839] Review Request: php-openpsa-universalfeedcreator - RSS and Atom feed generator
https://bugzilla.redhat.com/show_bug.cgi?id=1861839 --- Comment #6 from Remi Collet --- Created attachment 1712302 --> https://bugzilla.redhat.com/attachment.cgi?id=1712302=edit review.txt -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1861839] Review Request: php-openpsa-universalfeedcreator - RSS and Atom feed generator
https://bugzilla.redhat.com/show_bug.cgi?id=1861839 --- Comment #5 from Remi Collet --- > # Upstream repo does not contain a LICENSE file It have been recently added https://github.com/flack/UniversalFeedCreator/blob/master/LICENSE Find to add it later, on next version upgrade, ad LGPL doesn't require it to be present. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1861839] Review Request: php-openpsa-universalfeedcreator - RSS and Atom feed generator
https://bugzilla.redhat.com/show_bug.cgi?id=1861839 --- Comment #4 from Remi Collet --- > sed -e 's|/lib/|/|g' -i autoload.php TIPS: generate the autoolader directly in lib (--output=lib/autoload.php) or use --basedir option to avoid having this sed -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1081434] Review Request: ip2location - IP2location library
https://bugzilla.redhat.com/show_bug.cgi?id=1081434 --- Comment #44 from Peter Bieringer --- sorry: forgot to mention option c) c) IP2Location -> I will vote for c) instead of b) as this is the current definition also in spec file -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1871717] New: Review Request: php-league-mime-type-detection - Mime-type detection for Flysystem
https://bugzilla.redhat.com/show_bug.cgi?id=1871717 Bug ID: 1871717 Summary: Review Request: php-league-mime-type-detection - Mime-type detection for Flysystem Product: Fedora Version: rawhide Hardware: All OS: Linux Status: NEW Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: fed...@famillecollet.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Target Milestone: --- Classification: Fedora Spec URL: https://git.remirepo.net/cgit/rpms/php/league/php-league-mime-type-detection.git/plain/php-league-mime-type-detection.spec?id=29d48958a8bbb33fe1fa334abadbcdb57346ad44 SRPM URL: https://rpms.remirepo.net/SRPMS/php-league-mime-type-detection-1.4.0-1.remi.src.rpm Description: This package supplies a generic mime-type detection interface with a finfo based implementation. Autoloader: /usr/share/php/League/MimeTypeDetection/autoload.php Fedora Account System Username: remi -- New dependency of league/flysystem 1.1 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1081434] Review Request: ip2location - IP2location library
https://bugzilla.redhat.com/show_bug.cgi?id=1081434 --- Comment #43 from Peter Bieringer --- If no maintainer will be found, I can act as (ipv6calc is supporting that library anyhow) btw: RPM build from current source works fine (beside some minor compiler warnings, which I have already fixed in my fork): wget https://github.com/chrislim2888/IP2Location-C-Library/archive/8.0.9.tar.gz rpmbuild -ts 8.0.9.tar.gz rpmbuild --rebuild IP2Location-8.0.9-5.el8.src.rpm > in source included spec file is at least working => have I to apply for maintainer or can one trigger the workflow? Beside that, the RPM name should be proper defined in advance: a) suggestion from 2014: ip2location-c b) current github project: IP2Location-C-Library I would vote for b) in alignment to GeoIP-* -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1867999] Review Request: perl-GeoIP2 - Perl API for MaxMind's GeoIP2 web services and databases
https://bugzilla.redhat.com/show_bug.cgi?id=1867999 Bug 1867999 depends on bug 1867617, which changed state. Bug 1867617 Summary: Review Request: perl-MaxMind-DB-Reader - Read MaxMind database files and look up IP addresses https://bugzilla.redhat.com/show_bug.cgi?id=1867617 What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |RAWHIDE -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1867967] Review Request: perl-MaxMind-DB-Reader-XS - Fast XS implementation of MaxMind DB reader
https://bugzilla.redhat.com/show_bug.cgi?id=1867967 Bug 1867967 depends on bug 1867617, which changed state. Bug 1867617 Summary: Review Request: perl-MaxMind-DB-Reader - Read MaxMind database files and look up IP addresses https://bugzilla.redhat.com/show_bug.cgi?id=1867617 What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |RAWHIDE -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1867617] Review Request: perl-MaxMind-DB-Reader - Read MaxMind database files and look up IP addresses
https://bugzilla.redhat.com/show_bug.cgi?id=1867617 Petr Pisar changed: What|Removed |Added Status|ASSIGNED|CLOSED Fixed In Version||perl-MaxMind-DB-Reader-1.00 ||0014-1.fc34 Resolution|--- |RAWHIDE Last Closed||2020-08-24 06:11:24 --- Comment #4 from Petr Pisar --- Thank you for the review and the repository. I realized that excluding 32-bit architectures is not needed for this package and reworked the conditions to be buildable everywhere. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org
[Bug 1871345] Review Request: jakarta-saaj - SOAP with Attachments API for Java
https://bugzilla.redhat.com/show_bug.cgi?id=1871345 --- Comment #4 from Igor Raits --- (fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/jakarta-saaj -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org 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@lists.fedoraproject.org