[Bug 2235055] New: Review Request: scancode-toolkit - Scan code and detect licenses, copyrights, and more.
https://bugzilla.redhat.com/show_bug.cgi?id=2235055
Bug ID: 2235055 Summary: Review Request: scancode-toolkit - Scan code and detect licenses, copyrights, and more. Product: Fedora Version: rawhide Status: NEW Component: Package Review Assignee: nobody@fedoraproject.org Reporter: zebob.m@gmail.com QA Contact: extras-qa@fedoraproject.org CC: package-review@lists.fedoraproject.org Target Milestone: --- Classification: Fedora
Spec URL: https://eclipseo.fedorapeople.org/for-review/scancode-toolkit.spec SRPM URL: https://eclipseo.fedorapeople.org/for-review/scancode-toolkit-32.0.6-1.fc38....
Description: ScanCode is an open-source tool to scan code and detect licenses, copyrights, and more. It provides detailed information about discovered licenses, copyrights, and other important details in various formats.
Fedora Account System Username: eclipseo
To build it against the dependencies, use the following COPR in your rawhide mock.cfg:
[copr:copr.fedorainfracloud.org:eclipseo:scancode-toolkit] name=Copr repo for scancode-toolkit owned by eclipseo baseurl=https://download.copr.fedorainfracloud.org/results/eclipseo/scancode-toolkit... type=rpm-md skip_if_unavailable=True gpgcheck=1 gpgkey=https://download.copr.fedorainfracloud.org/results/eclipseo/scancode-toolkit... repo_gpgcheck=0 enabled=1 enabled_metadata=1
https://bugzilla.redhat.com/show_bug.cgi?id=2235055
Fedora Review Service fedora-review-bot@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- URL| |https://scancode-toolkit.re | |adthedocs.io/
--- Comment #1 from Fedora Review Service fedora-review-bot@fedoraproject.org --- Copr build: https://copr.fedorainfracloud.org/coprs/build/6345534 (failed)
Build log: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-rev...
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.
https://bugzilla.redhat.com/show_bug.cgi?id=2235055
Miroslav Suchý msuchy@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |msuchy@redhat.com Depends On| |2235065 (python-ftfy) Doc Type|--- |If docs needed, set a value Assignee|nobody@fedoraproject.org |msuchy@redhat.com
--- Comment #2 from Miroslav Suchý msuchy@redhat.com --- Deps need to be packaged first
No matching package to install: 'python3dist(commoncode) >= 31.0.2' No matching package to install: 'python3dist(container-inspector) >= 31' No matching package to install: 'python3dist(debian-inspector) >= 31' No matching package to install: 'python3dist(dparse2) >= 0.7' No matching package to install: 'python3dist(extractcode) >= 31' No matching package to install: 'python3dist(extractcode[full]) >= 31' No matching package to install: 'python3dist(fingerprints) >= 0.6' No matching package to install: 'python3dist(ftfy) >= 6' No matching package to install: 'python3dist(gemfileparser2) >= 0.9' No matching package to install: 'python3dist(intbitset) >= 3.0.2' No matching package to install: 'python3dist(jsonstreams) >= 0.5' No matching package to install: 'python3dist(license-expression) >= 30.1.1' No matching package to install: 'python3dist(packageurl-python) >= 0.9' No matching package to install: 'python3dist(packvers) >= 21' No matching package to install: 'python3dist(parameter-expansion-patched) >= 0.3.1' No matching package to install: 'python3dist(pip-requirements-parser) >= 32.0.1' No matching package to install: 'python3dist(pkginfo2) >= 30' No matching package to install: 'python3dist(plugincode) >= 32' No matching package to install: 'python3dist(pyahocorasick) >= 2' No matching package to install: 'python3dist(pygmars) >= 0.7' No matching package to install: 'python3dist(pymaven-patch) >= 0.2.8' No matching package to install: 'python3dist(saneyaml) >= 0.6' No matching package to install: 'python3dist(typecode) >= 30.0.1' No matching package to install: 'python3dist(typecode[full]) >= 30.0.1' No matching package to install: 'python3dist(urlpy)'
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=2235065 [Bug 2235065] Review Request: python-ftfy - Fixes mojibake and other glitches in Unicode text, after the fact
https://bugzilla.redhat.com/show_bug.cgi?id=2235055
Miroslav Suchý msuchy@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED
https://bugzilla.redhat.com/show_bug.cgi?id=2235055
Miroslav Suchý msuchy@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Depends On| |2235066 | |(python-gemfileparser2), | |2235067 (python-intbitset), | |2235068 | |(python-jsonstreams), | |2235069 | |(python-license-expression) | |, 2235071 | |(python-packageurl-python), | |2235072 (python-packvers), | |2235073 | |(python-parameter-expansion | |-patched), 2235075 | |(python-pip-requirements-pa | |rser), 2235076 | |(python-pkginfo2), 2235077 | |(python-plugincode), | |2235078 | |(python-pyahocorasick), | |2235079 (python-pygmars), | |2235080 | |(python-pymaven-patch), | |2235082 (python-saneyaml), | |2235084 (python-typecode)
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=2235066 [Bug 2235066] Review Request: python-gemfileparser2 - Library to parse Rubygem gemspec https://bugzilla.redhat.com/show_bug.cgi?id=2235067 [Bug 2235067] Review Request: python-intbitset - Python C-based extension implementing fast integer bit sets https://bugzilla.redhat.com/show_bug.cgi?id=2235068 [Bug 2235068] Review Request: python-jsonstreams - Python library for writing JSON documents as streams https://bugzilla.redhat.com/show_bug.cgi?id=2235069 [Bug 2235069] Review Request: python-license-expression - Utility library to parse, normalize and compare License expressions for Python https://bugzilla.redhat.com/show_bug.cgi?id=2235071 [Bug 2235071] Review Request: python-packageurl-python - Python implementation of the package url spec https://bugzilla.redhat.com/show_bug.cgi?id=2235072 [Bug 2235072] Review Request: python-packvers - Core utilities for Python packages https://bugzilla.redhat.com/show_bug.cgi?id=2235073 [Bug 2235073] Review Request: python-parameter-expansion-patched - POSIX Parameter Expansion in Python https://bugzilla.redhat.com/show_bug.cgi?id=2235075 [Bug 2235075] Review Request: python-pip-requirements-parser - Mostly correct pip requirements parsing library https://bugzilla.redhat.com/show_bug.cgi?id=2235076 [Bug 2235076] Review Request: python-pkginfo2 - API for querying the distutils metadata written in the PKG-INFO file https://bugzilla.redhat.com/show_bug.cgi?id=2235077 [Bug 2235077] Review Request: python-plugincode - Plugable functionality with plugins used by ScanCode toolkit https://bugzilla.redhat.com/show_bug.cgi?id=2235078 [Bug 2235078] Review Request: python-pyahocorasick - Python module (C extension and plain Python) implementing Aho-Corasick algorithm https://bugzilla.redhat.com/show_bug.cgi?id=2235079 [Bug 2235079] Review Request: python-pygmars - Craft simple regex-based small language lexers and parser https://bugzilla.redhat.com/show_bug.cgi?id=2235080 [Bug 2235080] Review Request: python-pymaven-patch - Library for working with Maven repositories via Python https://bugzilla.redhat.com/show_bug.cgi?id=2235082 [Bug 2235082] Review Request: python-saneyaml - Cleaner, simpler, safer and saner YAML parsing/serialization in Python https://bugzilla.redhat.com/show_bug.cgi?id=2235084 [Bug 2235084] Review Request: python-typecode - Comprehensive filetype and mimetype detection
https://bugzilla.redhat.com/show_bug.cgi?id=2235055
Benson Muite benson_muite@emailplus.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Depends On| |1748059
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=1748059 [Bug 1748059] Review Request: python-license-expression - Library to parse, compare, simplify and normalize license expressions
https://bugzilla.redhat.com/show_bug.cgi?id=2235055 Bug 2235055 depends on bug 2235069, which changed state.
Bug 2235069 Summary: Review Request: python-license-expression - Utility library to parse, normalize and compare License expressions for Python https://bugzilla.redhat.com/show_bug.cgi?id=2235069
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution|--- |DUPLICATE
https://bugzilla.redhat.com/show_bug.cgi?id=2235055
--- Comment #3 from Robert-André Mauchin 🐧 zebob.m@gmail.com --- Spec URL: https://eclipseo.fedorapeople.org/for-review/scancode-toolkit.spec SRPM URL: https://eclipseo.fedorapeople.org/for-review/scancode-toolkit-32.0.8-1.fc39....
https://bugzilla.redhat.com/show_bug.cgi?id=2235055
--- Comment #4 from Fedora Review Service fedora-review-bot@fedoraproject.org --- Copr build: https://copr.fedorainfracloud.org/coprs/build/6548905 (failed)
Build log: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-rev...
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.
https://bugzilla.redhat.com/show_bug.cgi?id=2235055 Bug 2235055 depends on bug 2235071, which changed state.
Bug 2235071 Summary: Review Request: python-packageurl-python - Python implementation of the package url spec https://bugzilla.redhat.com/show_bug.cgi?id=2235071
What |Removed |Added ---------------------------------------------------------------------------- Status|ON_QA |CLOSED Resolution|--- |ERRATA
https://bugzilla.redhat.com/show_bug.cgi?id=2235055 Bug 2235055 depends on bug 2235080, which changed state.
Bug 2235080 Summary: Review Request: python-pymaven-patch - Library for working with Maven repositories via Python https://bugzilla.redhat.com/show_bug.cgi?id=2235080
What |Removed |Added ---------------------------------------------------------------------------- Status|ON_QA |CLOSED Resolution|--- |ERRATA
https://bugzilla.redhat.com/show_bug.cgi?id=2235055
Robert-André Mauchin 🐧 zebob.m@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags| |needinfo?(msuchy@redhat.com | |)
--- Comment #5 from Robert-André Mauchin 🐧 zebob.m@gmail.com --- Every deps is now packaged, anyone up for the review? Thank you.
https://bugzilla.redhat.com/show_bug.cgi?id=2235055 Bug 2235055 depends on bug 2235078, which changed state.
Bug 2235078 Summary: Review Request: python-pyahocorasick - Python module (C extension and plain Python) implementing Aho-Corasick algorithm https://bugzilla.redhat.com/show_bug.cgi?id=2235078
What |Removed |Added ---------------------------------------------------------------------------- Status|POST |CLOSED Resolution|--- |CURRENTRELEASE
https://bugzilla.redhat.com/show_bug.cgi?id=2235055 Bug 2235055 depends on bug 2235067, which changed state.
Bug 2235067 Summary: Review Request: python-intbitset - Python C-based extension implementing fast integer bit sets https://bugzilla.redhat.com/show_bug.cgi?id=2235067
What |Removed |Added ---------------------------------------------------------------------------- Status|POST |CLOSED Resolution|--- |CURRENTRELEASE
https://bugzilla.redhat.com/show_bug.cgi?id=2235055 Bug 2235055 depends on bug 2235072, which changed state.
Bug 2235072 Summary: Review Request: python-packvers - Core utilities for Python packages https://bugzilla.redhat.com/show_bug.cgi?id=2235072
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution|--- |CURRENTRELEASE
https://bugzilla.redhat.com/show_bug.cgi?id=2235055 Bug 2235055 depends on bug 2235073, which changed state.
Bug 2235073 Summary: Review Request: python-parameter-expansion-patched - POSIX Parameter Expansion in Python https://bugzilla.redhat.com/show_bug.cgi?id=2235073
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution|--- |CURRENTRELEASE
https://bugzilla.redhat.com/show_bug.cgi?id=2235055
Miroslav Suchý msuchy@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags|needinfo?(msuchy@redhat.com | |) |
--- Comment #6 from Miroslav Suchý msuchy@redhat.com --- Ouch, I was under assumption that the deps are not yet packages as the related bugzillas were still open. I will proceed.
Hmmm, bumper: No matching package to install: 'python3dist(license-expression) >= 30.1.1' I am comaintainer for that packages. I will try to rebase it.
https://bugzilla.redhat.com/show_bug.cgi?id=2235055 Bug 2235055 depends on bug 2235082, which changed state.
Bug 2235082 Summary: Review Request: python-saneyaml - Cleaner, simpler, safer and saner YAML parsing/serialization in Python https://bugzilla.redhat.com/show_bug.cgi?id=2235082
What |Removed |Added ---------------------------------------------------------------------------- Status|ON_QA |CLOSED Resolution|--- |ERRATA
https://bugzilla.redhat.com/show_bug.cgi?id=2235055 Bug 2235055 depends on bug 2235065, which changed state.
Bug 2235065 Summary: Review Request: python-ftfy - Fixes mojibake and other glitches in Unicode text, after the fact https://bugzilla.redhat.com/show_bug.cgi?id=2235065
What |Removed |Added ---------------------------------------------------------------------------- Status|ON_QA |CLOSED Resolution|--- |ERRATA
https://bugzilla.redhat.com/show_bug.cgi?id=2235055 Bug 2235055 depends on bug 2235066, which changed state.
Bug 2235066 Summary: Review Request: python-gemfileparser2 - Library to parse Rubygem gemspec https://bugzilla.redhat.com/show_bug.cgi?id=2235066
What |Removed |Added ---------------------------------------------------------------------------- Status|ON_QA |CLOSED Resolution|--- |ERRATA
https://bugzilla.redhat.com/show_bug.cgi?id=2235055 Bug 2235055 depends on bug 2235068, which changed state.
Bug 2235068 Summary: Review Request: python-jsonstreams - Python library for writing JSON documents as streams https://bugzilla.redhat.com/show_bug.cgi?id=2235068
What |Removed |Added ---------------------------------------------------------------------------- Status|ON_QA |CLOSED Resolution|--- |ERRATA
https://bugzilla.redhat.com/show_bug.cgi?id=2235055
--- Comment #7 from Miroslav Suchý msuchy@redhat.com --- The license-expression 30.1.1 is finally in rawhide. So the build get past deps installation, but eventually fail on tests:
=========================== short test summary info ============================ FAILED tests/formattedcode/test_output_spdx.py::test_spdx_rdf_basic - assert ... XFAIL tests/cluecode/test_finder.py::TestUrl::test_misc_valid_unicode_or_punycode_urls_that_should_pass - reason: XFAIL tests/cluecode/test_finder.py::TestUrl::test_misc_valid_urls_that_should_pass - reason: XFAIL tests/textcode/test_pdf.py::TestPdf::test_numbered_text_lines_does_not_fail_on_autocad_test_pdf - Latest pdfminer.six from 2022 has a regression = 1 failed, 2684 passed, 1 skipped, 45243 deselected, 3 xfailed, 4000 warnings in 438.23s (0:07:18) =
And the whole build fails.
https://bugzilla.redhat.com/show_bug.cgi?id=2235055 Bug 2235055 depends on bug 2235075, which changed state.
Bug 2235075 Summary: Review Request: python-pip-requirements-parser - Mostly correct pip requirements parsing library https://bugzilla.redhat.com/show_bug.cgi?id=2235075
What |Removed |Added ---------------------------------------------------------------------------- Status|ON_QA |CLOSED Resolution|--- |ERRATA
https://bugzilla.redhat.com/show_bug.cgi?id=2235055 Bug 2235055 depends on bug 2235076, which changed state.
Bug 2235076 Summary: Review Request: python-pkginfo2 - API for querying the distutils metadata written in the PKG-INFO file https://bugzilla.redhat.com/show_bug.cgi?id=2235076
What |Removed |Added ---------------------------------------------------------------------------- Status|ON_QA |CLOSED Resolution|--- |ERRATA
https://bugzilla.redhat.com/show_bug.cgi?id=2235055 Bug 2235055 depends on bug 2235079, which changed state.
Bug 2235079 Summary: Review Request: python-pygmars - Craft simple regex-based small language lexers and parser https://bugzilla.redhat.com/show_bug.cgi?id=2235079
What |Removed |Added ---------------------------------------------------------------------------- Status|ON_QA |CLOSED Resolution|--- |ERRATA
https://bugzilla.redhat.com/show_bug.cgi?id=2235055 Bug 2235055 depends on bug 2235077, which changed state.
Bug 2235077 Summary: Review Request: python-plugincode - Plugable functionality with plugins used by ScanCode toolkit https://bugzilla.redhat.com/show_bug.cgi?id=2235077
What |Removed |Added ---------------------------------------------------------------------------- Status|ON_QA |CLOSED Resolution|--- |ERRATA
https://bugzilla.redhat.com/show_bug.cgi?id=2235055 Bug 2235055 depends on bug 2235084, which changed state.
Bug 2235084 Summary: Review Request: python-typecode - Comprehensive filetype and mimetype detection https://bugzilla.redhat.com/show_bug.cgi?id=2235084
What |Removed |Added ---------------------------------------------------------------------------- Status|ON_QA |CLOSED Resolution|--- |ERRATA
https://bugzilla.redhat.com/show_bug.cgi?id=2235055
--- Comment #8 from Miroslav Suchý msuchy@redhat.com --- Ping? How it is going?
https://bugzilla.redhat.com/show_bug.cgi?id=2235055
--- Comment #9 from Miroslav Suchý msuchy@redhat.com --- Gentle ping here. Or is there any blocker?
https://bugzilla.redhat.com/show_bug.cgi?id=2235055
David Timms dtimms@iinet.net.au changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |dtimms@iinet.net.au
--- Comment #10 from David Timms dtimms@iinet.net.au --- At the moment, mock from f39 bails out for both -rawhide and -40 (but seems unrelated dnf5): $ mock -r /etc/mock/fedora-rawhide-x86_64.cfg rebuild /home/media/media_disk/dev/scancode-toolkit/scancode-toolkit-32.0.8-1.fc39.src.rpm ... Start: cleaning package manager metadata Finish: cleaning package manager metadata INFO: enabled HW Info plugin INFO: Package manager dnf5 detected and used (direct choice) INFO: Buildroot is handled by package management downloaded with a bootstrap image: rpm-4.17.0-1.fc36.1.x86_64 python3-dnf-4.10.0-1.fc36.noarch python3-dnf-plugins-core-4.0.24-1.fc36.noarch Start: installing minimal buildroot with dnf5 execv(/usr/bin/dnf5) failed: No such file or directory ERROR: Exception(/home/media/media_disk/dev/scancode-toolkit/scancode-toolkit-32.0.8-1.fc39.src.rpm) Config(fedora-rawhide-x86_64) 0 minutes 2 seconds INFO: Results and/or logs in: /var/lib/mock/fedora-rawhide-x86_64/result ERROR: Command failed: # /usr/bin/systemd-nspawn -q -M 1f94d65f6514446b87cf2e1319ef738c -D /var/lib/mock/fedora-rawhide-x86_64-bootstrap/root -a --capability=cap_ipc_lock --bind=/tmp/mock-resolv.xp4chj56:/etc/resolv.conf --console=pipe --setenv=TERM=vt100 --setenv=SHELL=/bin/bash --setenv=HOME=/var/lib/mock/fedora-rawhide-x86_64/root/installation-homedir --setenv=HOSTNAME=mock --setenv=PATH=/usr/bin:/bin:/usr/sbin:/sbin '--setenv=PROMPT_COMMAND=printf "\033]0;<mock-chroot>\007"' '--setenv=PS1=<mock-chroot> \s-\v$ ' --setenv=LANG=C.UTF-8 --setenv=LC_MESSAGES=C.UTF-8 --resolv-conf=off /usr/bin/dnf5 --installroot /var/lib/mock/fedora-rawhide-x86_64/root/ --releasever 40 install @buildsys-build --setopt=deltarpm=False --setopt=allow_vendor_change=yes --allowerasing --setopt=tsflags=nocontexts execv(/usr/bin/dnf5) failed: No such file or directory
https://bugzilla.redhat.com/show_bug.cgi?id=2235055
--- Comment #11 from Miroslav Suchý msuchy@redhat.com --- mock -r fedora-rawhide-x86_64 --scrub=all will fix that.
https://bugzilla.redhat.com/show_bug.cgi?id=2235055
--- Comment #12 from Miroslav Suchý msuchy@redhat.com --- Gentle ping. I am looking forward to see this package in Fedora.
https://bugzilla.redhat.com/show_bug.cgi?id=2235055
Jerry James loganjerry@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Depends On| |2263338
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=2263338 [Bug 2263338] F40FailsToInstall: python3-pymaven-patch
https://bugzilla.redhat.com/show_bug.cgi?id=2235055
Xavier Bachelot xavier@bachelot.org changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |xavier@bachelot.org Flags| |fedora-review?
https://bugzilla.redhat.com/show_bug.cgi?id=2235055 Bug 2235055 depends on bug 2263338, which changed state.
Bug 2263338 Summary: F40FailsToInstall: python3-pymaven-patch https://bugzilla.redhat.com/show_bug.cgi?id=2263338
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |CLOSED Resolution|--- |WORKSFORME
https://bugzilla.redhat.com/show_bug.cgi?id=2235055
--- Comment #13 from Miroslav Suchý msuchy@redhat.com --- Ping. Can we proceed? Or should somebody else continue? I tested the original src.rpm and it still fails with the same tests failures.
https://bugzilla.redhat.com/show_bug.cgi?id=2235055
--- Comment #14 from Robert-André Mauchin 🐧 zebob.m@gmail.com --- *** Bug 2283259 has been marked as a duplicate of this bug. ***
https://bugzilla.redhat.com/show_bug.cgi?id=2235055
--- Comment #15 from Robert-André Mauchin 🐧 zebob.m@gmail.com --- *** Bug 2283258 has been marked as a duplicate of this bug. ***
https://bugzilla.redhat.com/show_bug.cgi?id=2235055
--- Comment #16 from Robert-André Mauchin 🐧 zebob.m@gmail.com --- Update
https://copr.fedorainfracloud.org/coprs/eclipseo/scancode-toolkit/build/7491...
Spec URL: https://eclipseo.fedorapeople.org/for-review/scancode-toolkit.spec SRPM URL: https://eclipseo.fedorapeople.org/for-review/scancode-toolkit-32.1.0-1.fc40....
Needs a new spdx-tools.
https://bugzilla.redhat.com/show_bug.cgi?id=2235055
--- Comment #17 from Fedora Review Service fedora-review-bot@fedoraproject.org --- Copr build: https://copr.fedorainfracloud.org/coprs/build/7492447 (failed)
Build log: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-rev...
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.
https://bugzilla.redhat.com/show_bug.cgi?id=2235055
Robert-André Mauchin 🐧 zebob.m@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Depends On| |2283281
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=2283281 [Bug 2283281] Review Request: python-pyshacl - Python validator for SHACL
https://bugzilla.redhat.com/show_bug.cgi?id=2235055
--- Comment #18 from Miroslav Suchý msuchy@redhat.com --- The summary should not end with a dot. https://docs.fedoraproject.org/en-US/packaging-guidelines/#_tags_and_section...
The patch should have linked the upstream issue or commented on why it was added. https://docs.fedoraproject.org/en-US/packaging-guidelines/PatchUpstreamStatu...
Otherwise, it looks good. Please build new spdx-tools in rawhide and update the above nitpicks and I will approve.
https://bugzilla.redhat.com/show_bug.cgi?id=2235055
--- Comment #19 from Robert-André Mauchin 🐧 zebob.m@gmail.com --- *** Bug 2283260 has been marked as a duplicate of this bug. ***
https://bugzilla.redhat.com/show_bug.cgi?id=2235055
Robert-André Mauchin 🐧 zebob.m@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags| |needinfo?(msuchy@redhat.com | |)
--- Comment #20 from Robert-André Mauchin 🐧 zebob.m@gmail.com --- Spec URL: https://eclipseo.fedorapeople.org/for-review/scancode-toolkit.spec SRPM URL: https://eclipseo.fedorapeople.org/for-review/scancode-toolkit-32.0.6-1.fc38....
I have addressed your points.
The patch is downstream only to remove some extra theme for the docs.
spdx-tools depends on 2283281 being reviewed and packaged.
https://bugzilla.redhat.com/show_bug.cgi?id=2235055
Miroslav Suchý msuchy@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags|fedora-review? |fedora-review+ |needinfo?(msuchy@redhat.com | |) |
--- Comment #21 from Miroslav Suchý msuchy@redhat.com --- All resolved. APPROVED. You are an experienced maintainer, so please build this one only after you finish with bug 2283281
https://bugzilla.redhat.com/show_bug.cgi?id=2235055
Fedora Admin user for bugzilla script actions fedora-admin-xmlrpc@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |POST
--- Comment #22 from Fedora Admin user for bugzilla script actions fedora-admin-xmlrpc@fedoraproject.org --- The Pagure repository was created at https://src.fedoraproject.org/rpms/scancode-toolkit
https://bugzilla.redhat.com/show_bug.cgi?id=2235055
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|POST |MODIFIED
--- Comment #23 from Fedora Update System updates@fedoraproject.org --- FEDORA-2024-f26252b1fb (scancode-toolkit-32.1.0-1.fc40) has been submitted as an update to Fedora 40. https://bodhi.fedoraproject.org/updates/FEDORA-2024-f26252b1fb
https://bugzilla.redhat.com/show_bug.cgi?id=2235055
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|MODIFIED |ON_QA
--- Comment #24 from Fedora Update System updates@fedoraproject.org --- FEDORA-2024-f26252b1fb has been pushed to the Fedora 40 testing repository. Soon you'll be able to install the update with the following command: `sudo dnf install --enablerepo=updates-testing --refresh --advisory=FEDORA-2024-f26252b1fb *` You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2024-f26252b1fb
See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.
https://bugzilla.redhat.com/show_bug.cgi?id=2235055 Bug 2235055 depends on bug 2283281, which changed state.
Bug 2283281 Summary: Review Request: python-pyshacl - Python validator for SHACL https://bugzilla.redhat.com/show_bug.cgi?id=2283281
What |Removed |Added ---------------------------------------------------------------------------- Status|ON_QA |CLOSED Resolution|--- |ERRATA
https://bugzilla.redhat.com/show_bug.cgi?id=2235055
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |ERRATA Status|ON_QA |CLOSED Last Closed| |2024-07-01 04:50:31
--- Comment #25 from Fedora Update System updates@fedoraproject.org --- FEDORA-2024-1eff6397d9 (scancode-toolkit-32.2.0-1.fc40) has been pushed to the Fedora 40 stable repository. If problem still persists, please make note of it in this bug report.
package-review@lists.fedoraproject.org
-
bugzilla＠redhat.com