Bug 185260

Summary:	Review Request: python-chm - Python package for CHM files handling
Product:	[Fedora] Fedora	Reporter:	Patrice Dumas <pertusus>
Component:	Package Review	Assignee:	John Mahowald <jpmahowald>
Status:	CLOSED NEXTRELEASE	QA Contact:	Fedora Package Reviews List <fedora-package-review>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	rawhide	CC:	lakshminaras2002
Target Milestone:	---	Flags:	gwync: fedora-cvs+
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2006-03-17 19:02:09 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:
Bug Depends On:
Bug Blocks:	163779, 185261

Description Patrice Dumas 2006-03-12 21:36:06 UTC

SRPM Name or Url: http://www.environnement.ens.fr/perso/dumas/fc-srpms/python-chm-0.8.2-1.src.rpm
Description: 

The python chm package provides three modules, chm, chmlib and extra,
which provide access to the API implemented by the C library chmlib
and some additional classes and functions. They are used to access
MS-ITSS encoded files - Compressed Html Help files (.chm).

Comment 1 Patrice Dumas 2006-03-14 21:38:05 UTC

Better handling of .pyo files here:

http://www.environnement.ens.fr/perso/dumas/fc-srpms/python-chm-0.8.2-2.src.rpm

Comment 2 John Mahowald 2006-03-17 16:47:27 UTC

Good:

- rpmlint checks return:
W: python-chm-debuginfo objdump-failed
Not critical.

- package meets naming guidelines
- package meets packaging guidelines, following python template
- license (GPL) OK, text in %doc, matches source
- spec file legible, in am. english
- source matches upstream
- package compiles on devel (x86_64)
- no missing BR
- no unnecessary BR
- no locales
- not relocatable
- owns all directories that it creates
- no duplicate files
- permissions ok
- %clean ok
- macro use consistent
- code, not content
- no need for -docs
- nothing in %doc affects runtime
- no need for .desktop file 

APPROVED

Comment 3 Patrice Dumas 2006-03-17 19:02:09 UTC

Builds in devel, thanks.

Comment 4 Narasimhan 2011-06-14 13:06:06 UTC

Package Change Request
======================
Package Name: python-chm
New Branches: el6
Owners: narasim sherry151 pertusus

I have requested commit access to fedora devel and epel 5 branches. Patrice Dumas has asked me to be the owner for el6 https://bugzilla.redhat.com/show_bug.cgi?id=713044. This package is required in el6 because chm2pdf is dependent on this package and security update to chm2pdf is required in el6.

Comment 5 Jason Tibbitts 2011-06-14 13:24:08 UTC

Git done (by process-git-requests).

Comment 6 Narasimhan 2012-08-07 14:05:18 UTC

Requesting to unretire the devel (master) branch. I maintain chm2pdf which requires python-chm.

Comment 7 Gwyn Ciesla 2012-08-07 14:22:32 UTC

Unretired, take ownership in pkgdb.

Comment 8 Narasimhan 2012-08-10 13:52:41 UTC

Package Change Request
======================
Package Name: python-chm
New Branches: f18
Owners: narasim

Comment 9 Gwyn Ciesla 2012-08-10 13:56:07 UTC

Git done (by process-git-requests).