Bug 1651759 - Review Request: rust-winapi-x86_64-pc-windows-gnu - Import libraries for the x86_64-pc-windows-gnu target.
Summary: Review Request: rust-winapi-x86_64-pc-windows-gnu - Import libraries for the ...
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Luis Bazan
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2018-11-20 17:28 UTC by Luis Segundo
Modified: 2018-11-21 17:09 UTC (History)
4 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2018-11-21 17:09:10 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Comment 1 Robert-André Mauchin 🐧 2018-11-20 21:57:33 UTC
Update your rust2rpm to the very latest version (from Rawhide), your generated SPEC use the old format. Also I don't see the point of packaging this, it seems Windows only like Winapi, why do you need it?

Comment 2 Luis Segundo 2018-11-21 01:15:33 UTC
Hello Robert,
I would like include this Package "dutree" more info https://docs.rs/dutree/0.2.9/dutree/

this package need the follow Dependencies
    getopts ^0.2
    regex ^0.2
    terminal_size ^0.1.7
    unicode-segmentation ^1.2.0
    unicode-width ^0.1.1

now "terminal_size" it's not include yet on fedora this one requiere the follow Dependencies

    libc ^0.2
    winapi ^0.3

"winapi" need the lib "winapi-x86_64-pc-windows-gnu" that I try to include


referecen
http://eminence.github.io/terminal-size/doc/terminal_size/index.html
https://docs.rs/winapi/0.3.6/winapi/

I will be waiting your comments

Thanks

Comment 3 Robert-André Mauchin 🐧 2018-11-21 10:59:04 UTC
This is not needed, when you package terminal_size, you need to patch the metadata (rust2rpm -p) to remove the windows only deps:


[target.'cfg(windows)'.dependencies]
winapi = { version = "0.3", features = ["winnt", "processenv", "wincon", "winbase"] }

Comment 4 Luis Segundo 2018-11-21 17:09:10 UTC
Hi Robert, 
thanks for the suggestion that worked for me,
I'll close this ticket

regards


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