Bug 425839 - Review Request: starplot-gliese3 - Stellar data set for use by the StarPlot tool
Summary: Review Request: starplot-gliese3 - Stellar data set for use by the StarPlot tool
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Parag AN(पराग)
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2007-12-16 14:51 UTC by Debarshi Ray
Modified: 2008-08-02 23:40 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2007-12-24 17:30:25 UTC
Type: ---
Embargoed:
panemade: fedora-review+
tcallawa: fedora-cvs+


Attachments (Terms of Use)

Description Debarshi Ray 2007-12-16 14:51:25 UTC
Spec URL: http://rishi.fedorapeople.org/starplot-gliese3.spec
SRPM URL: http://rishi.fedorapeople.org/starplot-gliese3-0.95-1.fc8.src.rpm


Description:

Stellar data set for use by the StarPlot tool from the Third Catalogue of
Nearby Stars (preliminary edition), Gliese and Jahreiss, 1991. The data set
was obtained from the archives of the Astronomical Data Center (ADC) at NASA
Goddard Space Flight Center.

Comment 1 Debarshi Ray 2007-12-16 16:51:09 UTC
Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=295802

Comment 2 Parag AN(पराग) 2007-12-17 04:45:40 UTC
rpmlint is silent.
source matches upstream.
c4f5435c0082bd8b9a7fb2a765f25414  gliese3-0.95.tar.gz
mock build is successful.

APPROVED.

Comment 3 Debarshi Ray 2007-12-17 07:17:43 UTC
New Package CVS Request
=======================
Package Name: starplot-gliese3
Short Description: Stellar data set for use by the StarPlot tool
Owners: rishi
Branches: F-7 F-8
InitialCC:
Cvsextras Commits: no

Comment 4 Mamoru TASAKA 2007-12-17 08:00:01 UTC
Like bug 425853, this srpm must also be fixed.

Comment 5 Parag AN(पराग) 2007-12-17 08:44:39 UTC
Okay here scriptlets are having problem.
rpm -ihv starplot-gliese3-0.95-1.fc8.noarch.rpm --excludedocs
Preparing...                ########################################### [100%]
   1:starplot-gliese3       ########################################### [100%]
*** Does not contain starconvert spec file.
error: %post(starplot-gliese3-0.95-1.fc8.noarch) scriptlet failed, exit status 1

Debarshi,
  Please follow https://bugzilla.redhat.com/process_bug.cgi#c7 and update package.

Comment 6 Parag AN(पराग) 2007-12-17 08:46:13 UTC
Please follow https://bugzilla.redhat.com/show_bug.cgi?id=425853#c7 and update
package.

Comment 7 Debarshi Ray 2007-12-17 09:08:18 UTC
We can not distribute the yale5.stars file as part of the package, since it
would be a violation of the license. This was discussed on
fedora-devel-list
(https://www.redhat.com/archives/fedora-devel-list/2007-November/msg00742.html)
and it was decided that they should be generated during %post and mentioned as
%ghost.

Comment 8 Debarshi Ray 2007-12-17 09:17:11 UTC
I will use %{_datadir}/%{name} instead of %{_docdir}/%{name}-%{version} to
generate the *.stars file. Will that be acceptable?

Comment 9 Parag AN(पराग) 2007-12-17 09:26:19 UTC
before you submit any change please cross check with following command
rpm -ihv starplot-gliese3-0.95-1.fc8.noarch.rpm --excludedocs

Comment 10 Tom "spot" Callaway 2007-12-17 16:15:55 UTC
cvs is done, but please fix the scriplet issues before committing.

Comment 11 Parag AN(पराग) 2007-12-19 15:16:55 UTC
Proposing this new fix,
Spec URL:  http://paragn.fedorapeople.org/starplot-gliese3.spec
SRPM URL:  http://paragn.fedorapeople.org/starplot-gliese3-0.95-2.fc8.src.rpm



Comment 12 Mamoru TASAKA 2007-12-19 15:32:27 UTC
IMO it is better that all files (.spec file, orig-data, etc ) from
various starplot-related packages are installed under %_datadir/starplot.

For this package, IMO it is better that .spec file, orig-data, etc...
are installed under %_datadir/starplot/gliese3, and for yale5, it is
%_datadir/starplot/yale5.

Comment 14 Mamoru TASAKA 2007-12-19 15:58:22 UTC
Seems good, however please replace gliese3 with %dataset appropriately
for consistency.

Comment 15 Parag AN(पराग) 2007-12-19 16:09:31 UTC
done :) with same release -3 links.
Thanks to you.


Comment 16 Parag AN(पराग) 2007-12-22 04:46:35 UTC
Debarshi,
Do you see any problem with updated package links in comment #13?

Comment 17 Debarshi Ray 2007-12-22 21:48:16 UTC
Looks fine to me.

Spec: http://rishi.fedorapeople.org/starplot-gliese3.spec
SRPM: http://rishi.fedorapeople.org/starplot-gliese3-0.95-1.fc8.src.rpm
Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=307172

I have merged Parag's changes into the above Spec/SRPM pair.

Comment 18 Debarshi Ray 2007-12-22 21:49:38 UTC
Sorry, make it
SRPM: http://rishi.fedorapeople.org/starplot-gliese3-0.95-2.fc8.src.rpm

Comment 19 Parag AN(पराग) 2007-12-24 06:09:01 UTC
APPROVED above SRPM.

You can now import above SRPM.


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