Bug 455152

Summary: Review Request: visual-4.beta26-1.fc9.i386.rpm - Visual Modelling Library
Product: [Fedora] Fedora Reporter: Brad Longo <brad.longo>
Component: Package ReviewAssignee: Nobody's working on this, feel free to take it <nobody>
Status: CLOSED DUPLICATE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: low    
Version: rawhideCC: fedora-package-review, notting, tomspur
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2008-07-13 19:51:49 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Brad Longo 2008-07-13 01:33:07 UTC
Spec URL: http://www.box.net/shared/5d56kmj40w
SRPM URL: http://www.box.net/shared/4ohvrw7pc0
Description: VPython is a visual modelling library for python which can be used to model physics problems.

Comment 1 Brad Longo 2008-07-13 02:19:22 UTC
I almost forgot.  This is my first package and I am seeking a sponsor.

Comment 2 Casey Dahlin 2008-07-13 16:09:52 UTC
1. You don't need to buildrequires gcc.
2. The Summary doesn't summarize much, and the description isn't very
descriptive. For the description, see if there's a paragraph in the README that
will work.
3. Your first changelog entry mentions some changes to the source. These should
be in a patch file.
4. Source should be specified as a URL to upstream

Comment 3 Casey Dahlin 2008-07-13 17:06:58 UTC
5. Use macros for FHS paths

Comment 4 Brad Longo 2008-07-13 18:46:53 UTC
The changes are made.  Rpm and specfile can be found with the same links still.

Comment 5 Casey Dahlin 2008-07-13 19:10:07 UTC
1. Delete the commented out code
2. --prefix=/usr/ should be --prefix=%{_prefix}
3. Break description up into multiple lines. No line should be longer than 80
characters.

Comment 6 Brad Longo 2008-07-13 19:53:02 UTC
The package has is being reworked so I have cancelled the review request.  There
is a stable verion I will work on to get it functional in Fedora.

Comment 7 Thomas Spura 2009-11-17 02:42:22 UTC

*** This bug has been marked as a duplicate of bug 537983 ***