Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: wings - A polygon mesh modeler https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=188208 ------- Additional Comments From tibbs@xxxxxxxxxxx 2006-04-26 20:08 EST ------- This one builds fine; thanks. Issues: rpmlint doesn't like the fact that you didn't include a changelog entry. I cannot fetch Source1; it looks like they've moved the file to sourceforge. I can get it from http://dl.sourceforge.net/wings/wings3d_manual1.6.1.pdf which I'll take to be the upstream. Most of the final size of the package is the manual, which I think qualifies as "large" and so should be split into a -docs package. Since it's not built from the main tarball, you could even build it from a separate srpm if you wanted. I don't know if you missed this comment earlier of if you decided against it, but it might be worth it to note that this package includes the POV exporter plugin since it's not part of the upstream source. There are tiny README files stuck in with the compiled code. They're kind of pointless; /usr/lib64/wings/plugins/autouv/README says "This directory contains Auto-UV." I guess they don't hurt anything but I'm not sure there's much point in packaging them. They certainly don't make sense as %doc. Review: * package meets naming and packaging guidelines. * specfile is properly named, is cleanly written and uses macros consistently. O license field matches the actual license. (The license is the same as erlang-esdl's, which is open source but doesn't strictly correspond to any particular license tag.) * license is open source-compatible and is included in the package. O source files match upstream (once I fix Source1): 11a55f0eb81261a84a29d05dd4afa999 wings-0.98.32b.tar.bz2 11a55f0eb81261a84a29d05dd4afa999 wings-0.98.32b.tar.bz2-srpm c01d19b4be79b647b8605dd32753da3a wings3d_manual1.6.1.pdf c01d19b4be79b647b8605dd32753da3a wings3d_manual1.6.1.pdf-srpm 791ec96aefb43ca63800324a115e99c6 wingspov-0.98.28_v1.tgz 791ec96aefb43ca63800324a115e99c6 wingspov-0.98.28_v1.tgz-srpm * BuildRequires are proper. * package builds in mock (development, i386 and x86_64). X rpmlint complains about the changelog entry * final provides and requires are sane. * shared libraries are present but not in standard locations so there's no need to call ldconfig. * package is not relocatable. * owns the directories it creates. * doesn't own any directories it shouldn't. * no duplicates in %files. * file permissions are appropriate. * %clean is present. O %check is not present (no test suite upstream) * code, not content. X documentation is large; a -docs subpackage is necessary. * %docs are not necessary for the proper functioning of the package. * no headers. * no pkgconfig files. * no libtool .la droppings. * a GUI app. desktop file is included, looks OK and is properly installed with desktop-file-install. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug, or are watching the QA contact.