[Bug 2173523] Review Request: qt6-qtspeech - Speech component
https://bugzilla.redhat.com/show_bug.cgi?id=2173523 --- Comment #11 from Jakub Kadlčík --- Copr build: https://copr.fedorainfracloud.org/coprs/build/5575930 (succeeded) Review template: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2173523-qt6-qtspeech/fedora-rawhide-x86_64/05575930-qt6-qtspeech/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. -- 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. https://bugzilla.redhat.com/show_bug.cgi?id=2173523 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173523] Review Request: qt6-qtspeech - Speech component
https://bugzilla.redhat.com/show_bug.cgi?id=2173523 --- Comment #10 from Jan Grulich --- (In reply to Benson Muite from comment #8) > Issues from Fedora-review: > Note: Directories without known owners: /usr/lib64/cmake/Qt6Qml, > /usr/lib64/cmake/Qt6Qml/QmlPlugins > This should be owned by qt6-qtdeclarative. I will have to fix that. > Use SPDX identifiers, need: > GPL-3.0-only WITH Qt-GPL-exception-1.0 > > Examples are under BSD-3-Clause > Some of the code can be under GPL-2.0-only Fixed. > Documentation is under GFDL-1.3-no-invariants-only Documentation is not built and shipped. > Is it possible to add a license breakdown in the spec file? Added. Rpmlint still reports a wrong license for the Qt one, but it is listed in Fedora docs. Spec URL: https://jgrulich.fedorapeople.org/qt6-qtspeech/qt6-qtspeech.spec SRPM URL: https://jgrulich.fedorapeople.org/qt6-qtspeech/qt6-qtspeech-6.4.2-1.fc37.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. https://bugzilla.redhat.com/show_bug.cgi?id=2173523 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 587978] Review Request: whatweb - Web scanner to identify what websites are running
https://bugzilla.redhat.com/show_bug.cgi?id=587978 Fabio Valentini changed: What|Removed |Added CC||decatho...@gmail.com --- Comment #40 from Fabio Valentini --- PS: The URL fragment hack to name GitHub tarball downloads has not been necessary for a few years: https://docs.fedoraproject.org/en-US/packaging-guidelines/SourceURL/#_git_tags -- You are receiving this mail because: You are always notified about changes to this product and component https://bugzilla.redhat.com/show_bug.cgi?id=587978 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173123] Review Request: python-postorius - Web UI for GNU Mailman
https://bugzilla.redhat.com/show_bug.cgi?id=2173123 --- Comment #6 from Fedora Update System --- FEDORA-2023-334747e1cb has been pushed to the Fedora 37 testing repository. Soon you'll be able to install the update with the following command: `sudo dnf upgrade --enablerepo=updates-testing --refresh --advisory=FEDORA-2023-334747e1cb` You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2023-334747e1cb 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 always notified about changes to this product and component You are on the CC list for the bug. https://bugzilla.redhat.com/show_bug.cgi?id=2173123 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173501] Review Request: pe-bear - Portable Executable analyzing tool with a friendly GUI
https://bugzilla.redhat.com/show_bug.cgi?id=2173501 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #6 from Fedora Update System --- FEDORA-2023-157c1e23c3 has been pushed to the Fedora 37 testing repository. Soon you'll be able to install the update with the following command: `sudo dnf install --enablerepo=updates-testing --refresh --advisory=FEDORA-2023-157c1e23c3 \*` You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2023-157c1e23c3 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 always notified about changes to this product and component You are on the CC list for the bug. https://bugzilla.redhat.com/show_bug.cgi?id=2173501 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173810] Review Request: python-cvdupdate - ClamAV Private Database Mirror Updater Tool
https://bugzilla.redhat.com/show_bug.cgi?id=2173810 --- Comment #4 from Jakub Kadlčík --- Copr build: https://copr.fedorainfracloud.org/coprs/build/5575638 (succeeded) Review template: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2173810-python-cvdupdate/fedora-rawhide-x86_64/05575638-python-cvdupdate/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. -- 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 https://bugzilla.redhat.com/show_bug.cgi?id=2173810 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173810] Review Request: python-cvdupdate - ClamAV Private Database Mirror Updater Tool
https://bugzilla.redhat.com/show_bug.cgi?id=2173810 --- Comment #3 from Jakub Kadlčík --- Created attachment 1946836 --> https://bugzilla.redhat.com/attachment.cgi?id=1946836&action=edit The .spec file difference from Copr build 5575629 to 5575638 -- 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 https://bugzilla.redhat.com/show_bug.cgi?id=2173810 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173810] Review Request: python-cvdupdate - ClamAV Private Database Mirror Updater Tool
https://bugzilla.redhat.com/show_bug.cgi?id=2173810 Orion Poplawski changed: What|Removed |Added Doc Type|--- |If docs needed, set a value --- Comment #2 from Orion Poplawski --- [fedora-review-service-build] -- 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. https://bugzilla.redhat.com/show_bug.cgi?id=2173810 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173810] Review Request: python-cvdupdate - ClamAV Private Database Mirror Updater Tool
https://bugzilla.redhat.com/show_bug.cgi?id=2173810 Jakub Kadlčík changed: What|Removed |Added URL||https://github.com/Cisco-Ta ||los/cvdupdate --- Comment #1 from Jakub Kadlčík --- Copr build: https://copr.fedorainfracloud.org/coprs/build/5575629 (succeeded) Review template: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2173810-python-cvdupdate/fedora-rawhide-x86_64/05575629-python-cvdupdate/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. -- 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. https://bugzilla.redhat.com/show_bug.cgi?id=2173810 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173810] New: Review Request: python-cvdupdate - ClamAV Private Database Mirror Updater Tool
https://bugzilla.redhat.com/show_bug.cgi?id=2173810 Bug ID: 2173810 Summary: Review Request: python-cvdupdate - ClamAV Private Database Mirror Updater Tool Product: Fedora Version: rawhide Hardware: All OS: Linux Status: NEW Component: Package Review Severity: medium Assignee: nob...@fedoraproject.org Reporter: or...@nwra.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Target Milestone: --- Classification: Fedora Spec URL: https://orion.fedorapeople.org/python-cvdupdate.spec SRPM URL: https://orion.fedorapeople.org/python-cvdupdate-1.1.1-1.fc39.src.rpm Description: A tool to download and update clamav databases and database patch files for the purposes of hosting your own database mirror. Fedora Account System Username: orion -- 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 https://bugzilla.redhat.com/show_bug.cgi?id=2173810 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2165381] Review Request: ancient - Modern decompressor for old data compression formats
https://bugzilla.redhat.com/show_bug.cgi?id=2165381 --- Comment #5 from Jakub Kadlčík --- Created attachment 1946834 --> https://bugzilla.redhat.com/attachment.cgi?id=1946834&action=edit The .spec file difference from Copr build 5355028 to 5575504 -- 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 https://bugzilla.redhat.com/show_bug.cgi?id=2165381 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2165381] Review Request: ancient - Modern decompressor for old data compression formats
https://bugzilla.redhat.com/show_bug.cgi?id=2165381 Jakub Kadlčík changed: What|Removed |Added URL||https://github.com/temisu/a ||ncient --- Comment #6 from Jakub Kadlčík --- Copr build: https://copr.fedorainfracloud.org/coprs/build/5575504 (succeeded) Review template: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2165381-ancient/fedora-rawhide-x86_64/05575504-ancient/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. -- 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 https://bugzilla.redhat.com/show_bug.cgi?id=2165381 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2165381] Review Request: ancient - Modern decompressor for old data compression formats
https://bugzilla.redhat.com/show_bug.cgi?id=2165381
--- Comment #4 from Charles R. Anderson ---
Thank you for the review. I have updated the package:
Spec URL: https://cra.fedorapeople.org/ancient/ancient.spec
SRPM URL: https://cra.fedorapeople.org/ancient/ancient-2.0.0-2.fc37.src.rpm
(In reply to Benson Muite from comment #2)
> a) There is a test script, but upstream has not included any test data.
> Maybe this could be added to the repo?
> c) Is it possible to add a smoke test, for example
> ancient --help
I added a test that verifies ancient decompressing Source0's gzip compression.
> b) Assume none of the algorithms are under a patent?
I'm not a patent attorney, but I assume so--these are either ancient file
formats or formats that are already implemented in other parts of Fedora.
Nothing here is called out as explicitly prohibited on
https://docs.fedoraproject.org/en-US/packaging-guidelines/what-can-be-packaged/
or the linked documents therein. I'm not sure what else is expected of a
packager on this point, especially considering that doing a patent review could
actually increase legal exposure.
> d) devel package does not need a license file as it will be pulled in from
> libs package
Removed.
(In reply to Ben Beasley from comment #3)
> Please don’t glob over a shared directory; see
> https://docs.fedoraproject.org/en-US/packaging-guidelines/#_explicit_lists.
> %{_bindir}/ancient
Done.
> > a) There is a test script, but upstream has not included any test data.
> > Maybe this could be added to the repo?
>
> Considering that this tool is intended for files from the 1980’s-1990’s, I
> strongly suspect that the test data is not included because it is all
> non-free.
Agreed. Hopefully the gzip test I implemented is sufficient.
--
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.
https://bugzilla.redhat.com/show_bug.cgi?id=2165381
___
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
Do not reply to spam, report it:
https://pagure.io/fedora-infrastructure/new_issue
[Bug 2152402] Review Request: librist - Library for Reliable Internet Stream Transport (RIST) protocol
https://bugzilla.redhat.com/show_bug.cgi?id=2152402 --- Comment #23 from Fedora Update System --- FEDORA-2023-17b8a46bfe has been pushed to the Fedora 36 stable repository. If problem still persists, please make note of it in this bug report. -- 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 https://bugzilla.redhat.com/show_bug.cgi?id=2152402 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173123] Review Request: python-postorius - Web UI for GNU Mailman
https://bugzilla.redhat.com/show_bug.cgi?id=2173123 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #5 from Fedora Update System --- FEDORA-2023-64d3cc740b has been pushed to the Fedora 38 testing repository. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2023-64d3cc740b 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 https://bugzilla.redhat.com/show_bug.cgi?id=2173123 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173751] Review Request: mod_auth_cas - unretire package
https://bugzilla.redhat.com/show_bug.cgi?id=2173751
--- Comment #8 from Carl George 🤠 ---
Thanks for submitting this re-review. This is a fairly old spec file, so it
will need a bit of cleaning up to bring it up to modern Fedora packaging
standards. Don't get discouraged by the following wall of text, it's not a
criticism of you, just things we need to work on in the package.
The conditional %_httpd_* macro definitions at the start of the spec file are
no longer necessary and should be removed. They're defined by httpd-devel in
all current releases of RHEL and Fedora.
Since all current RHEL and Fedora release includes httpd 2.4, the sections of
the spec that reference httpd 2.2 should probably be removed.
Optional suggestion, consider refreshing the summary and description with
sections of the upstream README. The first sentence of the first section
(without the period) would work well as the summary, and the first sentence of
the introduction section would work well as the description.
There are a few deprectated tags and commands that should be removed.
-Group: System Environment/Daemons
-BuildRoot: %(mktemp -ud %{_tmppath}/%{name}-%{version}-%{release}-XX)
-rm -rf %{buildroot}
-%clean
-rm -rf %{buildroot}
https://docs.fedoraproject.org/en-US/packaging-guidelines/#_tags_and_sections
The license field needs to use the new SPDX identifiers.
-License:APLv2
+License:Apache-2.0
https://docs.fedoraproject.org/en-US/legal/license-field/#_basic_policy
Since this is a C program, you'll need to have a buildrequires for gcc.
+BuildRequires: gcc
https://docs.fedoraproject.org/en-US/packaging-guidelines/C_and_C++/#_buildrequires_and_requires
This has a buildrequires on a deprecated package, pcre-devel. File an issue
with upstream to see if they're interested in porting this program over to
pcre2, and leave a comment above the pcre-devel line with a link to that issue.
Technically during reviews depending on deprecated packages isn't allowed, but
I think we can waive that since this is a re-review to unretire.
https://docs.fedoraproject.org/en-US/packaging-guidelines/deprecating-packages/
I don't see this in the guidelines, but other httpd modules packaged in Fedora
tend to require httpd-mmn, not httpd directly. This will also allow users to
install a smaller footprint of httpd with the httpd-core package on Fedora and
RHEL9.
-Requires: httpd
+Requires: httpd-mmn = %{_httpd_mmn}
Instead of hardcoding the version in %prep it would be better to use the
version macro, so that way when a new version is released it only has to be
updated on one line. And since the default value for -n is %{name}-%{version},
we can just leave it off entirely.
-%setup -q -n %{name}-1.2
+%setup -q
Switch to the modern %make_build and %make_install macros.
-make %{?_smp_mflags}
+%make_build
-make install DESTDIR=%{buildroot} LIBEXECDIR=%{_httpd_moddir}
+%make_install
https://docs.fedoraproject.org/en-US/packaging-guidelines/#_parallel_make
https://docs.fedoraproject.org/en-US/packaging-guidelines/#_why_the_makeinstall_macro_should_not_be_used
Instead of directly using the /var path, consider switching to
%{_localstatedir}.
-mkdir -p %{buildroot}/var/cache/httpd/%{name}
+mkdir -p %{buildroot}%{_localstatedir}/cache/httpd/%{name}
-%dir /var/cache/httpd/%{name}
+%dir %{_localstatedir}/cache/httpd/%{name}
Setting permissions to root:root is not necessary since that is the default.
-%defattr(-,root,root,-)
For the one directory that needs apache:apache permissions, consider doing it
in one line.
-%defattr(-,apache,apache,-)
-%dir %{_localstatedir}/cache/httpd/%{name}
+%dir %attr(-,apache,apache) %{_localstatedir}/cache/httpd/%{name}
https://docs.fedoraproject.org/en-US/packaging-guidelines/#_file_permissions
It's best practice to minimize usage of globs in %files. In this case they are
matching one file each, s
[Bug 2173751] Review Request: mod_auth_cas - unretire package
https://bugzilla.redhat.com/show_bug.cgi?id=2173751 --- Comment #7 from Jakub Kadlčík --- Copr build: https://copr.fedorainfracloud.org/coprs/build/5575004 (succeeded) Review template: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2173751-mod_auth_cas/fedora-rawhide-x86_64/05575004-mod_auth_cas/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. -- 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. https://bugzilla.redhat.com/show_bug.cgi?id=2173751 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173751] Review Request: mod_auth_cas - unretire package
https://bugzilla.redhat.com/show_bug.cgi?id=2173751 --- Comment #6 from Jakub Kadlčík --- Created attachment 1946802 --> https://bugzilla.redhat.com/attachment.cgi?id=1946802&action=edit The .spec file difference from Copr build 5574725 to 5575004 -- 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 https://bugzilla.redhat.com/show_bug.cgi?id=2173751 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173751] Review Request: mod_auth_cas - unretire package
https://bugzilla.redhat.com/show_bug.cgi?id=2173751 --- Comment #5 from Scott Williams --- Just trying to trigger the updated review. [fedora-review-service-build] Spec URL: https://download.copr.fedorainfracloud.org/results/vwbusguy/mod_auth_cas/fedora-rawhide-x86_64/05574866-mod_auth_cas/mod_auth_cas.spec SRPM URL: https://download.copr.fedorainfracloud.org/results/vwbusguy/mod_auth_cas/fedora-rawhide-x86_64/05574866-mod_auth_cas/mod_auth_cas-1.2-1.fc39.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. https://bugzilla.redhat.com/show_bug.cgi?id=2173751 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173751] Review Request: mod_auth_cas - unretire package
https://bugzilla.redhat.com/show_bug.cgi?id=2173751 --- Comment #4 from Scott Williams --- Spec URL: https://download.copr.fedorainfracloud.org/results/vwbusguy/mod_auth_cas/fedora-rawhide-x86_64/05574866-mod_auth_cas/mod_auth_cas.spec SRPM URL: https://download.copr.fedorainfracloud.org/results/vwbusguy/mod_auth_cas/fedora-rawhide-x86_64/05574866-mod_auth_cas/mod_auth_cas-1.2-1.fc39.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. https://bugzilla.redhat.com/show_bug.cgi?id=2173751 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173751] Review Request: mod_auth_cas - unretire package
https://bugzilla.redhat.com/show_bug.cgi?id=2173751 Scott Williams changed: What|Removed |Added Link ID||Red Hat Bugzilla 1934268 -- 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. https://bugzilla.redhat.com/show_bug.cgi?id=2173751 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173751] Review Request: mod_auth_cas - unretire package
https://bugzilla.redhat.com/show_bug.cgi?id=2173751 Scott Williams changed: What|Removed |Added CC||vwfoxg...@gmail.com --- Comment #3 from Scott Williams --- https://download.copr.fedorainfracloud.org/results/vwbusguy/mod_auth_cas/fedora-rawhide-x86_64/05574866-mod_auth_cas/mod_auth_cas-1.2-1.fc39.src.rpm https://download.copr.fedorainfracloud.org/results/vwbusguy/mod_auth_cas/fedora-rawhide-x86_64/05574866-mod_auth_cas/mod_auth_cas.spec -- 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. https://bugzilla.redhat.com/show_bug.cgi?id=2173751 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173751] Review Request: mod_auth_cas - unretire package
https://bugzilla.redhat.com/show_bug.cgi?id=2173751 --- Comment #2 from Timothy Hansen --- https://download.copr.fedorainfracloud.org/results/vwbusguy/mod_auth_cas/fedora-rawhide-x86_64/05574866-mod_auth_cas/ -- 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. https://bugzilla.redhat.com/show_bug.cgi?id=2173751 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173758] Review Request: apriltag - Visual fiducial system popular for robotics research
https://bugzilla.redhat.com/show_bug.cgi?id=2173758 Jakub Kadlčík changed: What|Removed |Added URL||https://april.eecs.umich.ed ||u/software/apriltag --- Comment #1 from Jakub Kadlčík --- Copr build: https://copr.fedorainfracloud.org/coprs/build/5574803 (succeeded) Review template: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2173758-apriltag/fedora-rawhide-x86_64/05574803-apriltag/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. -- 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 https://bugzilla.redhat.com/show_bug.cgi?id=2173758 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 587978] Review Request: whatweb - Web scanner to identify what websites are running
https://bugzilla.redhat.com/show_bug.cgi?id=587978
Otto Liljalaakso changed:
What|Removed |Added
Flags|needinfo?(otto.liljalaakso@ |fedora-review?
|iki.fi) |needinfo?(re...@seznam.cz)
--- Comment #39 from Otto Liljalaakso ---
Hello Michal,
yes let us finish this one.
> License:GPLv2+
Meanwhile, Fedora's system of noting the license has changed.
Now, SPDX identifiers must be used, so this need to be written as
GPL-2.0-or-later.
Reference: https://docs.fedoraproject.org/en-US/legal/license-field/
> Requires: %{_bindir}/ruby
I asked you to switch to this form, but it turns out I was wrong.
In the context of Requires, this actually must be written as /usr/bin/ruby.
Sorry for the noise.
Reference:
https://docs.fedoraproject.org/en-US/packaging-guidelines/#_file_and_directory_dependencies,
last paragraph
Other than these two small things, this review is ready.
Please correct them and I will flag the review as complete.
--
You are receiving this mail because:
You are always notified about changes to this product and component
https://bugzilla.redhat.com/show_bug.cgi?id=587978
___
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
Do not reply to spam, report it:
https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173758] New: Review Request: apriltag - Visual fiducial system popular for robotics research
https://bugzilla.redhat.com/show_bug.cgi?id=2173758 Bug ID: 2173758 Summary: Review Request: apriltag - Visual fiducial system popular for robotics research Product: Fedora Version: rawhide Hardware: All OS: Linux Status: NEW Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: log...@cottsay.net QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Target Milestone: --- Classification: Fedora Spec URL: https://cottsay.fedorapeople.org/apriltag/apriltag.spec SRPM URL: https://cottsay.fedorapeople.org/apriltag/apriltag-3.3.0-1.fc39.src.rpm Description: AprilTag is a visual fiducial system popular in robotics research. This package contains the most recent version of AprilTag, AprilTag 3, which includes a faster (>2x) detector, improved detection rate on small tags, flexible tag layouts, and pose estimation. AprilTag consists of a small C library with minimal dependencies. Fedora Account System Username: cottsay Target branches: rawhide f37 f36 epel9 epel8 epel7 Koji scratch build: https://koji.fedoraproject.org/koji/taskinfo?taskID=98082422 Thanks! -- 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. https://bugzilla.redhat.com/show_bug.cgi?id=2173758 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173751] Review Request: mod_auth_cas - unretire package
https://bugzilla.redhat.com/show_bug.cgi?id=2173751 Jakub Kadlčík changed: What|Removed |Added URL||http://www.ja-sig.org/wiki/ ||display/CASC/mod_auth_cas --- Comment #1 from Jakub Kadlčík --- Copr build: https://copr.fedorainfracloud.org/coprs/build/5574725 (succeeded) Review template: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2173751-mod_auth_cas/fedora-rawhide-x86_64/05574725-mod_auth_cas/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. -- 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. https://bugzilla.redhat.com/show_bug.cgi?id=2173751 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173751] Review Request: mod_auth_cas - unretire package
https://bugzilla.redhat.com/show_bug.cgi?id=2173751 Carl George 🤠 changed: What|Removed |Added Doc Type|--- |If docs needed, set a value CC||c...@redhat.com Assignee|nob...@fedoraproject.org|c...@redhat.com Flags||fedora-review? Status|NEW |ASSIGNED -- 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. https://bugzilla.redhat.com/show_bug.cgi?id=2173751 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173751] New: Review Request: mod_auth_cas - unretire package
https://bugzilla.redhat.com/show_bug.cgi?id=2173751 Bug ID: 2173751 Summary: Review Request: mod_auth_cas - unretire package Product: Fedora Version: rawhide Hardware: All OS: Linux Status: NEW Component: Package Review Severity: medium Assignee: nob...@fedoraproject.org Reporter: timhanse...@gmail.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Target Milestone: --- Classification: Fedora Spec URL: https://download.copr.fedorainfracloud.org/results/vwbusguy/mod_auth_cas/fedora-rawhide-x86_64/05574704-mod_auth_cas/mod_auth_cas.spec SRPM URL: https://download.copr.fedorainfracloud.org/results/vwbusguy/mod_auth_cas/fedora-rawhide-x86_64/05574704-mod_auth_cas/mod_auth_cas-1.2-0.fc39.src.rpm Description: mod_auth_cas is an Apache 2.2 and 2.4 compliant module that supports the CASv1 and CASv2 protocols Note: re-review to unretire package. Fedora Account System Username: timhansen46 vwbusguy -- 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. https://bugzilla.redhat.com/show_bug.cgi?id=2173751 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173743] Review Request: python-hyperkitty - A web interface to access GNU Mailman v3 archives
https://bugzilla.redhat.com/show_bug.cgi?id=2173743 --- Comment #2 from Michel Alexandre Salim --- Thanks! Unretirement request https://pagure.io/releng/issue/11314 -- 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. https://bugzilla.redhat.com/show_bug.cgi?id=2173743 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173743] Review Request: python-hyperkitty - A web interface to access GNU Mailman v3 archives
https://bugzilla.redhat.com/show_bug.cgi?id=2173743 Neal Gompa changed: What|Removed |Added Assignee|nob...@fedoraproject.org|ngomp...@gmail.com CC||ngomp...@gmail.com Doc Type|--- |If docs needed, set a value Flags||fedora-review+ Status|NEW |POST --- Comment #1 from Neal Gompa --- Package review notes: * Packaging follows contemporary Python packaging guidelines * Package licensing is correct and license files are installed * Package builds and installs successfully 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 https://bugzilla.redhat.com/show_bug.cgi?id=2173743 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173743] New: Review Request: python-hyperkitty - A web interface to access GNU Mailman v3 archives
https://bugzilla.redhat.com/show_bug.cgi?id=2173743 Bug ID: 2173743 Summary: Review Request: python-hyperkitty - A web interface to access GNU Mailman v3 archives Product: Fedora Version: rawhide Status: NEW Component: Package Review Assignee: nob...@fedoraproject.org Reporter: mic...@michel-slm.name QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Target Milestone: --- Classification: Fedora Spec URL: https://salimma.fedorapeople.org/python-hyperkitty.spec SRPM URL: https://salimma.fedorapeople.org/python-hyperkitty-1.3.7-1.fc39.src.rpm Description: HyperKitty is an open source Django application under development. It aims to provide a web interface to access GNU Mailman v3 archives. Fedora Account System Username: salimma -- 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. https://bugzilla.redhat.com/show_bug.cgi?id=2173743 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173178] Review Request: kde-inotify-survey - inotify monitor for kde
https://bugzilla.redhat.com/show_bug.cgi?id=2173178 --- Comment #7 from Steve Cossette --- Is there anything I can do to get this pushed through, or do we need to wait for upstream? Just checking to see if there's anything I can do, 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 https://bugzilla.redhat.com/show_bug.cgi?id=2173178 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2150094] Review Request: nodejs18 - JavaScript runtime
https://bugzilla.redhat.com/show_bug.cgi?id=2150094 Stephen Gallagher changed: What|Removed |Added Resolution|--- |RAWHIDE Status|NEW |CLOSED Last Closed||2023-02-27 18:38:04 -- 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. https://bugzilla.redhat.com/show_bug.cgi?id=2150094 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173716] Review Request: rust-test-case-macros - Procedural macro attribute for generating parametrized test cases
https://bugzilla.redhat.com/show_bug.cgi?id=2173716 Jakub Kadlčík changed: What|Removed |Added URL||https://crates.io/crates/te ||st-case-macros --- Comment #2 from Jakub Kadlčík --- Copr build: https://copr.fedorainfracloud.org/coprs/build/5574395 (succeeded) Review template: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2173716-rust-test-case-macros/fedora-rawhide-x86_64/05574395-rust-test-case-macros/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. -- 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. https://bugzilla.redhat.com/show_bug.cgi?id=2173716 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173710] Review Request: parlaylib - A toolkit for programming parallel shared memory algorithms
https://bugzilla.redhat.com/show_bug.cgi?id=2173710 Jakub Kadlčík changed: What|Removed |Added URL||https://github.com/cmuparla ||y/parlaylib --- Comment #1 from Jakub Kadlčík --- Copr build: https://copr.fedorainfracloud.org/coprs/build/5573921 (succeeded) Review template: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2173710-parlaylib/fedora-rawhide-x86_64/05573921-parlaylib/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. -- 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 https://bugzilla.redhat.com/show_bug.cgi?id=2173710 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173716] New: Review Request: rust-test-case-macros - Procedural macro attribute for generating parametrized test cases
https://bugzilla.redhat.com/show_bug.cgi?id=2173716 Bug ID: 2173716 Summary: Review Request: rust-test-case-macros - Procedural macro attribute for generating parametrized test cases Product: Fedora Version: rawhide Hardware: All OS: Linux Status: NEW Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: decatho...@gmail.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Target Milestone: --- Classification: Fedora Spec URL: https://decathorpe.fedorapeople.org/rust-test-case-macros.spec SRPM URL: https://decathorpe.fedorapeople.org/rust-test-case-macros-2.2.2-1.fc37.src.rpm Description: Provides #[test_case(...)] procedural macro attribute for generating parametrized test cases easily. Fedora Account System Username: decathorpe -- 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 https://bugzilla.redhat.com/show_bug.cgi?id=2173716 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173710] New: Review Request: parlaylib - A toolkit for programming parallel shared memory algorithms
https://bugzilla.redhat.com/show_bug.cgi?id=2173710 Bug ID: 2173710 Summary: Review Request: parlaylib - A toolkit for programming parallel shared memory algorithms Product: Fedora Version: rawhide Status: NEW Component: Package Review Assignee: nob...@fedoraproject.org Reporter: benson_mu...@emailplus.org QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Target Milestone: --- Classification: Fedora Spec URL: https://download.copr.fedorainfracloud.org/results/fed500/parlaylib/fedora-rawhide-x86_64/05573280-parlaylib/parlaylib.spec SRPM URL: https://download.copr.fedorainfracloud.org/results/fed500/parlaylib/fedora-rawhide-x86_64/05573280-parlaylib/parlaylib-2.1.5%5E20230215git137076b-1.fc39.src.rpm Description: ParlayLib is a header only C++ library for developing efficient parallel algorithms and software on shared-memory multicore machines. It provides additional tools and primitives that go beyond what is available in the C++ standard library, and simplifies the task of programming provably efficient and scalable parallel algorithms. It consists of a sequence data type (analogous to std::vector), many parallel routines and algorithms, a work-stealing scheduler to support nested parallelism, and a scalable memory allocator. Fedora Account System Username: fed500 -- 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. https://bugzilla.redhat.com/show_bug.cgi?id=2173710 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173523] Review Request: qt6-qtspeech - Speech component
https://bugzilla.redhat.com/show_bug.cgi?id=2173523 --- Comment #9 from Benson Muite --- Perhaps use ( GPL-3.0-only WITH Qt-GPL-exception-1.0 ) Though warning maybe a bug in Fedora-review -- 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. https://bugzilla.redhat.com/show_bug.cgi?id=2173523 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173123] Review Request: python-postorius - Web UI for GNU Mailman
https://bugzilla.redhat.com/show_bug.cgi?id=2173123 --- Comment #4 from Fedora Update System --- FEDORA-2023-334747e1cb has been submitted as an update to Fedora 37. https://bodhi.fedoraproject.org/updates/FEDORA-2023-334747e1cb -- 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. https://bugzilla.redhat.com/show_bug.cgi?id=2173123 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173523] Review Request: qt6-qtspeech - Speech component
https://bugzilla.redhat.com/show_bug.cgi?id=2173523 --- Comment #8 from Benson Muite --- Issues from Fedora-review: Note: Directories without known owners: /usr/lib64/cmake/Qt6Qml, /usr/lib64/cmake/Qt6Qml/QmlPlugins Use SPDX identifiers, need: GPL-3.0-only WITH Qt-GPL-exception-1.0 Examples are under BSD-3-Clause Some of the code can be under GPL-2.0-only Documentation is under GFDL-1.3-no-invariants-only Is it possible to add a license breakdown in the spec file? -- 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. https://bugzilla.redhat.com/show_bug.cgi?id=2173523 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173123] Review Request: python-postorius - Web UI for GNU Mailman
https://bugzilla.redhat.com/show_bug.cgi?id=2173123 Fedora Update System changed: What|Removed |Added Status|POST|MODIFIED --- Comment #3 from Fedora Update System --- FEDORA-2023-64d3cc740b has been submitted as an update to Fedora 38. https://bodhi.fedoraproject.org/updates/FEDORA-2023-64d3cc740b -- 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 https://bugzilla.redhat.com/show_bug.cgi?id=2173123 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173677] Review Request: regextester - Regex Tester for elementary OS
https://bugzilla.redhat.com/show_bug.cgi?id=2173677 Jakub Kadlčík changed: What|Removed |Added URL||https://github.com/artemanu ||frij/regextester --- Comment #1 from Jakub Kadlčík --- Copr build: https://copr.fedorainfracloud.org/coprs/build/5572964 (succeeded) Review template: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2173677-regextester/fedora-rawhide-x86_64/05572964-regextester/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. -- 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. https://bugzilla.redhat.com/show_bug.cgi?id=2173677 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173677] New: Review Request: regextester - Regex Tester for elementary OS
https://bugzilla.redhat.com/show_bug.cgi?id=2173677 Bug ID: 2173677 Summary: Review Request: regextester - Regex Tester for elementary OS Product: Fedora Version: rawhide Status: NEW Component: Package Review Assignee: nob...@fedoraproject.org Reporter: gw...@protonmail.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Target Milestone: --- Classification: Fedora Description: Regex Tester for elementary OS SRPM: https://fedorapeople.org/~limb/review/regextester/regextester-1.1.1-1.fc37.src.rpm SPEC: https://fedorapeople.org/~limb/review/regextester/regextester.spec -- 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 https://bugzilla.redhat.com/show_bug.cgi?id=2173677 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173622] Review Request: python-sphinxcontrib-jquery - Extension to include jQuery on newer Sphinx releases
https://bugzilla.redhat.com/show_bug.cgi?id=2173622 --- Comment #7 from Fedora Admin user for bugzilla script actions --- The Pagure repository was created at https://src.fedoraproject.org/rpms/python-sphinxcontrib-jquery -- 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 https://bugzilla.redhat.com/show_bug.cgi?id=2173622 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173622] Review Request: python-sphinxcontrib-jquery - Extension to include jQuery on newer Sphinx releases
https://bugzilla.redhat.com/show_bug.cgi?id=2173622
Tomáš Hrnčiar changed:
What|Removed |Added
Flags|fedora-review? |fedora-review+
Status|ASSIGNED|POST
--- Comment #6 from Tomáš Hrnčiar ---
The package is APPROVED.
Package Review
==
Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed
= 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.
[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", "BSD 0-Clause License", "MIT License".
10 files have unknown license. Detailed output of licensecheck in
/var/lib/copr-rpmbuild/results/python-sphinxcontrib-
jquery/licensecheck.txt
[x]: 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.
[-]: 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.
[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]: 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: No rpmlint messages.
[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 package is included in %license.
[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.
[-]: 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.
[-]: If the source package does not include license text(s) as a separate
file from upstream, the packager SHOULD query upstream to include it.
[x]: Final provides and requires are sane (see attachments).
[x]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text files sep
[Bug 2173622] Review Request: python-sphinxcontrib-jquery - Extension to include jQuery on newer Sphinx releases
https://bugzilla.redhat.com/show_bug.cgi?id=2173622 --- Comment #5 from Karolina Surma --- Used the upstream PR as the patch to Fedora. https://download.copr.fedorainfracloud.org/results/ksurma/sphinxcontrib-jquery/fedora-rawhide-x86_64/05572946-python-sphinxcontrib-jquery/python-sphinxcontrib-jquery.spec https://download.copr.fedorainfracloud.org/results/ksurma/sphinxcontrib-jquery/fedora-rawhide-x86_64/05572946-python-sphinxcontrib-jquery/python-sphinxcontrib-jquery-3.0.0-1.fc39.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. https://bugzilla.redhat.com/show_bug.cgi?id=2173622 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173665] Review Request: python-papermill - Parametrize and run Jupyter and nteract Notebooks
https://bugzilla.redhat.com/show_bug.cgi?id=2173665 Ankur Sinha (FranciscoD) changed: What|Removed |Added Blocks||1276941 (fedora-neuro), ||2145122 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 https://bugzilla.redhat.com/show_bug.cgi?id=2145122 [Bug 2145122] python-bluepyopt-1.13.178 is available -- 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. https://bugzilla.redhat.com/show_bug.cgi?id=2173665 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2165381] Review Request: ancient - Modern decompressor for old data compression formats
https://bugzilla.redhat.com/show_bug.cgi?id=2165381
Ben Beasley changed:
What|Removed |Added
CC||c...@musicinmybrain.net
--- Comment #3 from Ben Beasley ---
Please don’t glob over a shared directory; see
https://docs.fedoraproject.org/en-US/packaging-guidelines/#_explicit_lists.
You can change
%{_bindir}/*
to
%{_bindir}/ancient
in the %files list.
> a) There is a test script, but upstream has not included any test data. Maybe
> this could be added to the repo?
Considering that this tool is intended for files from the 1980’s-1990’s, I
strongly suspect that the test data is not included because it is all non-free.
--
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.
https://bugzilla.redhat.com/show_bug.cgi?id=2165381
___
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
Do not reply to spam, report it:
https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173665] Review Request: python-papermill - Parametrize and run Jupyter and nteract Notebooks
https://bugzilla.redhat.com/show_bug.cgi?id=2173665 Jakub Kadlčík changed: What|Removed |Added URL||https://pypi.org/pypi/paper ||mill --- Comment #2 from Jakub Kadlčík --- Copr build: https://copr.fedorainfracloud.org/coprs/build/5572933 (succeeded) Review template: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2173665-python-papermill/fedora-rawhide-x86_64/05572933-python-papermill/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. -- 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. https://bugzilla.redhat.com/show_bug.cgi?id=2173665 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173665] New: Review Request: python-papermill - Parametrize and run Jupyter and nteract Notebooks
https://bugzilla.redhat.com/show_bug.cgi?id=2173665 Bug ID: 2173665 Summary: Review Request: python-papermill - Parametrize and run Jupyter and nteract Notebooks Product: Fedora Version: rawhide Status: NEW Component: Package Review Assignee: nob...@fedoraproject.org Reporter: sanjay.an...@gmail.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Target Milestone: --- Classification: Fedora Spec URL: https://ankursinha.fedorapeople.org/python-papermill/python-papermill.spec SRPM URL: https://ankursinha.fedorapeople.org/python-papermill/python-papermill-2.4.0-2.fc39.src.rpm Description: papermill is a tool for parameterizing, executing, and analyzing Jupyter Notebooks. Papermill lets you: - parameterize notebooks - execute notebooks This opens up new opportunities for how notebooks can be used. For example: - Perhaps you have a financial report that you wish to run with different values on the first or last day of a month or at the beginning or end of the year, using parameters makes this task easier. - Do you want to run a notebook and depending on its results, choose a particular notebook to run next? You can now programmatically execute a workflow without having to copy and paste from notebook to notebook manually. Papermill takes an opinionated approach to notebook parameterization and execution based on our experiences using notebooks at scale in data pipelines. Fedora Account System Username: ankursinha -- 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 https://bugzilla.redhat.com/show_bug.cgi?id=2173665 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173665] Review Request: python-papermill - Parametrize and run Jupyter and nteract Notebooks
https://bugzilla.redhat.com/show_bug.cgi?id=2173665 --- Comment #1 from Ankur Sinha (FranciscoD) --- This package built on koji: https://koji.fedoraproject.org/koji/taskinfo?taskID=98070415 -- 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 https://bugzilla.redhat.com/show_bug.cgi?id=2173665 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173622] Review Request: python-sphinxcontrib-jquery - Extension to include jQuery on newer Sphinx releases
https://bugzilla.redhat.com/show_bug.cgi?id=2173622 Tomáš Hrnčiar changed: What|Removed |Added Assignee|nob...@fedoraproject.org|thrnc...@redhat.com CC||thrnc...@redhat.com Flags||fedora-review? Status|NEW |ASSIGNED -- 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 https://bugzilla.redhat.com/show_bug.cgi?id=2173622 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173622] Review Request: python-sphinxcontrib-jquery - Extension to include jQuery on newer Sphinx releases
https://bugzilla.redhat.com/show_bug.cgi?id=2173622 --- Comment #4 from Jakub Kadlčík --- Copr build: https://copr.fedorainfracloud.org/coprs/build/5572686 (succeeded) Review template: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2173622-python-sphinxcontrib-jquery/fedora-rawhide-x86_64/05572686-python-sphinxcontrib-jquery/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. -- 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. https://bugzilla.redhat.com/show_bug.cgi?id=2173622 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173622] Review Request: python-sphinxcontrib-jquery - Extension to include jQuery on newer Sphinx releases
https://bugzilla.redhat.com/show_bug.cgi?id=2173622 --- Comment #3 from Jakub Kadlčík --- Created attachment 1946734 --> https://bugzilla.redhat.com/attachment.cgi?id=1946734&action=edit The .spec file difference from Copr build 5572673 to 5572686 -- 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. https://bugzilla.redhat.com/show_bug.cgi?id=2173622 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173622] Review Request: python-sphinxcontrib-jquery - Extension to include jQuery on newer Sphinx releases
https://bugzilla.redhat.com/show_bug.cgi?id=2173622 --- Comment #2 from Karolina Surma --- Marked the missing license file. Spec: https://download.copr.fedorainfracloud.org/results/ksurma/sphinxcontrib-jquery/fedora-rawhide-x86_64/05572674-python-sphinxcontrib-jquery/python-sphinxcontrib-jquery.spec SRPM: https://download.copr.fedorainfracloud.org/results/ksurma/sphinxcontrib-jquery/fedora-rawhide-x86_64/05572674-python-sphinxcontrib-jquery/python-sphinxcontrib-jquery-3.0.0-1.fc39.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 https://bugzilla.redhat.com/show_bug.cgi?id=2173622 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173622] Review Request: python-sphinxcontrib-jquery - Extension to include jQuery on newer Sphinx releases
https://bugzilla.redhat.com/show_bug.cgi?id=2173622 Jakub Kadlčík changed: What|Removed |Added URL||https://github.com/sphinx-c ||ontrib/jquery/ --- Comment #1 from Jakub Kadlčík --- Copr build: https://copr.fedorainfracloud.org/coprs/build/5572673 (succeeded) Review template: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2173622-python-sphinxcontrib-jquery/fedora-rawhide-x86_64/05572673-python-sphinxcontrib-jquery/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. -- 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 https://bugzilla.redhat.com/show_bug.cgi?id=2173622 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173622] New: Review Request: python-sphinxcontrib-jquery - Extension to include jQuery on newer Sphinx releases
https://bugzilla.redhat.com/show_bug.cgi?id=2173622 Bug ID: 2173622 Summary: Review Request: python-sphinxcontrib-jquery - Extension to include jQuery on newer Sphinx releases Product: Fedora Version: rawhide Hardware: All OS: Linux Status: NEW Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: ksu...@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/ksurma/sphinxcontrib-jquery/fedora-rawhide-x86_64/05572500-python-sphinxcontrib-jquery/python-sphinxcontrib-jquery.spec SRPM URL: https://download.copr.fedorainfracloud.org/results/ksurma/sphinxcontrib-jquery/fedora-rawhide-x86_64/05572500-python-sphinxcontrib-jquery/python-sphinxcontrib-jquery-3.0.0-1.fc39.src.rpm Description: sphinxcontrib-jquery is a Sphinx extension that ensures that jQuery is always installed for use in Sphinx themes or extensions. It is currently required by python-sphinx_rtd_theme to work. Fedora Account System Username: ksurma -- 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. https://bugzilla.redhat.com/show_bug.cgi?id=2173622 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 1816301] Review Request: openfoam - computational fluid dynamics
https://bugzilla.redhat.com/show_bug.cgi?id=1816301 Jakub Kadlčík changed: What|Removed |Added URL||http://www.openfoam.com --- Comment #31 from Jakub Kadlčík --- Copr build: https://copr.fedorainfracloud.org/coprs/build/5572620 (failed) Build log: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-1816301-openfoam/fedora-rawhide-x86_64/05572620-openfoam2212/builder-live.log.gz Please make sure the package builds successfully at least for Fedora Rawhide. - If the build failed for unrelated reasons (e.g. temporary network unavailability), please ignore it. - If the build failed because of missing BuildRequires, please make sure they are listed in the "Depends On" field --- 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. -- 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 https://bugzilla.redhat.com/show_bug.cgi?id=1816301 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 1816301] Review Request: openfoam - computational fluid dynamics
https://bugzilla.redhat.com/show_bug.cgi?id=1816301 --- Comment #30 from mark.ole...@esi-group.com --- see if these work Updated Spec URL: https://download.copr.fedorainfracloud.org/results/openfoam/openfoam/fedora-37-x86_64/05164428-openfoam2212/openfoam2212.spec Updated SRPM URL: https://download.copr.fedorainfracloud.org/results/openfoam/openfoam/fedora-37-x86_64/05164428-openfoam2212/openfoam2212-2212.1-221221.fc37.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. https://bugzilla.redhat.com/show_bug.cgi?id=1816301 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 1816301] Review Request: openfoam - computational fluid dynamics
https://bugzilla.redhat.com/show_bug.cgi?id=1816301 --- Comment #29 from Ankur Sinha (FranciscoD) --- Ah, could you put a link to the updated spec/srpm please? I use `fedora-review` for the first round, and it picks the spec/srpm from the latest comment in the bug here. -- 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. https://bugzilla.redhat.com/show_bug.cgi?id=1816301 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 1816301] Review Request: openfoam - computational fluid dynamics
https://bugzilla.redhat.com/show_bug.cgi?id=1816301 --- Comment #28 from mark.ole...@esi-group.com --- Thanks for the heads-up on this. I've opened a corresponding issue (https://develop.openfoam.com/Development/openfoam/-/issues/2714) for more follow ups. Do you need a newer spec file? I see from the log that you are building v2006 - the last release was v2212 (Dec 2022). [Hard to believe that it's been that long already] -- 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. https://bugzilla.redhat.com/show_bug.cgi?id=1816301 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 1816301] Review Request: openfoam - computational fluid dynamics
https://bugzilla.redhat.com/show_bug.cgi?id=1816301 --- Comment #27 from Ankur Sinha (FranciscoD) --- Hi Mark, The build fails for rawhide related to a GCC change from the looks of it: ``` primitives/strings/word/word.C:180:32: error: 'uintptr_t' was not declared in this scope 180 | buf << "0x" << std::hex << uintptr_t(ptr); |^ primitives/strings/word/word.C:33:1: note: 'uintptr_t' is defined in header ''; did you forget to '#include '? 32 | #include +++ |+#include 33 | make: *** [/builddir/build/BUILD/OpenFOAM-v2006/wmake/rules/General/transform:35: /builddir/build/BUILD/OpenFOAM-v2006/build/linux64GccDPInt32Opt/src/OpenFOAM/primitives/strings/word/word.o] Error 1 make: *** Waiting for unfinished jobs Done logging to 'log.build' + : + '[' -f log.build ']' ++ cat log.build ++ sed -ne 's/.*bin *= *\([0-9][0-9]*\).*/\1/p;' ++ sed -ne '$p' + bins= ++ cat log.build ++ sed -ne 's/.*lib *= *\([0-9][0-9]*\).*/\1/p;' ++ sed -ne '$p' + libs= + '[' 0 = 0 ']' ``` Should be an easy fix (but there may be more places where these tiny tweaks are required): https://gcc.gnu.org/gcc-13/porting_to.html I'll do another round of review once it builds. Cheers, Ankur -- 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 https://bugzilla.redhat.com/show_bug.cgi?id=1816301 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2165669] Review Request: python-fspath - Handling path names and executables more comfortable
https://bugzilla.redhat.com/show_bug.cgi?id=2165669 --- Comment #6 from Jakub Kadlčík --- Created attachment 1946692 --> https://bugzilla.redhat.com/attachment.cgi?id=1946692&action=edit The .spec file difference from Copr build 5366420 to 5569287 -- 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. https://bugzilla.redhat.com/show_bug.cgi?id=2165669 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2165669] Review Request: python-fspath - Handling path names and executables more comfortable
https://bugzilla.redhat.com/show_bug.cgi?id=2165669 Jakub Kadlčík changed: What|Removed |Added URL||http://return42.github.io/f ||spath --- Comment #7 from Jakub Kadlčík --- Copr build: https://copr.fedorainfracloud.org/coprs/build/5569287 (succeeded) Review template: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2165669-python-fspath/fedora-rawhide-x86_64/05569287-python-fspath/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. -- 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. https://bugzilla.redhat.com/show_bug.cgi?id=2165669 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2165669] Review Request: python-fspath - Handling path names and executables more comfortable
https://bugzilla.redhat.com/show_bug.cgi?id=2165669 Ankur Sinha (FranciscoD) changed: What|Removed |Added Flags|needinfo?(sanjay.ankur@gmai | |l.com) | --- Comment #5 from Ankur Sinha (FranciscoD) --- Spec URL: https://ankursinha.fedorapeople.org/python-fspath/python-fspath.spec SRPM URL: https://ankursinha.fedorapeople.org/python-fspath/python-fspath-20230218-1.fc39.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. https://bugzilla.redhat.com/show_bug.cgi?id=2165669 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173523] Review Request: qt6-qtspeech - Speech component
https://bugzilla.redhat.com/show_bug.cgi?id=2173523 --- Comment #7 from Jakub Kadlčík --- Copr build: https://copr.fedorainfracloud.org/coprs/build/5569125 (succeeded) Review template: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2173523-qt6-qtspeech/fedora-rawhide-x86_64/05569125-qt6-qtspeech/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. -- 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. https://bugzilla.redhat.com/show_bug.cgi?id=2173523 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173523] Review Request: qt6-qtspeech - Speech component
https://bugzilla.redhat.com/show_bug.cgi?id=2173523 --- Comment #6 from Jakub Kadlčík --- Created attachment 1946683 --> https://bugzilla.redhat.com/attachment.cgi?id=1946683&action=edit The .spec file difference from Copr build 5569034 to 5569125 -- 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 https://bugzilla.redhat.com/show_bug.cgi?id=2173523 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173523] Review Request: qt6-qtspeech - Speech component
https://bugzilla.redhat.com/show_bug.cgi?id=2173523 --- Comment #5 from Jan Grulich --- (In reply to Jakub Kadlčík from comment #4) > Copr build: > https://copr.fedorainfracloud.org/coprs/build/5569034 > (failed) > > Build log: > https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora- > review-2173523-qt6-qtspeech/fedora-rawhide-x86_64/05569034-qt6-qtspeech/ > builder-live.log.gz > > Please make sure the package builds successfully at least for Fedora Rawhide. > > - If the build failed for unrelated reasons (e.g. temporary network > unavailability), please ignore it. > - If the build failed because of missing BuildRequires, please make sure they > are listed in the "Depends On" field > > > --- > 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. Fixed. Missing BR: qt6-qtmultimedia-devel qt6-qtdeclarative-devel. Now it should build. -- 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 https://bugzilla.redhat.com/show_bug.cgi?id=2173523 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2051859] Review Request: fbf-mitra-font : Experimental Monospace Bengali Font
https://bugzilla.redhat.com/show_bug.cgi?id=2051859 Parag AN(पराग) changed: What|Removed |Added Flags|fedora-review? | |needinfo?(panem...@gmail.co | |m) | Assignee|panem...@gmail.com |nob...@fedoraproject.org --- Comment #7 from Parag AN(पराग) --- Sorry, no time currently to review this package. -- 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. https://bugzilla.redhat.com/show_bug.cgi?id=2051859 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
needinfo canceled: [Bug 2051859] Review Request: fbf-mitra-font : Experimental Monospace Bengali Font
Product: Fedora Version: rawhide Component: Package Review Parag AN(पराग) has canceled Package Review 's request for Parag AN(पराग) 's needinfo: Bug 2051859: Review Request: fbf-mitra-font : Experimental Monospace Bengali Font https://bugzilla.redhat.com/show_bug.cgi?id=2051859 --- Comment #7 from Parag AN(पराग) --- Sorry, no time currently to review this package. ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173501] Review Request: pe-bear - Portable Executable analyzing tool with a friendly GUI
https://bugzilla.redhat.com/show_bug.cgi?id=2173501 Fedora Update System changed: What|Removed |Added Status|POST|MODIFIED --- Comment #5 from Fedora Update System --- FEDORA-2023-157c1e23c3 has been submitted as an update to Fedora 37. https://bodhi.fedoraproject.org/updates/FEDORA-2023-157c1e23c3 -- 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. https://bugzilla.redhat.com/show_bug.cgi?id=2173501 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173523] Review Request: qt6-qtspeech - Speech component
https://bugzilla.redhat.com/show_bug.cgi?id=2173523 Benson Muite changed: What|Removed |Added CC||benson_mu...@emailplus.org Status|NEW |ASSIGNED Flags||fedora-review? Assignee|nob...@fedoraproject.org|benson_mu...@emailplus.org -- 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. https://bugzilla.redhat.com/show_bug.cgi?id=2173523 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173501] Review Request: pe-bear - Portable Executable analyzing tool with a friendly GUI
https://bugzilla.redhat.com/show_bug.cgi?id=2173501 Benson Muite changed: What|Removed |Added CC||benson_mu...@emailplus.org --- Comment #4 from Benson Muite --- Any reason not to package bundled bearparser and capstone separately? -- 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 https://bugzilla.redhat.com/show_bug.cgi?id=2173501 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2168929] Review Request: liboqs - implementation of PQ crypto algorithms
https://bugzilla.redhat.com/show_bug.cgi?id=2168929 Dmitry Belyavskiy changed: What|Removed |Added Link ID||Gitlab ||fedora/legal/fedora-license ||-data/-/issues/158 -- 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 https://bugzilla.redhat.com/show_bug.cgi?id=2168929 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2168929] Review Request: liboqs - implementation of PQ crypto algorithms
https://bugzilla.redhat.com/show_bug.cgi?id=2168929 Dmitry Belyavskiy changed: What|Removed |Added Flags|needinfo?(dbelyavs@redhat.c | |om) | --- Comment #18 from Dmitry Belyavskiy --- Not yet, AFAIK. Is it the last show-stopper? -- 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 https://bugzilla.redhat.com/show_bug.cgi?id=2168929 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173501] Review Request: pe-bear - Portable Executable analyzing tool with a friendly GUI
https://bugzilla.redhat.com/show_bug.cgi?id=2173501 --- Comment #3 from Fedora Admin user for bugzilla script actions --- The Pagure repository was created at https://src.fedoraproject.org/rpms/pe-bear -- 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 https://bugzilla.redhat.com/show_bug.cgi?id=2173501 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2165381] Review Request: ancient - Modern decompressor for old data compression formats
https://bugzilla.redhat.com/show_bug.cgi?id=2165381
--- Comment #2 from Benson Muite ---
Package Review
==
Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed
= MUST items =
C/C++:
[x]: Package does not contain kernel modules.
[x]: Package contains no static executables.
[x]: If your application is a C or C++ application you must list a
BuildRequires against gcc, gcc-c++ or clang.
[x]: ldconfig not called in %post and %postun for Fedora 28 and later.
[x]: Package does not contain any libtool archives (.la)
[x]: Rpath absent or only used for internal libs.
[x]: Development (unversioned) .so files in -devel subpackage, if present.
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.
[x]: License field in the package spec file matches the actual license.
Note: Checking patched sources after %prep for licenses. Licenses
found: "BSD 2-Clause License", "Unknown or generated", "bzip2 and
libbzip2 License v1.0.6". 171 files have unknown license. Detailed
output of licensecheck in
/home/FedoraPackaging/reviews/ancient/2165381-ancient/licensecheck.txt
[x]: License file installed when any subpackage combination is installed.
[x]: If the package is under multiple licenses, the licensing breakdown
must be documented in the spec.
[x]: %build honors applicable compiler flags or justifies otherwise.
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[ ]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.
[x]: 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.
[-]: 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]: Useful -debuginfo package or justification otherwise.
[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 package is included in %license.
[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
= SHOULD items =
Generic:
[-]: If the source package does not include license text(s) as a separate
file from upstream, the packager SHOULD query upstream to include it.
[x]: Final provides and requires are sane (see attachments).
[x]: Fully versioned dependency in subpackages if applicable.
Note: No Requires: %{name}%{?_isa} = %{version}-%{release} in ancient-
devel , ancient-libs
[ ]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[=]: Sources are verified with gpgverify first in %prep if upstream
publishes signatures.
Note: gpgverify is not used.
[x]: Package should compile and build into bina
[Bug 2173523] Review Request: qt6-qtspeech - Speech component
https://bugzilla.redhat.com/show_bug.cgi?id=2173523 --- Comment #4 from Jakub Kadlčík --- Copr build: https://copr.fedorainfracloud.org/coprs/build/5569034 (failed) Build log: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2173523-qt6-qtspeech/fedora-rawhide-x86_64/05569034-qt6-qtspeech/builder-live.log.gz Please make sure the package builds successfully at least for Fedora Rawhide. - If the build failed for unrelated reasons (e.g. temporary network unavailability), please ignore it. - If the build failed because of missing BuildRequires, please make sure they are listed in the "Depends On" field --- 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. -- 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 https://bugzilla.redhat.com/show_bug.cgi?id=2173523 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173523] Review Request: qt6-qtspeech - Speech component
https://bugzilla.redhat.com/show_bug.cgi?id=2173523 --- Comment #3 from Jakub Kadlčík --- Created attachment 1946675 --> https://bugzilla.redhat.com/attachment.cgi?id=1946675&action=edit The .spec file difference from Copr build 5569008 to 5569034 -- 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. https://bugzilla.redhat.com/show_bug.cgi?id=2173523 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173523] Review Request: qt6-qtspeech - Speech component
https://bugzilla.redhat.com/show_bug.cgi?id=2173523 --- Comment #2 from Jan Grulich --- (In reply to Jakub Kadlčík from comment #1) > Copr build: > https://copr.fedorainfracloud.org/coprs/build/5569008 > (failed) > > Build log: > https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora- > review-2173523-qt6-qtspeech/fedora-rawhide-x86_64/05569008-qt6-qtspeech/ > builder-live.log.gz > > Please make sure the package builds successfully at least for Fedora Rawhide. > > - If the build failed for unrelated reasons (e.g. temporary network > unavailability), please ignore it. > - If the build failed because of missing BuildRequires, please make sure they > are listed in the "Depends On" field > > > --- > 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. Fixed. It was missing BR: gcc-c++ cmake ninja-build. -- 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 https://bugzilla.redhat.com/show_bug.cgi?id=2173523 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173523] Review Request: qt6-qtspeech - Speech component
https://bugzilla.redhat.com/show_bug.cgi?id=2173523 Jakub Kadlčík changed: What|Removed |Added URL||http://www.qt.io --- Comment #1 from Jakub Kadlčík --- Copr build: https://copr.fedorainfracloud.org/coprs/build/5569008 (failed) Build log: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2173523-qt6-qtspeech/fedora-rawhide-x86_64/05569008-qt6-qtspeech/builder-live.log.gz Please make sure the package builds successfully at least for Fedora Rawhide. - If the build failed for unrelated reasons (e.g. temporary network unavailability), please ignore it. - If the build failed because of missing BuildRequires, please make sure they are listed in the "Depends On" field --- 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. -- 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. https://bugzilla.redhat.com/show_bug.cgi?id=2173523 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173523] Review Request: qt6-qtspeech - Speech component
https://bugzilla.redhat.com/show_bug.cgi?id=2173523 Jan Grulich changed: What|Removed |Added CC||ngomp...@gmail.com Blocks||1917559 (qt6-reviews) Alias||qt6-qtspeech Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1917559 [Bug 1917559] Qt6-related package review tracker -- 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 https://bugzilla.redhat.com/show_bug.cgi?id=2173523 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
[Bug 2173523] New: Review Request: qt6-qtspeech - Speech component
https://bugzilla.redhat.com/show_bug.cgi?id=2173523 Bug ID: 2173523 Summary: Review Request: qt6-qtspeech - Speech component Product: Fedora Version: rawhide Hardware: All OS: Linux Status: NEW Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: jgrul...@redhat.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Target Milestone: --- Classification: Fedora Spec URL: https://jgrulich.fedorapeople.org/qt6-qtspeech/qt6-qtspeech.spec SRPM URL: https://jgrulich.fedorapeople.org/qt6-qtspeech/qt6-qtspeech-6.4.2-1.fc37.src.rpm Description: The module enables a Qt application to support accessibility features such as text-to-speech, which is useful for end-users who are visually challenged or cannot access the application for whatever reason. The most common use case where text-to-speech comes in handy is when the end-user is driving and cannot attend the incoming messages on the phone. In such a scenario, the messaging application can read out the incoming message. Qt Serial Port provides the basic functionality, which includes configuring, I/O operations, getting and setting the control signals of the RS-232 pinouts. Fedora Account System Username: jgrulich -- 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. https://bugzilla.redhat.com/show_bug.cgi?id=2173523 ___ 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 Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
