Spec URL: https://principis.fedorapeople.org/rust-path-clean.spec SRPM URL: https://principis.fedorapeople.org/rust-path-clean-1.0.1-1.fc41.src.rpm Description: A Rust implementation of cleanname or path.Clean. Fedora Account System Username: principis
Copr build: https://copr.fedorainfracloud.org/coprs/build/7202793 (succeeded) Review template: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2271120-rust-path-clean/fedora-rawhide-x86_64/07202793-rust-path-clean/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.
Added comment for patch Spec URL: https://principis.fedorapeople.org/rust-path-clean.spec SRPM URL: https://principis.fedorapeople.org/rust-path-clean-1.0.1-1.fc41.src.rpm
Package looks good, with a minor issue. The LICENSE-APACHE file only contains the license header, not the actual license text for Apache-2.0. The full text is required to be included in redistributed sources. Please report this as an upstream issue.
Thanks! Opened an issue upstream and added the correct license in the spec. Spec URL: https://principis.fedorapeople.org/rust-path-clean.spec SRPM URL: https://principis.fedorapeople.org/rust-path-clean-1.0.1-1.fc41.src.rpm
Upstream issue: https://github.com/danreeves/path-clean/issues/15
Thanks! Can you use the URL to upstream license file instead of plain file name? Something like this should do it: """ # Missing Apache license: https://github.com/danreeves/path-clean/issues/15 Source1: https://github.com/danreeves/path-clean/raw/3876d7c/LICENSE-APACHE """
I resubmitted this package myself because this review seems to be stalled. *** This bug has been marked as a duplicate of bug 2290369 ***