Bug 2327778

rust2rpm.toml:
[package]
extra-sources = [
    { "number" = 1, "file" = "LICENSE", "comments" = [ "LICENSE is copied from azure-cvm-tooling github repo"] },
]
 
[scripts]
prep.pre = [ "cp -f %{SOURCE1} ." ]

# add to %files:
# %license %{crate_instdir}/LICENSE

Spec URL: https://download.copr.fedorainfracloud.org/results/uril/trustee-attester/fedora-rawhide-x86_64/08334635-rust-az-cvm-vtpm/rust-az-cvm-vtpm.spec
SRPM URL: https://download.copr.fedorainfracloud.org/results/uril/trustee-attester/fedora-rawhide-x86_64/08334635-rust-az-cvm-vtpm/rust-az-cvm-vtpm-0.7.1-1.fc42.src.rpm

rust2rpm.toml as in comment 1

# spec-file generated with:
rust2rpm az-cvm-vtpm 0.7.1 -I

Please use a URL for Source1 to make it clear that you're not adding some random "LICENSE" file, but the actual one from upstream.
If this has not yet happened, please file a ticket with upstream to include the license texts in all published crates.

Copr build:
https://copr.fedorainfracloud.org/coprs/build/8341290
(failed)

Build log:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2327778-rust-az-cvm-vtpm/fedora-rawhide-x86_64/08341290-rust-az-cvm-vtpm/builder-live.log.gz

Please make sure the package builds successfully at least for Fedora Rawhide.

- If the build failed for unrelated reasons (e.g. temporary network
  unavailability), please ignore it.
- If the build failed because of missing BuildRequires, please make sure they
  are listed in the "Depends On" field


---
This comment was created by the fedora-review-service
https://github.com/FrostyX/fedora-review-service

If you want to trigger a new Copr build, add a comment containing new
Spec and SRPM URLs or [fedora-review-service-build] string.

Thanks Fabio

(In reply to Fabio Valentini from comment #3)
> Please use a URL for Source1 to make it clear that you're not adding some
> random "LICENSE" file, but the actual one from upstream.

Added a URL for Source1.
It points to the commit of the crate, in case it changes in the main branch.

> If this has not yet happened, please file a ticket with upstream to include
> the license texts in all published crates.

I've sent a patch

Spec URL: https://download.copr.fedorainfracloud.org/results/uril/trustee-attester/fedora-rawhide-x86_64/08341698-rust-az-cvm-vtpm/rust-az-cvm-vtpm.spec
SRPM URL: https://download.copr.fedorainfracloud.org/results/uril/trustee-attester/fedora-rawhide-x86_64/08341698-rust-az-cvm-vtpm/rust-az-cvm-vtpm-0.7.1-1.fc42.src.rpm

Created attachment 2061226 [details]
The .spec file difference from Copr build 8341290 to 8341736

Copr build:
https://copr.fedorainfracloud.org/coprs/build/8341736
(failed)

Build log:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2327778-rust-az-cvm-vtpm/fedora-rawhide-x86_64/08341736-rust-az-cvm-vtpm/builder-live.log.gz

Please make sure the package builds successfully at least for Fedora Rawhide.

- If the build failed for unrelated reasons (e.g. temporary network
  unavailability), please ignore it.
- If the build failed because of missing BuildRequires, please make sure they
  are listed in the "Depends On" field


---
This comment was created by the fedora-review-service
https://github.com/FrostyX/fedora-review-service

If you want to trigger a new Copr build, add a comment containing new
Spec and SRPM URLs or [fedora-review-service-build] string.

Looks good to me now, thank you!

Can you add a link to the PR you filed as a comment to the additional LICENSE Source file?
https://github.com/kinvolk/azure-cvm-tooling/pull/64

===

Package was generated with rust2rpm, simplifying the review.

✅❌❓🫤

✅ package contains only permissible content
✅ 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 and is acceptable for Fedora
🫤 license file is included with %license in %files (temporarily manually included from upstream git)
✅ package complies with Rust Packaging Guidelines

Package APPROVED.

===

Recommended post-import rust-sig tasks:

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

- add @rust-sig with "commit" access as package co-maintainer
  (should happen automatically)

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

- track package in koschei for all built branches
  (should happen automatically once rust-sig is co-maintainer)

Thank you Fabio.
I'll add a link to the PR in the comments above Source1

The Pagure repository was created at https://src.fedoraproject.org/rpms/rust-az-cvm-vtpm

Spec URL: https://download.copr.fedorainfracloud.org/results/uril/trustee-attester/fedora-rawhide-x86_64/08581458-rust-az-cvm-vtpm/rust-az-cvm-vtpm.spec
SRPM URL: https://download.copr.fedorainfracloud.org/results/uril/trustee-attester/fedora-rawhide-x86_64/08581458-rust-az-cvm-vtpm/rust-az-cvm-vtpm-0.7.1-1.fc42.src.rpm

- Added a link to the PR in the comments above Source1

Spec URL: https://download.copr.fedorainfracloud.org/results/uril/trustee-attester/fedora-rawhide-x86_64/08584527-rust-az-cvm-vtpm/rust-az-cvm-vtpm.spec
SRPM URL: https://download.copr.fedorainfracloud.org/results/uril/trustee-attester/fedora-rawhide-x86_64/08584527-rust-az-cvm-vtpm/rust-az-cvm-vtpm-0.7.1-1.fc42.src.rpm

- supported_arches x86_64

az-cvm-vtpm supports Intel (TDX) and AMD (SEV-SNP)

FEDORA-2025-7ba288f15e (rust-az-cvm-vtpm-0.7.1-3.fc42) has been submitted as an update to Fedora 42.
https://bodhi.fedoraproject.org/updates/FEDORA-2025-7ba288f15e

FEDORA-2025-7ba288f15e (rust-az-cvm-vtpm-0.7.1-3.fc42) has been pushed to the Fedora 42 stable repository.
If problem still persists, please make note of it in this bug report.