https://bugzilla.redhat.com/show_bug.cgi?id=2174145
Bug ID: 2174145 Summary: Review Request: rust-build-fs-tree - Generate a filesystem tree from a macro or a YAML tree Product: Fedora Version: rawhide Status: NEW Component: Package Review Assignee: nobody@fedoraproject.org Reporter: michel@michel-slm.name QA Contact: extras-qa@fedoraproject.org CC: package-review@lists.fedoraproject.org Target Milestone: --- Classification: Fedora
Spec URL: https://salimma.fedorapeople.org/rust-build-fs-tree.spec SRPM URL: https://salimma.fedorapeople.org/rust-build-fs-tree-0.3.1-1.fc37.src.rpm
Description: Generate a filesystem tree from a macro or a YAML tree.
Fedora Account System Username: salimma
https://bugzilla.redhat.com/show_bug.cgi?id=2174145
Jakub Kadlčík jkadlcik@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- URL| |https://crates.io/crates/bu | |ild-fs-tree
--- Comment #1 from Jakub Kadlčík jkadlcik@redhat.com --- Copr build: https://copr.fedorainfracloud.org/coprs/build/5580129 (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=2174145
Michel Alexandre Salim michel@michel-slm.name changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks| |2174146
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=2174146 [Bug 2174146] Review Request: rust-wax - Opinionated and portable globs that can be matched against paths and directory trees
https://bugzilla.redhat.com/show_bug.cgi?id=2174145
Michel Alexandre Salim michel@michel-slm.name changed:
What |Removed |Added ---------------------------------------------------------------------------- Depends On| |2174143, 2174144 Doc Type|--- |If docs needed, set a value
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=2174143 [Bug 2174143] Review Request: rust-pipe-trait - Make it possible to chain regular functions https://bugzilla.redhat.com/show_bug.cgi?id=2174144 [Bug 2174144] Review Request: rust-text-block-macros - Create a multiline string literal
https://bugzilla.redhat.com/show_bug.cgi?id=2174145 Bug 2174145 depends on bug 2174143, which changed state.
Bug 2174143 Summary: Review Request: rust-pipe-trait - Make it possible to chain regular functions https://bugzilla.redhat.com/show_bug.cgi?id=2174143
What |Removed |Added ---------------------------------------------------------------------------- Status|POST |CLOSED Resolution|--- |RAWHIDE
https://bugzilla.redhat.com/show_bug.cgi?id=2174145 Bug 2174145 depends on bug 2174144, which changed state.
Bug 2174144 Summary: Review Request: rust-text-block-macros - Create a multiline string literal https://bugzilla.redhat.com/show_bug.cgi?id=2174144
What |Removed |Added ---------------------------------------------------------------------------- Status|POST |CLOSED Resolution|--- |RAWHIDE
https://bugzilla.redhat.com/show_bug.cgi?id=2174145
Fabio Valentini decathorpe@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED Assignee|nobody@fedoraproject.org |decathorpe@gmail.com Flags| |fedora-review? CC| |decathorpe@gmail.com
--- Comment #2 from Fabio Valentini decathorpe@gmail.com --- Two comments:
- v0.3.1 is no longer the latest version, please update to use a more recent release (if possible) - It looks like you also removed some optional dependencies that are only used for building the binaries, did you use a rust2rpm config file for this? The metadata patch doesn't account for the missing subpackages.
https://bugzilla.redhat.com/show_bug.cgi?id=2174145
Fabio Valentini decathorpe@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags| |needinfo?(michel@michel-slm | |.name)
https://bugzilla.redhat.com/show_bug.cgi?id=2174145
Michel Alexandre Salim michel@michel-slm.name changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|CLOSED |ASSIGNED Resolution|NOTABUG |--- Alias| |review-rust-build-fs-tree Keywords| |Reopened
--- Comment #4 from Michel Alexandre Salim michel@michel-slm.name --- Reopening, sorry for the delay.
https://bugzilla.redhat.com/show_bug.cgi?id=2174145
Fabio Valentini decathorpe@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags| |fedora-review+ Status|ASSIGNED |POST
--- Comment #5 from Fabio Valentini decathorpe@gmail.com --- Package was generated with rust2rpm, simplifying the review.
- package builds and installs without errors on rawhide - test suite is run and all unit tests pass ! latest version of the crate is packaged (wax depends on ^0.3) - license matches upstream specification (MIT) and is acceptable for Fedora - license file is included with %license in %files - package complies with Rust Packaging Guidelines
Package APPROVED.
===
Recommended post-import rust-sig tasks:
- add @rust-sig with "commit" access as package co-maintainer
- set bugzilla assignee overrides to @rust-sig (optional)
- set up package on release-monitoring.org: project: $crate homepage: https://crates.io/crates/$crate backend: crates.io version scheme: semantic version filter: alpha;beta;rc;pre distro: Fedora Package: rust-$crate
- track package in koschei for all built branches
===
Two comments:
- It would be great if you could poke the project for the "wax" crate to update their dependency, v0.3.1 is quite old. - Please make sure that the removed feature subpackages are correct. For example, I think you could remove the "structopt" subpackage as well.
https://bugzilla.redhat.com/show_bug.cgi?id=2174145
--- Comment #6 from Fedora Admin user for bugzilla script actions fedora-admin-xmlrpc@fedoraproject.org --- The Pagure repository was created at https://src.fedoraproject.org/rpms/rust-build-fs-tree
https://bugzilla.redhat.com/show_bug.cgi?id=2174145
--- Comment #7 from Michel Alexandre Salim michel@michel-slm.name --- Thanks! Filed https://github.com/olson-sean-k/wax/issues/43 asking the wax developer to update. Unfortunately build-fs-tree does not include any changelog so... I'll let them figure it out. There's also been no commit to wax since last August...
https://bugzilla.redhat.com/show_bug.cgi?id=2174145
Michel Alexandre Salim michel@michel-slm.name changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed In Version| |rust-build-fs-tree-0.3.1-2. | |fc39 Resolution|--- |RAWHIDE Status|POST |CLOSED Last Closed|2023-05-20 00:45:29 |2023-06-15 16:54:28
https://bugzilla.redhat.com/show_bug.cgi?id=2174145
--- Comment #8 from Fedora Update System updates@fedoraproject.org --- FEDORA-2023-43512b4d4f has been submitted as an update to Fedora 38. https://bodhi.fedoraproject.org/updates/FEDORA-2023-43512b4d4f
https://bugzilla.redhat.com/show_bug.cgi?id=2174145
--- Comment #9 from Fedora Update System updates@fedoraproject.org --- FEDORA-2023-5187dc9252 has been submitted as an update to Fedora 37. https://bodhi.fedoraproject.org/updates/FEDORA-2023-5187dc9252
https://bugzilla.redhat.com/show_bug.cgi?id=2174145
--- Comment #10 from Fedora Update System updates@fedoraproject.org --- FEDORA-EPEL-2023-49aa6f1164 has been submitted as an update to Fedora EPEL 9. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2023-49aa6f1164
https://bugzilla.redhat.com/show_bug.cgi?id=2174145
--- Comment #11 from Fedora Update System updates@fedoraproject.org --- FEDORA-2023-43512b4d4f has been pushed to the Fedora 38 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-43512b4d4f *` You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2023-43512b4d4f
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=2174145
--- Comment #12 from Fedora Update System updates@fedoraproject.org --- FEDORA-EPEL-2023-49aa6f1164 has been pushed to the Fedora EPEL 9 testing repository.
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2023-49aa6f1164
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=2174145
--- Comment #13 from Fedora Update System updates@fedoraproject.org --- FEDORA-2023-5187dc9252 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-5187dc9252 *` You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2023-5187dc9252
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=2174145
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Resolution|RAWHIDE |ERRATA
--- Comment #14 from Fedora Update System updates@fedoraproject.org --- FEDORA-2023-43512b4d4f has been pushed to the Fedora 38 stable repository. If problem still persists, please make note of it in this bug report.
https://bugzilla.redhat.com/show_bug.cgi?id=2174145
--- Comment #15 from Fedora Update System updates@fedoraproject.org --- FEDORA-EPEL-2023-49aa6f1164 has been pushed to the Fedora EPEL 9 stable repository. If problem still persists, please make note of it in this bug report.
https://bugzilla.redhat.com/show_bug.cgi?id=2174145
--- Comment #16 from Fedora Update System updates@fedoraproject.org --- FEDORA-2023-5187dc9252 has been pushed to the Fedora 37 stable repository. If problem still persists, please make note of it in this bug report.
package-review@lists.fedoraproject.org