Bug 868713

Summary: Review Request: repsnapper - RepRap control software
Product: [Fedora] Fedora Reporter: Alon Levy <alevy>
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: unspecified Docs Contact:
Priority: unspecified    
Version: 18CC: dblechte, mhroncok, notting, package-review, volker27
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-01-28 06:39:39 EST Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
Bug Depends On:    
Bug Blocks: 201449    
Attachments:
Description Flags
spec file none

Description Alon Levy 2012-10-21 18:22:08 EDT
Created attachment 631128 [details]
spec file

Description of problem:

spec file: https://gitorious.org/fedora-packages-under-review/fedora-packages-under-review/blobs/r1/repsnapper/repsnapper.spec

source rpm: https://gitorious.org/fedora-packages-under-review/fedora-packages-under-review/blobs/r1/repsnapper/repsnapper-2.1.0b02-1.fc18.src.rpm


rpmlint results:

repsnapper.spec:12: W: macro-in-comment %{version}
repsnapper.spec:13: W: macro-in-comment %setup
repsnapper.spec: W: invalid-url Source0: repsnapper-2.1.0b02.tar.gz
0 packages and 1 specfiles checked; 0 errors, 3 warnings.

The first two seem harmless to me. I could fix it if I told %setup the name of the tarball (which is just %{version} if you use the download url in my helper script, downloading from github, and this will also solve the third warning.

Alon
Comment 1 Volker Fröhlich 2012-10-22 15:26:51 EDT
Please use this form for future requests: https://bugzilla.redhat.com/bugzilla/enter_bug.cgi?product=Fedora&format=fedora-review
Comment 2 Volker Fröhlich 2012-10-22 15:34:22 EDT
Drop (most likely) all version requirements, compare http://fedoraproject.org/wiki/Packaging:Guidelines#Requires

License should be stated as GPLv2.

Source1 should use the name macro.

Maximum allowed linelength is 80 characters.

Locales are not handled properly: http://fedoraproject.org/wiki/Packaging:Guidelines#Handling_Locale_Files

Drop the name from "Summary".

Please offer your files in a way, where the URLs are directly accessable.
Comment 3 Alon Levy 2012-10-23 10:33:37 EDT
Thanks for the review!

Updated source package and spec:

http://people.freedesktop.org/~alon/repsnapper.spec
http://people.freedesktop.org/~alon/repsnapper-2.1.0b02-2.fc18.src.rpm

URLs should be directly accessible.

Alon
Comment 4 Volker Fröhlich 2012-10-23 13:56:12 EDT
Build fails for me with mock on F17 and Rawhide (haven't tried F18).
Comment 5 Alon Levy 2012-10-24 07:44:46 EDT
Added missing intltools & libtools dependencies

new srpm: http://people.freedesktop.org/~alon/repsnapper-2.1.0b02-3.fc18.src.rpm
new spec: http://people.freedesktop.org/~alon/repsnapper.spec
Comment 6 Volker Fröhlich 2012-10-24 17:22:44 EDT
Please make the build verbose!

There seem to be 6 bundled libraries in the "libraries" directory. Please use system libraries: http://fedoraproject.org/wiki/Packaging:Guidelines#Duplication_of_system_libraries

vmmlib seems to exist in Fedora and so does clipper (under the name "polyclipping").

amf's license is LGPLv3+, which actually does not allow repsnapper to be GPLv2. See https://fedoraproject.org/wiki/Licensing:Main?rd=Licensing#GPL_Compatibility_Matrix

Please report that upstream!

lmfit -- http://joachimwuttke.de/src/
libreprap -- http://reprap.org/wiki/Libreprap
poly2tri -- http://code.google.com/p/poly2tri/
Comment 7 Miro Hrončok 2013-01-27 13:40:13 EST
Alon are you still interested in this?
Comment 8 Alon Levy 2013-01-28 03:38:47 EST
I'm not currently working on it, although I'd like this to be part of fedora.
Comment 9 Miro Hrončok 2013-01-28 06:39:39 EST
OK, I'm going to do it. Will mark this bug as a duplicate, when ready.
Comment 10 Miro Hrončok 2013-02-04 14:07:17 EST

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