Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 1961856
Summary: | F35FailsToInstall: rust-cursive+bear-lib-terminal-devel, rust-cursive+blt-backend-devel, rust-cursive+pancurses-backend-devel, rust-cursive+pancurses-devel | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Miro Hrončok <mhroncok> |
Component: | rust-cursive | Assignee: | Davide Cavalca <dcavalca> |
Status: | CLOSED WORKSFORME | QA Contact: | |
Severity: | unspecified | Docs Contact: | |
Priority: | unspecified | ||
Version: | rawhide | CC: | dcavalca, decathorpe, rust-sig |
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Whiteboard: | |||
Fixed In Version: | rust-cursive-0.16.3-2.fc35 | Doc Type: | If docs needed, set a value |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2021-05-25 19:30:18 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | 1962269 | ||
Bug Blocks: | 1927313 |
Description
Miro Hrončok
2021-05-18 20:21:29 UTC
Uh, this was working when I tested it locally so I probably messed something up while importing. Define "working locally"? Those broken dependencies are caused by optional features that are not enabled by default, and hence do not affect "buildability", only "installability" of all subpackages. Unless you build in mock with "--postinstall" (which I recommend doing for Rust packages) you wouldn't see any errors. Though the package review should definitely have found these non-installable subpackages ... I manually did mock --install afair, I suspect I missed some of the subpackages when doing that. I didn't know about --postinstall, will definitely use that the next time, thanks! Will poke at this tomorrow and try to get it sorted out. I have reviewed rust-pancurses for you. I assume you're going to remove the "bear-lib-terminal" dependency and "blt-backend" features instead of packaging that C library as well? You might be interested in a relatively new "rust2rpm" feature. You could add something like this to a .rust2rpm.conf file in rust-cursive's dist-git repo: ``` [DEFAULT] unwanted-features = bear-lib-terminal blt-backend ``` This will prevent rust2pm from automatically generating feature subpackages for those two optional dependencies / features, without having to manually patch them out (reducing the burden for rebasing patches on version updates). You can look at https://src.fedoraproject.org/rpms/rust-reqwest/blob/rawhide/f/.rust2rpm.conf for an in-use example of the syntax. Thanks! I looked briefly at packaging that C library earlier today -- it's doable but it has three bundled dependencies, one of which is under a custom license that would need to go through legal. I'm checking with the below folks if that's something we actually need or not (as below is the reason I was packaging cursive in the first place). I'll also submit pancurses for f34 as soon as ncurses makes it to stable (which should be tomorrow). Ok, below only uses crossterm and doesn't need this, so we're good. Hello, Please note that this comment was generated automatically. If you feel that this output has mistakes, please contact me via email (mhroncok). All subpackages of a package against which this bug was filled are now installable or removed from Fedora 35. Thanks for taking care of it! |