Bug 868713 - Review Request: repsnapper - RepRap control software
Review Request: repsnapper - RepRap control software
Status: CLOSED DUPLICATE of bug 907585
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
Unspecified Unspecified
unspecified Severity unspecified
: ---
: ---
Assigned To: Nobody's working on this, feel free to take it
Fedora Extras Quality Assurance
Depends On:
  Show dependency treegraph
Reported: 2012-10-21 18:22 EDT by Alon Levy
Modified: 2014-08-04 18:09 EDT (History)
5 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2013-01-28 06:39:39 EST
Type: Bug
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)
spec file (1.66 KB, text/x-rpm-spec)
2012-10-21 18:22 EDT, Alon Levy
no flags Details

  None (edit)
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.

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:


URLs should be directly accessible.

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 ***

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