Bug 972590 - Review Request: sugar-starchart - Display a map of the sky showing the position of the visible stars
Review Request: sugar-starchart - Display a map of the sky showing the positi...
Status: CLOSED ERRATA
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Parag AN(पराग)
Fedora Extras Quality Assurance
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2013-06-10 03:40 EDT by Danishka Navin
Modified: 2013-08-14 23:04 EDT (History)
3 users (show)

See Also:
Fixed In Version: sugar-starchart-15-3.fc19
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2013-08-14 22:50:55 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
panemade: fedora‑review+
limburgher: fedora‑cvs+


Attachments (Terms of Use)

  None (edit)
Description Danishka Navin 2013-06-10 03:40:40 EDT
Spec URL: http://snavin.fedorapeople.org/packages/sugar-starchart/sugar-starchart.spec

SRPM URL: http://snavin.fedorapeople.org/packages/sugar-starchart/sugar-starchart-15-1.fc18.src.rpm

Description: 

This activity will display a map of the sky showing the position of the visible stars, some of the larger and brighter deep-sky objects (DSOs), the "classical" planets, the sun and the moon. 

Fedora Account System Username: snavin
Comment 1 Parag AN(पराग) 2013-06-10 04:37:39 EDT
Review:-

+ Mock build successful for f20

- rpmlint on generated rpms gave
sugar-starchart.noarch: E: script-without-shebang /usr/share/sugar/activities/StarChart.activity/activity/activity.info
sugar-starchart.noarch: E: script-without-shebang /usr/share/sugar/activities/StarChart.activity/activity/StarChart.svg
sugar-starchart.noarch: E: wrong-script-end-of-line-encoding /usr/share/sugar/activities/StarChart.activity/activity/StarChart.svg
2 packages and 0 specfiles checked; 3 errors, 0 warnings.

+ Source verified with upstream as (sha256sum)
srpm tarball     : c5e1e4275303b8d60f335a5d73804535daebcb3c312e89546436ec478192ec94
upstream tarball : c5e1e4275303b8d60f335a5d73804535daebcb3c312e89546436ec478192ec94

+ License is valid and its text is available

+ rest looks as per packaging guidelines


Suggestions
1) As suggested in previous other reviews change --prefix=%{buildroot}/%{_prefix} to --prefix=%{buildroot}%{_prefix}


2) rpmlint errors can be fixed by adding following line in %prep
chmod 644 activity/*
Comment 3 Parag AN(पराग) 2013-06-10 05:03:18 EDT
Looks good.

APPROVED.
Comment 4 Danishka Navin 2013-06-10 05:05:39 EDT
New Package SCM Request
=======================
Package Name: sugar-starchart
Short Description: Display a map of the sky showing the position of the visible stars
Owners: snavin
Branches: f17 f18 f19
InitialCC:
Comment 5 Gwyn Ciesla 2013-06-10 08:42:24 EDT
Git done (by process-git-requests).
Comment 6 Fedora Update System 2013-08-04 23:54:09 EDT
sugar-starchart-15-3.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/sugar-starchart-15-3.fc18
Comment 7 Fedora Update System 2013-08-04 23:54:20 EDT
sugar-starchart-15-3.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/sugar-starchart-15-3.fc19
Comment 8 Fedora Update System 2013-08-05 20:19:44 EDT
sugar-starchart-15-3.fc18 has been pushed to the Fedora 18 testing repository.
Comment 9 Fedora Update System 2013-08-14 22:50:55 EDT
sugar-starchart-15-3.fc18 has been pushed to the Fedora 18 stable repository.
Comment 10 Fedora Update System 2013-08-14 23:04:25 EDT
sugar-starchart-15-3.fc19 has been pushed to the Fedora 19 stable repository.

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