Red Hat Bugzilla – Bug 1295710
Review Request: wml - WML is a free and extensible Webdesigner's off-line HTML generation toolkit for Unix
Last modified: 2016-01-19 06:11:54 EST
Spec URL: http://people.redhat.com/nmavrogi/fedora/wml/wml.spec
SRPM URL: http://people.redhat.com/nmavrogi/fedora/wml/wml-2.4.1-1.fc24.src.rpm
Description: WML is a free and extensible Webdesigner's off-line HTML generation toolkit for Unix, distributed under the GNU General Public License (GPL v2).
While not trivial and idiot proof WML provides most of the core
features real hackers always wanted for HTML generation. A typical
use case of WML today is the generation of HTML-based static pages
for archivable documentation.
Fedora Account System Username: nmav
The Perl dependencies are wrong. Perl language structures code into Perl modules and the dependencies in the spec files must be specified on them. Not on the packages. For example if the code used "GD" module, the spec file should read "perl(GD)", not "perl-GD".
Also you are missing many of them. For example, cmake/pod2man-wrapper.pl is executed at build time, there should be BuildRequires on perl(File::Copy), perl(File::Temp), perl(Getopt::Long), perl(strict), perl(warnings).
I didn't examined the build process thoroughly, but I think build-time dependency on perl(GD) (no perl-GD) is not needed because none of the three files that use it does look like being executed then building the package. Maybe they could if you run tests. It would be good to run tests in the %check section.
Thanks. I've updated the spec file. The test suite has some issue when run from the same directory so I've skipped it.
The URL and Source address are usable. Ok.
Source0 archive is original (SHA-256: c4c77fd279aab6ea56d5b34d2f17a31c59ebe5933a2002d25916ef082bcc008f). Ok.
Summary verified from README. Ok.
Description verified from README. Ok.
FIX: The license tag (GPL+) is wrong. For example README states "GPL v2", COPYRIGHT states GPLv2+. Or wml_aux/txt2html/LICENSE is BSD. And then there is a bunch of bundled libraries, like 15 years copy of pcre in wml_backend/p2_mp4h/pcre/ that is (BSD or GPLv2).
Actually I'm giving up this review until the libraries will be unbundled (and their files pruned in the %prep section to make sure the bundled code is not used). See COPYRIGHT.OTHER for incomplete list of them. Many of them are dozen years old frozen copies. This is rely example how not to program and to package.
The spec file has been adapted for the licenses included, and the author has been notified about bundling. Given the new fedora bundling policies and the fact that this is a program that doesn't interact with other components I don't see bundling as a blocker to including it in fedora.
Unbundling will happen when it happens upstream.
Feel free to attract different reviewer. I'm not going to read 1,6 MB of compressed sources.