Bug 663384 - Review Request: scap-workbench - GUI tool that provides scanning, tailoring, editing and validation functionality for SCAP content
Summary: Review Request: scap-workbench - GUI tool that provides scanning, tailoring,...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Tomas Mraz
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2010-12-15 15:48 UTC by Maros Barabas
Modified: 2013-12-03 14:34 UTC (History)
6 users (show)

Fixed In Version: scap-workbench-0.2.3-1.fc15
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-08-16 09:00:14 UTC
Type: ---
Embargoed:
tmraz: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Maros Barabas 2010-12-15 15:48:42 UTC
Spec URL: https://fedorahosted.org/scap-workbench/attachment/wiki/WikiStart/scap-workbench.spec?format=raw

SRPM URL: https://fedorahosted.org/scap-workbench/attachment/wiki/WikiStart/scap-workbench-0.1.0-1.fc13.src.rpm?format=raw

Description: Scap Workbench is a GUI tool that provides scanning, tailoring, editing and validation functionality for SCAP content. It uses openscap library to access SCAP functionalities.

Comment 1 Radek Vokál 2011-01-07 14:50:04 UTC
The debuging to console should be probably turned-off.

Comment 2 Tomas Mraz 2011-01-10 12:34:34 UTC
Please add a desktop entry for the GUI application. According to the guidelines GUI applications should have an appropriate desktop entry.
https://fedoraproject.org/wiki/Packaging/Guidelines#Desktop_files

Comment 4 Tomas Mraz 2011-02-02 10:14:50 UTC
You should post a full new src.rpm for download - at least that is what's required by the review rules.

Comment 6 Tomas Mraz 2011-02-02 17:23:30 UTC
I suggest a better formatting of the Name in the desktop file - SCAP Workbench perhaps? But this is by no means a blocker.

I think the license is GPLv3+ as see the ...or any later version... clause in the header comments of the sources.

%{_sysconfdir}/scap-workbench directory is not owned by the package.

You use /usr/share instead of %{_datadir}.

Comment 7 Maros Barabas 2011-02-04 12:41:34 UTC
SRPM updated, URL is the same as in comment #5.

Comment 8 Tomas Mraz 2011-02-04 17:17:27 UTC
rpmlint -v scap-workbench-0.2.0-1.fc14.src.rpm scap-workbench-0.2.0-1.fc14.noarch.rpm
scap-workbench.src: I: checking
scap-workbench.noarch: I: checking
2 packages and 0 specfiles checked; 0 errors, 0 warnings.

Rpmlint is silent. I have reviewed the SPEC file and the package. I've also tried to run the utility and it seems that at least the application window appears. I did not test the functionality though.
The package now conforms to the Fedora packaging guidelines and is thus APPROVED.

Comment 9 Peter Vrabec 2011-02-08 16:49:03 UTC
New Package SCM Request
=======================
Package Name: scap-workbench
Short Description: GUI tool that provides scanning, tailoring, editing and validation functionality for SCAP content
Owners: mbarabas pvrabec 
Branches: f14 f15

Comment 10 Bill Nottingham 2011-02-08 20:04:32 UTC
f15 is not a valid branch - please resubmit with correct branches.

Comment 11 Peter Vrabec 2011-02-09 08:32:59 UTC
New Package SCM Request
=======================
Package Name: scap-workbench
Short Description: GUI tool that provides scanning, tailoring, editing and
validation functionality for SCAP content
Owners: mbarabas pvrabec 
Branches: f14

Comment 12 Jason Tibbitts 2011-02-10 13:58:39 UTC
Git done (by process-git-requests).

Comment 13 Martin Preisler 2013-12-03 13:24:43 UTC
Package Change Request
======================
Package Name: scap-workbench
New Branches: el6
Owners: mpreisle

Comment 14 Gwyn Ciesla 2013-12-03 14:34:36 UTC
Git done (by process-git-requests).


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