Fedora Merge Review: newt
Initial Owner: email@example.com
First off, does newt have an upstream? Or is Red Hat the upstream? If there
is an upstream, a URL tag would be good if possible, and either a full URL in
the Source tag, or some instruction on making the tarball from CVS or
whatever is needed.
Some rpmlint complaints:
W: newt summary-ended-with-dot A development library for text mode user
W: newt-devel summary-ended-with-dot Newt windowing toolkit development files.
E: newt tag-not-utf8 %changelog
E: newt non-utf8-spec-file newt.spec
This is due to Trond's name; in the changelog; a quick pass through iconv
W: newt no-url-tag
Depends on whether there's an upstream to link to.
E: newt configure-without-libdir-spec
rpmlint is confused by the commented out "./configure" line.
W: newt macro-in-%changelog release
W: newt macro-in-%changelog version
E: newt script-without-shebang /usr/lib64/python2.5/site-packages/snack.py
This is executable; if the user is supposed to run it, it should have a
shebang line. If not, then it shouldn't be executable. It has no main, so
I'm pretty sure it's not supposed to be run.
After that, the only real issue is the static library. This is obviously
justified as the installer needs it, so all that's needed is a bit of
justification in a comment and a -static subpackage.
I'll attach a patch which fixes up the summaries, the non-utf8 bits, and the
macros in %changelog. It also splits the static library off into a -static
subpackage and includes some justification. (Obviously the installer will
need to be fixed to pull in the -static package.)
After this patch, all that remains are the questions of the where upstream is
and where the source comes from.
X can't compare source with upstream. (Perhaps this is the upstream.)
* package meets naming and versioning guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* dist tag is present.
* build root is correct.
* license field matches the actual license.
* license is open source-compatible.
* license text included in package.
? latest version is being packaged.
* BuildRequires are proper.
* compiler flags are appropriate.
* %clean is present.
* package builds in mock (development, x86_64).
* package installs properly
* debuginfo package looks complete.
X rpmlint has valid complaints
* final provides and requires are sane:
snack = 0.52.5-1.fc7
newt = 0.52.5-1.fc7
python(abi) = 2.5
newt-devel = 0.52.5-1.fc7
newt = 0.52.5
* %check is not present; no test suite upstream
* shared libraries present; ldconfig are added to the regular linker search paths.
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* find-lang used appropriately.
* scriptlets are OK (ldconfig)
* code, not content.
* documentation is small, so no -docs subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.
* headers are in the -devel subpackage.
* no pkgconfig files.
X static libraries present, not in -static and no justification.
* no libtool .la droppings.
Created attachment 148984 [details]
Suggested specfile patch fixing several issues.
Thanks for the review. Red Hat is the upstream, CVS is at elvis.redhat.com, you
can checkout sources with
cvs -z3 -d :pserver:firstname.lastname@example.org:/usr/local/CVS co newt
Could you include info in the specfile which details how to create the tarball
you're using? See http://fedoraproject.org/wiki/Packaging/SourceURL#RevisionControl
I'm not sure that just pulling the CVS HEAD is the best idea, because you lose
repeatability as new code is committed. Usually you want to pull a tag.
Ok, should be fixed in newt-0.52.6-1.fc7.
Created attachment 157617 [details]
Patch to split off newt-python relative to current rawhide
i wanted to suggest creating simple project page for newt, which would include
source tarball downloads.
there are some projects that depends on newt (for example, partimage), thus
such a thing would be beneficial for users of these packages.
some distributions are doing some repackaging themselves (debian, for example),
but that's far from optimal.
if this would be more appropriate as a new report, i'll gladly file one.
I think the plan is to move projects from elvis CVS to hosted.fedoraproject.org,
but I'm not sure when it will happen.
Source and URL tag updated in newt-0.52.9-1.fc9.
Upstream tarballs are at https://fedorahosted.org/releases/n/e/newt/.
Wow, somehow this got set to the modified state which causes it to drop completely out of every report I run to indicate the tickets I should be working on. I have no idea why anyone would ever use that state, because it sure screws things up. Let me take another look at things.
Any progress? What's blocking this review?
Well, damn, somehow I keep dropping this. I'll look at it today.
Looks like "today" turned into "tomorrow", but here we go.
I checked out and built the latest rawhide branch of newt. rpmlint is down to:
newt.x86_64: W: shared-lib-calls-exit /usr/lib64/libnewt.so.0.52.10
which is a bad idea and arguably a bug in the library, but not a review blocker.
newt-static.x86_64: W: no-documentation
which is fine.
So rpmlint is fine now, and upon inspection the other issues I had are fixed as well. Really the only thing I see currently is that the license on the code is rather unclear. There's nothing in the tarball that I can see which specifies which version of the LGPL is in use, which would, according to clause 13 of the included COPYING file, indicate that we are free to choose any version (i.e. LGPLv2+). Only the included spec file indicates a specific version, and I'm not absolutely certain whether or not that's a sufficient statement of intent from a legal standpoint.
Still, since I believe you folks are the upstream and the specfile in the tarball does say LGPLv2 so I'm going to say that things are OK, but I do strongly suggest that you clarify the licensing, preferrably by including the proper license blocks at the start of the source files as recommended by the COPYING file, or at minimum at least say "version 2.1 of the LGPL only" somewhere in the documentation or code.
APPROVED, and closing.