Bug 2007995 - Review Request: rust-yansi - Rust terminal color painting library
Summary: Review Request: rust-yansi - Rust terminal color painting library
Keywords:
Status: CLOSED DUPLICATE of bug 2079341
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
unspecified
medium
Target Milestone: ---
Assignee: Michel Lind
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2021-09-26 20:55 UTC by Jan Baudisch
Modified: 2022-05-03 21:21 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2022-05-03 21:21:51 UTC
Type: ---
Embargoed:
michel: fedora-review?


Attachments (Terms of Use)

Comment 1 Michel Lind 2022-02-03 05:48:52 UTC
Taking this review (removing the blocker on FE-NEEDSPONSOR since you're sponsored now!)

One change request: so the %bcond_without you see at the top of the spec as generated by rust2rpm actually controls whether that flag is enabled or not, see https://rpm-software-management.github.io/rpm/manual/conditionalbuilds.html

so:

%bcond_without check means %{with check} by default, unless compiled --without check

If the Rust compiler we have is too old for the test suite, then you just flip that definition to be %bcond_with check, and put a comment above it, no need to comment out the entire %check section

This makes it easy to test locally if you think checks can be enabled again:

fedpkg srpm (or rpmbuild -bs rust-yansi.spec)
mock /path/to/rust-yansi-*.src.rpm --with check

Comment 2 Jan Baudisch 2022-02-13 13:02:31 UTC
Thank you for pointing this out. I changed it and updated the spec and srpm.

Comment 3 Jan Baudisch 2022-05-03 21:21:51 UTC

*** This bug has been marked as a duplicate of bug 2079341 ***


Note You need to log in before you can comment on or make changes to this bug.