Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Review Request: tcl-tclxml - XML parsing utilities for Tcl
https://bugzilla.redhat.com/show_bug.cgi?id=476926
Summary: Review Request: tcl-tclxml - XML parsing utilities for Tcl Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: wart@kobold.org QAContact: extras-qa@fedoraproject.org CC: notting@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora
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 Description: 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.
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=476926
Marcela Maslanova mmaslano@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|nobody@fedoraproject.org |mmaslano@redhat.com
--- Comment #1 from Marcela Maslanova mmaslano@redhat.com 2008-12-18 02:45:14 EDT --- 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 devel: 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 gui: 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.
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=476926
Marcela Maslanova mmaslano@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED
--- Comment #2 from Marcela Maslanova mmaslano@redhat.com 2008-12-18 03:34:47 EDT --- 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 http://koji.fedoraproject.org/koji/taskinfo?taskID=1005993
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=476926
--- Comment #3 from Wart wart@kobold.org 2008-12-18 19:04:42 EDT --- (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 http://koji.fedoraproject.org/koji/taskinfo?taskID=1005993
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:
http://www.kobold.org/~wart/fedora/queue/tcl-tclxml.spec http://www.kobold.org/~wart/fedora/queue/tcl-tclxml-3.2-2.fc11.src.rpm
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=476926
Marcela Maslanova mmaslano@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-review+
--- Comment #4 from Marcela Maslanova mmaslano@redhat.com 2008-12-19 03:25:03 EDT --- I'm sorry I read libtcl instead of libtcl. In this case everything is ok.
APPROVED
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=476926
Wart wart@kobold.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
--- Comment #5 from Wart wart@kobold.org 2008-12-19 11:07:48 EDT --- New Package CVS Request ======================= Package Name: tcl-tclxml Short Description: XML parsing utilities for Tcl Owners: wart Branches: devel InitialCC:
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=476926
Kevin Fenzi kevin@tummy.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
--- Comment #6 from Kevin Fenzi kevin@tummy.com 2008-12-20 23:10:02 EDT --- cvs done.
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=476926
Wart wart@kobold.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |RAWHIDE
--- Comment #7 from Wart wart@kobold.org 2008-12-22 11:10:12 EDT --- Imported and built. Many thanks for the review!
package-review@lists.fedoraproject.org