[Bug 193889] New: Review Request: ht2html - The www.python.org Web site generator

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=193889

Summary: Review Request: ht2html - The www.python.org Web site
                    generator
           Product: Fedora Extras
           Version: devel
          Platform: All
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: normal
         Component: Package Review
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]
         QAContact: [e-mail]

Spec URL: http://people.redhat.com/ifoox/extras/ht2html.spec
SRPM URL: http://people.redhat.com/ifoox/extras/ht2html-2.0-1jpp_1fc.src.rpm
Description:
This script is called ht2html because it generates .html files from .ht template files. The format of these .ht files is essentially normal HTML, with a set of optional RFC 2822-like headers at the top of the file. These headers specify certain options that ht2html's various classes support.

This is my first package submission to Fedora Extras, so I need a sponsor.

[Bug 193889] Review Request: ht2html - The www.python.org Web site generator

[Red Hat Bugzilla]

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: ht2html - The www.python.org Web site generator

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=193889

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
OtherBugsDependingO|                            |177841
              nThis|                            |

[Bug 193889] Review Request: ht2html - The www.python.org Web site generator

[Red Hat Bugzilla]

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: ht2html - The www.python.org Web site generator

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=193889

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
OtherBugsDependingO|                            |193898
              nThis|                            |

[Bug 193889] Review Request: ht2html - The www.python.org Web site generator

[Red Hat Bugzilla]

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: ht2html - The www.python.org Web site generator

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=193889

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

------- Additional Comments From [e-mail]  2006-06-23 12:40 EST -------
Just some comments until Hans feels confident sponsoring you:

Please just use a plain integer for the release number.

I suggest not compressing ht2html; it saves all of eleven bytes makes
maintanence incrementally nore difficult.

Source0: isn't a URL, and in addition that source file isn't available from
upstream.  (They only supply a .gz file.)  Suggest using
http://dl.sf.net/%%7Bname%7D/%%7Bname%7D-%%7Bversion%7D.tar.gz

Please don't use Vendor or Distribution.

I'm not sure why you have BR: python-devel; this package just copies files into
place.

I can find no information indicating that this software is in the public domain.
 Can you provide a reference?

RPM will compile all of the .py files; you will need to %ghost the .pyo files
which are generated.

[Bug 193889] Review Request: ht2html - The www.python.org Web site generator

[Red Hat Bugzilla]

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: ht2html - The www.python.org Web site generator

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=193889

------- Additional Comments From [e-mail]  2006-06-23 16:39 EST -------
Hi Jason, thanks for your comments. I've updated a new spec file and SRPM here:
http://people.redhat.com/ifoox/extras/ht2html.spec
http://people.redhat.com/ifoox/extras/ht2html-2.0-1jpp_2fc.src.rpm
> Please just use a plain integer for the release number.

This package is taken from jpackage.org, and I'd like to keep the versioning
consistent with theirs. Is the non-numeric release a big problem here?
> I suggest not compressing ht2html; it saves all of eleven bytes makes
> maintanence incrementally nore difficult.

Done.
> Source0: isn't a URL, and in addition that source file isn't available from
> upstream.  (They only supply a .gz file.)  Suggest using
> http://dl.sf.net/%%7Bname%7D/%%7Bname%7D-%%7Bversion%7D.tar.gz

Done.
> Please don't use Vendor or Distribution.

Done.
> I'm not sure why you have BR: python-devel; this package just copies files into
> place.

You're right, done.
> I can find no information indicating that this software is in the public domain.
>  Can you provide a reference?

There seems to be no mention of licensing in the software itself, but I found
mention of  in the sourceforge net. However, rpmlint tells me that both 'Python
License' and '' are invalid. Is there a cannonical way to call this license?
> RPM will compile all of the .py files; you will need to %ghost the .pyo files
> which are generated.

I've %ghosted the .pyo files, and listed *.py and *.pyc files as seperate
entries in the %files section.

[Bug 193889] Review Request: ht2html - The www.python.org Web site generator

[Red Hat Bugzilla]

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: ht2html - The www.python.org Web site generator

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=193889

------- Additional Comments From [e-mail]  2006-06-23 16:41 EST -------
(In reply to comment #2)
> > I can find no information indicating that this software is in the public domain.
> >  Can you provide a reference?
>
> There seems to be no mention of licensing in the software itself, but I found
> mention of  in the sourceforge net. However, rpmlint tells me that both 'Python
> License' and '' are invalid. Is there a cannonical way to call this license?

Sorry this should read:
There seems to be no mention of licensing in the software itself, but I found
mention of Python License (CNRI Python License) in the sourceforge net. However,
rpmlint tells me that both 'Python
License' and 'Python License (CNRI Python License)' are invalid. Is there a
cannonical way to call this license?

[Bug 193889] Review Request: ht2html - The www.python.org Web site generator

[Red Hat Bugzilla]

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: ht2html - The www.python.org Web site generator

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=193889

------- Additional Comments From [e-mail]  2006-06-23 19:35 EST -------
(In reply to comment #2)
> This package is taken from jpackage.org, and I'd like to keep the versioning
> consistent with theirs. Is the non-numeric release a big problem here?

The guidelines are clear: http://fedoraproject.org/wiki/Packaging/NamingGuidelines

Non-Numeric Version in Release

There are two cases in which non-numeric versions occur in the Release field:

* Pre-release packages
    * Snapshot packages

This package is neither.  We want simple integer release numbers when possible.
 Just imagine what this package would look like with a proper dist tag added:
"ht2html-2.0-1jpp_2fc.fc6.rpm".  That's just insane.
> There seems to be no mention of licensing in the software itself, but I found
> mention of  in the sourceforge net. However, rpmlint tells me that both 'Python
> License' and 'Python License (CNRI Python License)' are invalid. Is there a

cannonical way to call this license?

rpmlint can be a bit confusing; in this case, the valid licenses accepted are
overridden by a Fedora-specific file /usr/share/rpmlint/config.  The string to
use is "Python Software Foundation License".  However, honestly with absolutely
no license mentioned in the source, you really do need to contact upstream and
get some sort of statement.  When you get that, include the correspondence in
the package.  (In a perfect world they'd make a new release which includes a
license statement, but this package is pretty old so I doubt that would happen.)

[Bug 193889] Review Request: ht2html - The www.python.org Web site generator

[Red Hat Bugzilla]

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: ht2html - The www.python.org Web site generator

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=193889

------- Additional Comments From [e-mail]  2006-06-26 13:41 EST -------
New files:
http://people.redhat.com/ifoox/extras/ht2html-2.0-1.src.rpm
http://people.redhat.com/ifoox/extras/ht2html.spec

(In reply to comment #4)
> Non-Numeric Version in Release

I fixed this, changing the release to simply 1.
> > There seems to be no mention of licensing in the software itself, but I found
> > mention of  in the sourceforge net. However, rpmlint tells me that both 'Python
> > License' and 'Python License (CNRI Python License)' are invalid. Is there a
>
> cannonical way to call this license?
>
> rpmlint can be a bit confusing; in this case, the valid licenses accepted are
> overridden by a Fedora-specific file /usr/share/rpmlint/config.  The string to
> use is "Python Software Foundation License".  However, honestly with absolutely
> no license mentioned in the source, you really do need to contact upstream and
> get some sort of statement.  When you get that, include the correspondence in
> the package.  (In a perfect world they'd make a new release which includes a
> license statement, but this package is pretty old so I doubt that would happen.)

On friday the mailing list archives on sourceforge were unavailable. Today I
found a message [1] to the ML from the main author of ht2html, stating that it
is licensed under the PSF license. So I chaned the license to Python Software
Foundation License".

[1] - http://sourceforge.net/mailarchive/forum.php?thread_id=1785154&forum_id=...

[Bug 193889] Review Request: ht2html - The www.python.org Web site generator

[Red Hat Bugzilla]

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: ht2html - The www.python.org Web site generator

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=193889

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
         AssignedTo|[e-mail] |[e-mail]
OtherBugsDependingO|163776                      |163778
              nThis|                            |

------- Additional Comments From [e-mail]  2006-07-24 11:54 EST -------
Let's try to get things moving.  Igor, I'll sponsor you.  I'll work up a full
review in a minute, but one thing that I can immediately see should be done is
to take the mailing list correspondence and include it in the package so that
there's some statement of the license there.

[Bug 193889] Review Request: ht2html - The www.python.org Web site generator

[Red Hat Bugzilla]

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: ht2html - The www.python.org Web site generator

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=193889

------- Additional Comments From [e-mail]  2006-07-24 13:34 EST -------
Hi Jason,

Thanks again for looking at this. Here are the new files:

New files:
http://people.redhat.com/ifoox/extras/ht2html-2.0-2.src.rpm
http://people.redhat.com/ifoox/extras/ht2html.spec

I just included a comment that references the ML thread above just above the
License tag.

Should I also include a %{?dist} tag in the release?

[Bug 193889] Review Request: ht2html - The www.python.org Web site generator

[Red Hat Bugzilla]

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: ht2html - The www.python.org Web site generator

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=193889

------- Additional Comments From [e-mail]  2006-07-24 13:52 EST -------
Honestly I'd actually copy the relevant text of that message into a file that
you include as %doc, and also include the URL of the original along with a bit
of explanation.  Someone with an installed package who wants to check the
license won't have access to the specfile.  I also don't think including a just
URL is sufficient, especially given how difficult sourceforge can be to reach
sometimes.

I personally would always use the dist tag because of the amount of work it
saves when maintaining identical packages across multiple distribution releases.
 But it's up to you.

I note that the final package has no requirement on Python.  I believe this is a
blocker.

Review:
* source files match upstream:
   925d359f7db48c44ed0bc3044cebd3f0  ht2html-2.0.tar.gz
* package meets naming and packaging guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
X dist tag is present (not a blocker but you probably want to use it)
* build root is correct.
* license field matches the actual license.
X license is open source-compatible.  Statement of license should be included as
%doc in the package.
* latest version is being packaged.
* BuildRequires are proper.
* %clean is present.
* package builds in mock (development, x86_64).
* noarch package; no debuginfo.
* rpmlint is silent.
X final provides and requires: missing python requirement.
   ht2html = 2.0-2
  =
   /bin/sh
   /usr/bin/env
* %check is not present; no test suite upstream.
* no shared libraries are present.
* 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.
* no scriptlets present.
* code, not content.
* documentation is small, so no -docs subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.
* no headers.
* no pkgconfig files.
* no libtool .la droppings.
* not a GUI app.

[Bug 193889] Review Request: ht2html - The www.python.org Web site generator

[Red Hat Bugzilla]

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: ht2html - The www.python.org Web site generator

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=193889

------- Additional Comments From [e-mail]  2006-07-24 14:47 EST -------
New files:
http://people.redhat.com/ifoox/extras/ht2html-2.0-3.src.rpm
http://people.redhat.com/ifoox/extras/ht2html.spec

I've created a LICENSE file that explains the situation and provides the actualy
license, as well as quotes and references the ML post.

I've added a Requires for Python and also a %{?dist} to the release.

[Bug 193889] Review Request: ht2html - The www.python.org Web site generator

[Red Hat Bugzilla]

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: ht2html - The www.python.org Web site generator

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=193889

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
OtherBugsDependingO|163778, 177841              |163779
              nThis|                            |

------- Additional Comments From [e-mail]  2006-07-24 19:58 EST -------
Unfortunately the Python requirement results in a package that won't install:

Error: Missing Dependency: Python is needed by package ht2html

The requirement should be for "python".  Other than that one issue, everything
looks good.  The information you include in LICENSE is especially complete.

Apart from changing the case of a single letter, I think this is ready to go, so
I'll go ahead and approve and you can fix it when you check in.

I see you've already done the CLA, so go ahead and request your cvsextras and
fedorabugs memberships and I'll take care of them.  Then you can check in and
request your builds.  Let me know if you have any problems.

APPROVED

[Bug 193889] Review Request: ht2html - The www.python.org Web site generator

[Red Hat Bugzilla]

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: ht2html - The www.python.org Web site generator

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=193889

------- Additional Comments From [e-mail]  2006-08-08 18:14 EST -------
Any reason this hasn't been built yet?  I see that it's in CVS and has branched
for FC-5, but I don't see any packages in the repository and of course this bug
hasn't been closed.

[Bug 193889] Review Request: ht2html - The www.python.org Web site generator

[Red Hat Bugzilla]

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: ht2html - The www.python.org Web site generator

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=193889

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |NEEDINFO_REPORTER

[Bug 193889] Review Request: ht2html - The www.python.org Web site generator

[Red Hat Bugzilla]

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: ht2html - The www.python.org Web site generator

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=193889

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEEDINFO_REPORTER           |NEEDINFO
               Flag|                            |needinfo?([e-mail])

------- Additional Comments From [e-mail]  2006-08-30 02:01 EST -------
It's been another three weeks.  Is anything going to happen here?

[Bug 193889] Review Request: ht2html - The www.python.org Web site generator

[Red Hat Bugzilla]

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: ht2html - The www.python.org Web site generator

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=193889

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEEDINFO                    |CLOSED
         Resolution|                            |NEXTRELEASE
               Flag|needinfo?([e-mail]) |

------- Additional Comments From [e-mail]  2006-09-02 12:23 EST -------
My appologies for the very long delay, I was away and couldn't attend to these
packages. They have now been built.

[Bug 193889] Review Request: ht2html - The www.python.org Web site generator

[Red Hat Bugzilla]

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: ht2html - The www.python.org Web site generator

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=193889

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|normal                      |medium
           Priority|normal                      |medium

[Bug 193889] Review Request: ht2html - The www.python.org Web site generator

[Red Hat Bugzilla]

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: ht2html - The www.python.org Web site generator

https://bugzilla.redhat.com/show_bug.cgi?id=193889

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
            Product|Fedora Extras               |Fedora
            Version|devel                       |rawhide

[Bug 193889] Review Request: ht2html - The www.python.org Web site generator

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

https://bugzilla.redhat.com/show_bug.cgi?id=193889

Milos Jakubicek  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]
               Flag|                            |fedora-cvs?

--- Comment #14 from Milos Jakubicek  2010-08-02 02:10:51 EDT ---
Package Change Request
======================
Package Name: ht2html
New Branches: EL-5 EL-6
Owners: mjakubicek rlandmann

[Bug 193889] Review Request: ht2html - The www.python.org Web site generator

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

https://bugzilla.redhat.com/show_bug.cgi?id=193889

--- Comment #15 from Kevin Fenzi  2010-08-02 12:22:51 EDT ---
GIT done (by process-git-requests).