Bug 1295230 - (hawaii-workspace) Review Request: hawaii-workspace - Hawaii workspace, applications and plugins
Review Request: hawaii-workspace - Hawaii workspace, applications and plugins
Status: CLOSED NEXTRELEASE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Rex Dieter
Fedora Extras Quality Assurance
:
Depends On:
Blocks: qt-reviews
  Show dependency treegraph
 
Reported: 2016-01-03 14:54 EST by Pier Luigi Fiorini
Modified: 2016-01-22 23:42 EST (History)
3 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2016-01-22 23:42:45 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
rdieter: fedora‑review+


Attachments (Terms of Use)

  None (edit)
Description Pier Luigi Fiorini 2016-01-03 14:54:04 EST
Spec URL: https://fedorapeople.org/~plfiorini/srpms/hawaii-unstable/hawaii-workspace.spec
SRPM URL: https://fedorapeople.org/~plfiorini/srpms/hawaii-unstable/hawaii-workspace-0.5.0-1.fc23.src.rpm
Description: Hawaii runtime components.
Fedora Account System Username: plfiorini
Comment 1 Upstream Release Monitoring 2016-01-06 04:16:18 EST
plfiorini's scratch build of hawaii-workspace-0.5.0-1.fc23.src.rpm for rawhide completed http://koji.fedoraproject.org/koji/taskinfo?taskID=12432693
Comment 2 Rex Dieter 2016-01-15 10:38:34 EST
naming: ok

sources: ok
8e29d8d69bf6ffe04177ad0907f395ff  hawaii-workspace-0.5.0.tar.xz

licensing: ok

files: ok, but

1.  the filelist is small, so SHOULD track files more closely instead of using globs everywhere, with something like:
%files
...
%{_bindir}/hawaii-polkit-agent
%{_bindir}/hawaii-screenshot
%{_sysconfdir}/xdg/hawaii-polkit-agent.desktop
%{_datadir}/applications/org.hawaiios.screenshot.desktop


2. MUST validate all application .desktop files, adding
BuildRequires: desktop-file-utils
...
%check
desktop-file-validate %{buildroot}%{_datadir}/applications/org.hawaiios.screenshot.desktop


3. SHOULD omit deprecated rpm tags
Group:


All in all, pretty simple and clean, only one mostly-cosmetic MUST blocker (2)... I'll leave it to you to add prior to doing any official builds.


APPROVED.
Comment 3 Pier Luigi Fiorini 2016-01-16 02:46:51 EST
> 1.  the filelist is small, so SHOULD track files more closely instead of using globs everywhere, with something like:

Done

> 2. MUST validate all application .desktop files, adding

Done

> 3. SHOULD omit deprecated rpm tags
> Group:

Done, removed the Group tag


Updated the spec and srpm

Spec URL: https://fedorapeople.org/~plfiorini/srpms/hawaii-unstable/take2/hawaii-workspace.spec
SRPM URL: https://fedorapeople.org/~plfiorini/srpms/hawaii-unstable/take2/hawaii-workspace-0.5.0-1.fc23.src.rpm
Comment 4 Upstream Release Monitoring 2016-01-16 02:56:58 EST
plfiorini's scratch build of hawaii-workspace-0.5.0-1.fc23.src.rpm for rawhide completed http://koji.fedoraproject.org/koji/taskinfo?taskID=12570615
Comment 5 Gwyn Ciesla 2016-01-16 18:09:56 EST
Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/hawaii-workspace

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