Bug 226494
Summary: | Merge Review: tk | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Nobody's working on this, feel free to take it <nobody> |
Component: | Package Review | Assignee: | Wart <wart> |
Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Package Reviews List <fedora-package-review> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | mmaslano, redhat-bugzilla |
Target Milestone: | --- | Flags: | wart:
fedora-review+
|
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2007-08-23 12:20:05 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
Nobody's working on this, feel free to take it
2007-01-31 21:10:57 UTC
Marcela, again, again and again: Watch out _before_ you cause unowned/orphaned directories, please! The directory /usr/share/tk8.4 is no longer owned by any package but should be owned by tk (like before): --- tk.spec 2007-02-15 16:25:56.000000000 +0100 +++ tk.spec.rsc 2007-02-16 14:56:01.000000000 +0100 @@ -105,11 +105,7 @@ %files %defattr(-,root,root,-) %{_bindir}/wish* -%{_datadir}/%{name}%{majorver}/demos/ -%{_datadir}/%{name}%{majorver}/images/ -%{_datadir}/%{name}%{majorver}/msgs/ -%{_datadir}/%{name}%{majorver}/*.tcl -%{_datadir}/%{name}%{majorver}/tclIndex +%{_datadir}/%{name}%{majorver} %{_libdir}/lib%{name}%{majorver}.so %{_libdir}/%{name}%{majorver} %{_mandir}/man1/* rpmlint output: E: tk invalid-soname /usr/lib/libtk8.4.so libtk8.4.so As noted in the Tcl merge review, this is a historical oddity that would probably break lots of stuff if changed. The rpmlint warning can be ignored. W: tk dangerous-command-in-%pre rm It appears that the %pre script attempts to remove a link named /usr/{lib,lib64}/tk8.4. Why does it do this? Is this an attempt to clean up a link that was erroneously made in a previous version of Tk? GOOD ==== * Package named appropriately * Sources matches upstream * Spec file legible and in Am. English * BSD license ok, license file included * Build dependencies ok * No locales * No need for a .desktop file * ldconfig called correctly for libraries in %{_libdir} * Not relocatable * No duplicate %files * No need for -doc subpackage * headers and unversioned .so files in -devel subpackage MUSTFIX ======= * The 'URL:' tag is the -devel package description is redundant; the url defaults to the one specified for the main package. * As mentioned in comment #1, the /usr/share/tk8.4 directory is unowned. The patch in comment #1 will fix it, and simplify the %files section. * Mixed use of $RPM_BUILD_ROOT and %{buildroot}. Choose either one and use it consistently. * Move the %{_mandir}/mann/* man pages out of the -devel subpackage and into the main package; these are man pages for the script-level API, not the C-level API. The %{_mandir}/man3/* man pages can remain in the -devel subpackage. * The TOPDIR=$PWD at the top of %build is unnecessary. Remove it. NOTES ===== * package does not currently build because Tcl is not providing the private header files in /usr/include/tcl-private. This is a bug in the tcl package, not tk. * The summary shouldn't contain the package name. You should drop the 'Tk' from the summary and capitalize the 'g': Graphical toolkit for the Tcl scripting language > W: tk dangerous-command-in-%pre rm
It should remove old directory before installation, if exists.
All MUSTFIX items fixed. Directory ownership looks ok nw. /Resetting the 'assigned to' field based on the recently adopted review guidelines: https://www.redhat.com/archives/fedora-maintainers/2007-February/msg00682.html |