Bug 2140674 - Review Request: rust-memfd - Pure-Rust library to work with Linux memfd and sealing
Summary: Review Request: rust-memfd - Pure-Rust library to work with Linux memfd and s...
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Richard W.M. Jones
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 2124697
TreeView+ depends on / blocked
 
Reported: 2022-11-07 15:30 UTC by Fabio Valentini
Modified: 2022-11-08 20:29 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2022-11-08 20:29:10 UTC
Type: ---
Embargoed:
rjones: fedora-review+


Attachments (Terms of Use)

Description Fabio Valentini 2022-11-07 15:30:03 UTC
Spec URL: https://decathorpe.fedorapeople.org/rust-memfd.spec
SRPM URL: https://decathorpe.fedorapeople.org/rust-memfd-0.6.1-1.fc37.src.rpm

Description:
Pure-Rust library to work with Linux memfd and sealing.

Fedora Account System Username: decathorpe

koji scratch build for rawhide: https://koji.fedoraproject.org/koji/taskinfo?taskID=93911689

An earlier version of this package has previously been available, but has been retired for more than 8 weeks, so this review is also pre-requisite for package unretirement:
https://src.fedoraproject.org/rpms/rust-memfd

Note that the build failure on s390x is due to an issue with the rustix package, not a problem with this one, and I'll try to fix it ASAP.

Comment 1 Richard W.M. Jones 2022-11-07 16:02:57 UTC
The package was generated by a trusted maintainer using rust2rpm so I'm
going to do a reduced review.

Upstream: https://github.com/lucab/memfd-rs

The latest version is 0.6.1.  (This is not tagged so github thinks
the latest is 0.4.1).  The latest version is therefore being packaged.

The license tag matches upstream (dual MIT, Apache) and is correct.

There are no downstream changes or patches.

The spec file is legible, matches the packaging guidelines, and is
largely generated.

The scratch build above failed on s390x.  I did another one
to see if the failure is temporary or permanent:
https://koji.fedoraproject.org/koji/taskinfo?taskID=93912721
And it is.  I think this needs to be fixed (upstream probably)
before this package can go into Fedora.

I downloaded and tested the package by compiling libblkio against it
(see bug 2124697).  I had to add this patch to libblkio:
https://gitlab.com/libblkio/libblkio/-/merge_requests/143
Anyway, it worked fine.

Comment 2 Fabio Valentini 2022-11-07 16:15:56 UTC
> And it is.  I think this needs to be fixed (upstream probably)
> before this package can go into Fedora.

No, that one is probably my fault.
It looks like some well-intentioned downstream changes to the rustix crate (to make it possible to run more tests) were broken, see my previous comment:

> Note that the build failure on s390x is due to an issue with the rustix package, not a problem with this one, and I'll try to fix it ASAP.

I'm testing a fix right now, and will submit a new scratch build once the fix has landed in rawhide.

Comment 3 Fabio Valentini 2022-11-07 16:28:07 UTC
Meh, it looks like my downstream change just plastered over an actual problem with rustix on s390x.
I've reported the problem here: https://github.com/bytecodealliance/rustix/issues/439

Comment 4 Fabio Valentini 2022-11-07 19:04:04 UTC
Nevermind, looks like our tooling for creating some of our downstream patches don't like the way rustix does things ...

With the fix for that problem in rustix submitted to rawhide, a new memfd scratch build now succeeded:
https://koji.fedoraproject.org/koji/taskinfo?taskID=93916424

Comment 5 Richard W.M. Jones 2022-11-07 19:17:35 UTC
***
This package is APPROVED by rjones
***

Comment 6 Fabio Valentini 2022-11-07 19:25:11 UTC
Great, thanks! I've filed this ticket to request package unretirement:
https://pagure.io/releng/issue/11122

Comment 7 Fabio Valentini 2022-11-08 20:29:10 UTC
Package unretired, submitted, and built:
https://koji.fedoraproject.org/koji/buildinfo?buildID=2085881
https://bodhi.fedoraproject.org/updates/FEDORA-2022-83ed82c28e
Thanks everybody!


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