Red Hat Bugzilla – Bug 476926
Review Request: tcl-tclxml - XML parsing utilities for Tcl
Last modified: 2008-12-22 11:10:12 EST
Spec URL: http://www.kobold.org/~wart/fedora/queue/tcl-tclxml.spec
SRPM URL: http://www.kobold.org/~wart/fedora/queue/tcl-tclxml-3.2-1.fc9.src.rpm
This package is a version bump, rename (to conform with Tcl packaging guidelines), and reorganization of the existing tclxml and tcldom packages. With this new upstream release, tclxml, tcldom, and tclxslt have all been merged into a single tclxml tarball. The shared libraries have also been merged, so it is no longer possible to keep the tclxml and tcldom packages separate.
This update will be for F11 only. As far as I can tell, nothing depends on the Obsolete *-libxml and *-expat packages, so they should not need to be listed in a separate Provides:
Since this involves some major changes to the spec file, I am requesting a new package review.
OK source files match upstream:
OK package meets naming and versioning guidelines.
OK specfile is properly named, is cleanly written and uses macros consistently.
OK dist tag is present.
OK build root is correct.
OK license field BSD matches the actual license.
OK license is open source-compatible.
OK latest version is being packaged.
OK BuildRequires are proper.
OK %clean is present.
OK package builds in mock (Rawhide/x86_64).
OK debuginfo package looks complete.
OK rpmlint is silent.
tcl-tclxml.x86_64: E: explicit-lib-dependency tcllib
tcl-tclxml.x86_64: W: obsolete-not-provided tclxml-libxml2
tcl-tclxml.x86_64: W: obsolete-not-provided tclxml-expat
tcl-tclxml.x86_64: W: obsolete-not-provided tcldom-expat
tcl-tclxml.x86_64: W: obsolete-not-provided tcldom-libxml2
tcl-tclxml-devel.x86_64: W: no-documentation
tcl-tclxml-devel.x86_64: W: spurious-executable-perm /usr/lib64/tcl8.5/tclxml-3.2/libTclxmlstub3.2.a
tcl-tclxml-gui.x86_64: W: no-documentation
OK final provides and requires look sane.
OK no shared libraries are added to the regular linker search paths.
OK owns the directories it creates.
OK no duplicates in %files.
OK file permissions are appropriate.
OK no scriptlets present.
OK code, not content.
OK documentation is small, so no -docs subpackage is necessary.
OK %docs are not necessary for the proper functioning of the package.
OK no headers.
OK no pkgconfig files.
OK no libtool .la droppings.
The provides probably won't be needed as you said. I run:
repoquery --whatrequires --alldeps tclxml-expat and no other dependencies were found.
What about the permission in devel package? Also is it the dependency on tcllib necessary? Here is build in mock without tcllib
(In reply to comment #2)
> What about the permission in devel package? Also is it the dependency on tcllib
> necessary? Here is build in mock without tcllib
tcllib is a runtime dependency, not a buildtime dependency (hence the use of Requires: as opposed to BuildRequires:). From the tclxml readme file:
"In order for the Tcl-only parser to resolve external entities, the tcllib package must be installed."
Updated package that fixes the stub library permissions:
I'm sorry I read libtcl instead of libtcl. In this case everything is ok.
New Package CVS Request
Package Name: tcl-tclxml
Short Description: XML parsing utilities for Tcl
Imported and built. Many thanks for the review!