Bug 1295710

Summary:	Review Request: wml - WML is a free and extensible Webdesigner's off-line HTML generation toolkit for Unix
Product:	[Fedora] Fedora	Reporter:	Nikos Mavrogiannopoulos <nmavrogi>
Component:	Package Review	Assignee:	Nobody's working on this, feel free to take it <nobody>
Status:	CLOSED WONTFIX	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	rawhide	CC:	package-review, ppisar
Target Milestone:	---
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2020-07-10 06:47:23 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Nikos Mavrogiannopoulos 2016-01-05 10:26:30 UTC

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

Comment 1 Petr Pisar 2016-01-06 11:24:36 UTC

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.

Comment 2 Nikos Mavrogiannopoulos 2016-01-06 13:45:21 UTC

Thanks. I've updated the spec file. The test suite has some issue when run from the same directory so I've skipped it.

Comment 3 Petr Pisar 2016-01-14 14:45:55 UTC

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.

Comment 4 Nikos Mavrogiannopoulos 2016-01-19 10:52:22 UTC

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.

https://bitbucket.org/shlomif/website-meta-language/issues/2/

Comment 5 Petr Pisar 2016-01-19 11:11:54 UTC

Feel free to attract different reviewer. I'm not going to read 1,6 MB of compressed sources.

Comment 6 Package Review 2020-07-10 00:54:18 UTC

This is an automatic check from review-stats script.

This review request ticket hasn't been updated for some time. We're sorry
it is taking so long. If you're still interested in packaging this software
into Fedora repositories, please respond to this comment clearing the
NEEDINFO flag.

You may want to update the specfile and the src.rpm to the latest version
available and to propose a review swap on Fedora devel mailing list to increase
chances to have your package reviewed. If this is your first package and you
need a sponsor, you may want to post some informal reviews. Read more at
https://fedoraproject.org/wiki/How_to_get_sponsored_into_the_packager_group.

Without any reply, this request will shortly be considered abandoned
and will be closed.
Thank you for your patience.