Bug 2214386

Summary: Review Request: rust-aes-siv - AES-SIV Misuse-Resistant Authenticated Encryption Cipher
Product: [Fedora] Fedora Reporter: Fabio Valentini <decathorpe>
Component: Package ReviewAssignee: Kalev Lember <klember>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: klember, package-review
Target Milestone: ---Flags: klember: fedora-review+
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2023-07-03 16:35:49 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 2214354, 2214355, 2214383    
Bug Blocks:    

Description Fabio Valentini 2023-06-12 19:53:33 UTC
Spec URL: https://decathorpe.fedorapeople.org/rust-aes-siv.spec
SRPM URL: https://decathorpe.fedorapeople.org/rust-aes-siv-0.7.0-1.fc38.src.rpm

Description:
Pure Rust implementation of the AES-SIV Misuse-Resistant Authenticated
Encryption Cipher (RFC 5297) with optional architecture-specific
hardware acceleration.

Fedora Account System Username: decathorpe

Comment 1 Fabio Valentini 2023-06-12 19:55:53 UTC
COPR build:
https://copr.fedorainfracloud.org/coprs/decathorpe/ntpd-rs/build/6067399/

Comment 2 Kalev Lember 2023-07-02 20:59:11 UTC
Fedora review rust-aes-siv-0.7.0-1.fc38.src.rpm 2023-07-02

$ rpmlint rust-aes-siv*
=================================================================================== rpmlint session starts ===================================================================================
rpmlint: 2.4.0
configuration:
    /usr/lib/python3.11/site-packages/rpmlint/configdefaults.toml
    /etc/xdg/rpmlint/fedora-legacy-licenses.toml
    /etc/xdg/rpmlint/fedora-spdx-licenses.toml
    /etc/xdg/rpmlint/fedora.toml
    /etc/xdg/rpmlint/scoring.toml
    /etc/xdg/rpmlint/users-groups.toml
    /etc/xdg/rpmlint/warn-on-functions.toml
checks: 31, packages: 9

rust-aes-siv+alloc-devel.noarch: W: no-documentation
rust-aes-siv+default-devel.noarch: W: no-documentation
rust-aes-siv+getrandom-devel.noarch: W: no-documentation
rust-aes-siv+heapless-devel.noarch: W: no-documentation
rust-aes-siv+pmac-devel.noarch: W: no-documentation
rust-aes-siv+std-devel.noarch: W: no-documentation
rust-aes-siv+stream-devel.noarch: W: no-documentation
==================================================== 9 packages and 0 specfiles checked; 0 errors, 7 warnings, 0 badness; has taken 0.4 s ====================================================

+ OK
! needs attention

+ rpmlint output looks good
+ The package is named according to Fedora packaging guidelines
+ The spec file name matches the base package name.
+ The package meets the Packaging Guidelines
+ The package is licensed with a Fedora approved license and meets the
  Licensing Guidelines.
+ The license field in the spec file matches the actual license
+ The license texts (Apache-2.0, MIT) are included in %license
+ Spec file is written in American English
+ Spec file is legible
+ Upstream sources match the sources in the srpm
  SHA512 (aes-siv-0.7.0.crate) = e8be452c062dfd83d9147c6f05799e462e8732ca6fd04438da9619b9726d823a253bbc39251f1b71e271dc28644c9fcd90b36a95a5392b6c410b66548e10da83
  SHA512 (Download/aes-siv-0.7.0.crate) = e8be452c062dfd83d9147c6f05799e462e8732ca6fd04438da9619b9726d823a253bbc39251f1b71e271dc28644c9fcd90b36a95a5392b6c410b66548e10da83
+ Package builds in koji
n/a ExcludeArch bugs filed
+ BuildRequires look sane
n/a locale handling
+ Package does not bundle copies of system libraries
n/a Package isn't relocatable
+ Package owns all the directories it creates
+ No duplicate files in %files (CHANGELOG.md, LICENSE-APACHE, LICENSE-MIT, README.md are listed twice but that's expected with the rust2rpm generator)
+ Permissions are properly set
+ Consistent use of macros
+ The package must contain code or permissible content
n/a Large documentation files should go in -doc subpackage
+ Files marked %doc should not affect the runtime of application
n/a Static libraries should be in -static
+ Development files should be in -devel
n/a -devel must require the fully versioned base
+ Packages should not contain libtool .la files
n/a Proper .desktop file handling
+ Doesn't own files or directories already owned by other packages
+ Filenames are valid UTF-8
+ Package does not depend on deprecated packages

APPROVED

Comment 3 Fabio Valentini 2023-07-03 15:58:54 UTC
Thanks for the review!

Comment 4 Fedora Admin user for bugzilla script actions 2023-07-03 16:01:22 UTC
The Pagure repository was created at https://src.fedoraproject.org/rpms/rust-aes-siv

Comment 5 Fabio Valentini 2023-07-03 16:35:49 UTC
Imported and built:
https://bodhi.fedoraproject.org/updates/FEDORA-2023-1170925ac4