Bug 1983877 - Review Request: rust-uefi-run - Run UEFI applications
Summary: Review Request: rust-uefi-run - Run UEFI applications
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Fabio Valentini
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 1983876
Blocks:
TreeView+ depends on / blocked
 
Reported: 2021-07-20 04:45 UTC by Davide Cavalca
Modified: 2022-08-13 02:51 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2022-08-01 22:46:23 UTC
Type: ---
Embargoed:
decathorpe: fedora-review+


Attachments (Terms of Use)

Description Davide Cavalca 2021-07-20 04:45:15 UTC
Spec URL: https://dcavalca.fedorapeople.org/review/rust-uefi-run/rust-uefi-run.spec
SRPM URL: https://dcavalca.fedorapeople.org/review/rust-uefi-run/rust-uefi-run-0.3.3-1.fc35.src.rpm

Description:
This helper application takes an EFI executable, builds a FAT filesystem around
it, adds a startup script and runs qemu to run the executable.

It does not require root permissions since it uses the fatfs crate to build the
filesystem image directly without involving mkfs, mount, etc.

Fedora Account System Username: dcavalca

Comment 1 Robert-André Mauchin 🐧 2022-01-23 20:17:13 UTC
 - License ok
 - Latest version packaged
 - Builds in mock
 - No rpmlint errors
 - Checks pass
 - Conforms to Packaging Guidelines

Package approved

Please regenerate the .spec file with rust2rpm 20 before building this package for Fedora.

Comment 2 Fabio Valentini 2022-01-23 20:29:09 UTC
Note that this package builds a binary, so you need to make sure the License tag for the "-n %{crate}" subpackage reflects the licenses of all statically linked components. Right now this is not done at all.

Comment 3 Robert-André Mauchin 🐧 2022-01-23 21:26:27 UTC
Ha yeah I forgot about that. Weren't there a crate for that?

Comment 4 Fabio Valentini 2022-01-23 22:36:11 UTC
There is cargo-license, but it obviously does not take into account any of our downstream patches or modifications.

I usually use this bash script inside the mock chroot after running a build with "--without check":

for i in $(rpm -qa | grep "rust-.*-devel"); do
   rpm -q $i --qf "%{LICENSE}\n";
done | sort | uniq

It gives you a list of all licenses of installed rust-foo-devel packages in the buildroot. From those you can construct the "effective" license of the subpackage that contains the binary.

Alternatively, if you want a list of all Rust crates and their licenses in detail, you can install dnf-utils into the mock chroot and run something like:

dnf repoquery --cacheonly "rust-*-devel" --installed --qf "# %{LICENSE}: %{source_name} %{version}"

Comment 5 Davide Cavalca 2022-07-11 04:05:24 UTC
Effective license calculation:
(ASL 2.0 or MIT) and MIT and (MIT OR Apache-2.0) and (MIT or ASL 2.0) and  (Unlicense or MIT) = ASL 2.0 and MIT and Unlicense

Comment 6 Davide Cavalca 2022-07-11 04:08:40 UTC
Spec URL: https://dcavalca.fedorapeople.org/review/rust-uefi-run/rust-uefi-run.spec
SRPM URL: https://dcavalca.fedorapeople.org/review/rust-uefi-run/rust-uefi-run-0.5.0-1.fc37.src.rpm

Changelog:
- re-run rust2rpm
- update to 0.5.0
- add effective license for binary subpackage

Comment 7 Fabio Valentini 2022-07-29 09:49:51 UTC
Package was generated with rust2rpm, simplifying the review.

- package builds and installs without errors on rawhide
- test suite is run and all unit tests pass
- latest version of the crate is packaged
- license matches upstream specification (MIT) and is acceptable for Fedora-
- license for binary subpackage reflects statically linked dependencies
- license file is included with %license in %files
- package complies with Rust Packaging Guidelines

Package APPROVED.

===

Recommended post-import rust-sig tasks:

- add @rust-sig with "commit" access as package co-maintainer

- set bugzilla assignee overrides to @rust-sig (optional)

- set up package on release-monitoring.org:
  project: $crate
  homepage: https://crates.io/crates/$crate
  backend: crates.io
  version scheme: semantic
  version filter: alpha;beta;rc;pre
  distro: Fedora
  Package: rust-$crate

- track package in koschei for all built branches

===

Please also add the list of licenses that played a role in your license determination for the binary subpackage as a comment above the License tag.

Comment 8 Davide Cavalca 2022-08-01 16:10:42 UTC
Thanks!

$ fedpkg request-repo rust-uefi-run 1983877
https://pagure.io/releng/fedora-scm-requests/issue/46104
$ fedpkg request-branch --all-releases --repo rust-uefi-run
https://pagure.io/releng/fedora-scm-requests/issue/46105
https://pagure.io/releng/fedora-scm-requests/issue/46106

Comment 9 Gwyn Ciesla 2022-08-01 17:03:37 UTC
(fedscm-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/rust-uefi-run

Comment 10 Fedora Update System 2022-08-01 22:44:57 UTC
FEDORA-2022-7db1f694a5 has been submitted as an update to Fedora 37. https://bodhi.fedoraproject.org/updates/FEDORA-2022-7db1f694a5

Comment 11 Fedora Update System 2022-08-01 22:46:23 UTC
FEDORA-2022-7db1f694a5 has been pushed to the Fedora 37 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 12 Fedora Update System 2022-08-03 20:40:38 UTC
FEDORA-2022-3c2683a334 has been submitted as an update to Fedora 36. https://bodhi.fedoraproject.org/updates/FEDORA-2022-3c2683a334

Comment 13 Fedora Update System 2022-08-04 01:00:37 UTC
FEDORA-2022-b50af6aa3c has been submitted as an update to Fedora 35. https://bodhi.fedoraproject.org/updates/FEDORA-2022-b50af6aa3c

Comment 14 Fedora Update System 2022-08-04 02:42:32 UTC
FEDORA-2022-3c2683a334 has been pushed to the Fedora 36 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf install --enablerepo=updates-testing --refresh --advisory=FEDORA-2022-3c2683a334 \*`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2022-3c2683a334

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Comment 15 Fedora Update System 2022-08-05 01:24:46 UTC
FEDORA-2022-b50af6aa3c has been pushed to the Fedora 35 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf install --enablerepo=updates-testing --refresh --advisory=FEDORA-2022-b50af6aa3c \*`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2022-b50af6aa3c

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Comment 16 Fedora Update System 2022-08-12 01:38:03 UTC
FEDORA-2022-3c2683a334 has been pushed to the Fedora 36 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 17 Fedora Update System 2022-08-13 02:51:28 UTC
FEDORA-2022-b50af6aa3c has been pushed to the Fedora 35 stable repository.
If problem still persists, please make note of it in this bug report.


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