Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 1933794 - Review Request: secvarctl - A secure variable management tool for POWER's Secure Boot
Summary: Review Request: secvarctl - A secure variable management tool for POWER's Sec...
Keywords:
Status: ASSIGNED
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Dan Horák
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: PPCTracker
TreeView+ depends on / blocked
 
Reported: 2021-03-01 17:57 UTC by Nick Child
Modified: 2021-06-18 07:18 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed:
Type: ---
Embargoed:
dan: fedora-review+

Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
IBM Linux Technology Center 191742 0 None None None 2021-03-02 12:20:31 UTC

Description Nick Child 2021-03-01 17:57:40 UTC 
Spec URL: https://github.com/open-power/secvarctl/releases/download/v0.1/secvarctl.spec
SRPM URL: https://github.com/open-power/secvarctl/releases/download/v0.1/secvarctl-0.1-1.fc33.src.rpm
Description: secvarctl is a collection of sub-commands for reading, writing and updating secure variables for POWER's Secure Boot.
The sub-commands are:
	-read , prints info on secure variables
	-write , updates secure variable with new signed authenticated file
	-validate , validates format of given file
	-verify , determines if new variable updates are correctly signed/formatted
	-generate , create relevant files for secure variable management
Fedora Account System Username: nnac123

Koji Build: https://koji.fedoraproject.org/koji/taskinfo?taskID=62891320
Comment 1 Dan Horák 2021-03-10 12:37:54 UTC 
formal review is here, see the notes explaining OK* and BAD statuses below:

OK	source files match upstream:
	    3ca081c124e9d0ffb6bad4c249adb228b4b5cb56  v0.1.tar.gz
OK	package meets naming and versioning guidelines.
OK	specfile is properly named, is cleanly written and uses macros consistently.
OK	dist tag is present.
OK	license field matches the actual license.
OK	license is open source-compatible (ASL 2.0). License text included in package.
OK	latest version is being packaged.
OK	BuildRequires are proper.
OK	compiler flags are appropriate.
OK	package builds in mock (Rawhide/ppc64le).
OK	debuginfo package looks complete.
OK	rpmlint is silent.
OK	final provides and requires look sane.
N/A	%check is present and all tests pass.
OK	no shared libraries are added to the regular linker search paths.
OK	owns the directories it creates.
OK	doesn't own any directories it shouldn't.
OK	no duplicates in %files.
OK	file permissions are appropriate.
OK	no scriptlets present.
OK	code, not content.
OK	documentation is small, so no -docs subpackage is necessary.
OK	%docs are not necessary for the proper functioning of the package.
OK	no headers.
OK	no pkgconfig files.
OK	no libtool .la droppings.
OK	not a GUI app.

The package is APPROVED.
Comment 2 Tomas Hrcka 2021-03-11 22:37:32 UTC 
(fedscm-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/secvarctl
Note You need to log in before you can comment on or make changes to this bug.