Bug 1907017 - Review Request: rust-libnotcurses-sys - Rust wrappers for the Notcurses TUI library
Summary: Review Request: rust-libnotcurses-sys - Rust wrappers for the Notcurses TUI l...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Robert-André Mauchin 🐧
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2020-12-12 10:51 UTC by Nick Black
Modified: 2022-02-21 09:50 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2022-02-21 09:50:04 UTC
Type: ---
Embargoed:
zebob.m: fedora-review+


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Bugzilla 1907014 0 medium RELEASE_PENDING Review Request: rust-cty - Type aliases to C types like c_int for use with bindgen 2022-08-30 12:14:15 UTC

Description Nick Black 2020-12-12 10:51:51 UTC
Spec URL: https://nick-black.com/tabpower/fedora-rust/rust-libnotcurses-sys.spec
SRPM URL: https://nick-black.com/tabpower/fedora-rust/rust-libnotcurses-sys-2.0.10-1.fc32.src.rpm
Description: Rust wrappers for the Notcurses TUI library
Fedora Account System Username: nickblack

Created with rust2rpm.

Comment 1 Nick Black 2020-12-12 10:52:46 UTC
Depends on 1907016 and 1907014.

Comment 2 Fabio Valentini 2020-12-12 12:25:13 UTC
Corrected bug title for correct package name and summary.
Also no longer depends on rust-serial_test, it is already packaged.

Comment 3 Nick Black 2021-01-24 07:19:12 UTC
I've updated the spec to include %license, which was missing in 1907014. I've rebuilt the SRPM against f34, and updated to the current upstream version 2.1.7.

https://nick-black.com/tabpower/fedora-rust/rust-libnotcurses-sys.spec
https://nick-black.com/tabpower/fedora-rust/rust-libnotcurses-sys-2.0.10-1.fc34.src.rpm

Comment 4 Igor Raits 2021-01-24 10:24:59 UTC
I think you still have copied wrong version :)

~ via 🐍 v3.9.1 
❯ curl https://nick-black.com/tabpower/fedora-rust/rust-libnotcurses-sys.spec -s | grep %license

~ via 🐍 v3.9.1 
❯

Comment 5 Nick Black 2021-01-24 10:35:58 UTC
augh, i missed the 'fedora-rust' :/

[dank@localhost ~]$ curl https://nick-black.com/tabpower/fedora-rust/rust-libnotcurses-sys.spec | grep -i license
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  1972  100  1972    0     0  89636      0 --:--:-- --:--:-- --:--:-- 89636
# Upstream license specification: Apache-2.0
License:        ASL 2.0
%license LICENSE-APACHE
[dank@localhost ~]$

Comment 6 Robert-André Mauchin 🐧 2021-01-30 18:42:05 UTC
License is not in the crate:

+ cp -pr LICENSE-APACHE /builddir/build/BUILDROOT/rust-libnotcurses-sys-2.1.7-1.fc34.x86_64/usr/share/licenses/rust-libnotcurses-sys-devel
cp: cannot stat 'LICENSE-APACHE': No such file or directory
+ :
+ RPM_EC=0
++ jobs -p
+ exit 0
error: File not found: /builddir/build/BUILDROOT/rust-libnotcurses-sys-2.1.7-1.fc34.x86_64/usr/share/licenses/rust-libnotcurses-sys-devel/LICENSE-APACHE
RPM build errors:
    File not found: /builddir/build/BUILDROOT/rust-libnotcurses-sys-2.1.7-1.fc34.x86_64/usr/share/licenses/rust-libnotcurses-sys-devel/LICENSE-APACHE


Upstream license file is LICENSE and it is not included in the crate. Ask upstream to do so.

Comment 7 Nick Black 2021-02-01 20:41:56 UTC
Good catch regarding the `LICENSE`, thanks. I've added it to the crate, and it will be present with the 2.1.8 release. I'm expecting that any day now, so I'll go ahead and push a new spec/SRPM when it drops.

Comment 8 Nick Black 2021-02-03 17:56:24 UTC
I've:

1) released a new 2.1.8 upstream crate, containing `LICENSE`
2) amended the spec file to point at `LICENSE` rather than `LICENSE-APACHE`
3) updated the spec file for 2.1.8
4) rebuilt the SRPM

Spec URL: https://nick-black.com/tabpower/fedora-rust/rust-libnotcurses-sys.spec
SRPM URL: https://nick-black.com/tabpower/fedora-rust/rust-libnotcurses-sys-2.1.8-1.fc34.src.rpm

The dep is now on libnotcurses 2.1.8, which is slated for rollout to f34.

Feel free to PTAL, and thanks for your patience!

Comment 9 Robert-André Mauchin 🐧 2021-02-04 23:26:36 UTC
 - There is a typo here, I assume you meant "Feb"

Wed Fed 03 09:37:51 EST 2021 nick black <dankamongmen> - 2.1.8-1

 - License ok
 - Latest version packaged
 - Builds in mock
 - No rpmlint errors
 - Conforms to Packaging Guidelines

Package approved.

Comment 10 Nick Black 2021-02-05 03:30:05 UTC
Fixed the typo. Thanks so much!

Comment 11 Nick Black 2021-02-05 03:34:43 UTC
Module requested at https://pagure.io/releng/fedora-scm-requests/issue/32031

Comment 12 Mohan Boddu 2021-02-08 16:43:43 UTC
(fedscm-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/rust-libnotcurses-sys

Comment 13 Mattia Verga 2022-02-21 09:50:04 UTC
Package is in repositories (although it is FTB), closing review.


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