https://bugzilla.redhat.com/show_bug.cgi?id=870049
Bug ID: 870049 QA Contact: extras-qa@fedoraproject.org Severity: medium Version: rawhide Priority: medium CC: notting@redhat.com, package-review@lists.fedoraproject.org Assignee: nobody@fedoraproject.org Summary: Review Request: motif - Run-time libraries and programs Regression: --- Story Points: --- Classification: Fedora OS: Linux Reporter: twoerner@redhat.com Type: --- Documentation: --- Hardware: All Mount Type: --- Status: NEW Component: Package Review Product: Fedora
Spec URL: http://twoerner.fedorapeople.org/Motif/motif.spec SRPM URL: http://twoerner.fedorapeople.org/Motif/motif-2.3.4-1.fc17.src.rpm Description: Motif run-time and development environment Fedora Account System Username: twoerner
https://bugzilla.redhat.com/show_bug.cgi?id=870049
--- Comment #1 from Peter Lemenkov lemenkov@gmail.com --- Why do we need to enable static libraries?
https://bugzilla.redhat.com/show_bug.cgi?id=870049
--- Comment #2 from Peter Lemenkov lemenkov@gmail.com --- %defattr(-,root,root) should be removed - it's no longer needed.
https://bugzilla.redhat.com/show_bug.cgi?id=870049
--- Comment #3 from Thomas Woerner twoerner@redhat.com --- We have had static libs in the package for RHEL since years, it might be used.
https://bugzilla.redhat.com/show_bug.cgi?id=870049
--- Comment #4 from Peter Lemenkov lemenkov@gmail.com --- (In reply to comment #3)
We have had static libs in the package for RHEL since years, it might be used.
A very weak reasoning because there are bugs/issues in RHEL which are not fixed for years. Maybe this is exactly that case.
Anyway if you absolutely certain that we need static libs (I'm pretty sure we don't) you must pack them separately.
* https://fedoraproject.org/wiki/Packaging:Guidelines#Packaging_Static_Librari...
https://bugzilla.redhat.com/show_bug.cgi?id=870049
--- Comment #5 from Thomas Woerner twoerner@redhat.com --- O.k. new package is in place:
- new sub package for static libraries - added /etc/X11/mwm directory - removed defattrs
BTW: rpmlint is complaining on tags in man pages (warnings).
https://bugzilla.redhat.com/show_bug.cgi?id=870049
Peter Lemenkov lemenkov@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED Assignee|nobody@fedoraproject.org |lemenkov@gmail.com Flags| |fedora-review?
--- Comment #6 from Peter Lemenkov lemenkov@gmail.com --- Successful Koji scratchbuild for Rawhide:
* http://koji.fedoraproject.org/koji/taskinfo?taskID=4626409
https://bugzilla.redhat.com/show_bug.cgi?id=870049
--- Comment #7 from Peter Lemenkov lemenkov@gmail.com --- REVIEW:
Legend: + = PASSED, - = FAILED, 0 = Not Applicable
+ rpmlint is NOT silent (1290 warnings!) but almost all of these warnings are about undefined macros in man-pages which isn't that harmful. You should take a look at them and report upstream. The rest of rpmlint messages are listed below:
Auriga ~: rpmlint Desktop/motif-* | grep -v manual-page-warning motif.src: W: spelling-error %description -l en_US mwm -> mm, mam, mom
^^^ false positive.
motif.x86_64: W: shared-lib-calls-exit /usr/lib64/libUil.so.4.0.4 exit@GLIBC_2.2.5
^^^ that's a bad architectural design but it's not a blocker.
motif-static.x86_64: W: no-documentation
^^^ It's not required here.
5 packages and 0 specfiles checked; 0 errors, 1290 warnings. Auriga ~:
+ The package is named according to the Package Naming Guidelines. + The spec file name matches the base package %{name}, in the format %{name}.spec. + The package meets the Packaging Guidelines. + The package is licensed with a Fedora approved license and meets the Licensing Guidelines. + The License field in the package spec file matches the actual license (LGPL v2 or later). + The file, containing the text of the license(s) for the package, is included in %doc. + The spec file is written in American English. + The spec file for the package is legible. + The sources used to build the package, match the upstream source, as provided in the spec URL. + The package successfully compiles and builds into binary rpms on at least one primary architecture. See koji link above. + All build dependencies are listed in BuildRequires. 0 No need to handle locales. + The package stores shared library files in some of the dynamic linker's default paths, and it calls ldconfig in %post and %postun. + The package does NOT bundle copies of system libraries. 0 The package is not designed to be relocatable. + The package owns all directories that it creates. + The package does not list a file more than once in the spec file's %files listings. + Permissions on files are set properly. + The package has a %clean section, which contains rm -rf %{buildroot} (or $RPM_BUILD_ROOT). + The package consistently uses macros. + The package contains code, or permissible content. 0 No extremely large documentation files. + Anything, the package includes as %doc, does not affect the runtime of the application. + Header files are stored in a -devel package. + Static libraries are stored in a -static package. 0 No pkgconfig(.pc) files. + The library file(s) that end in .so (without suffix) is(are) stored in a -devel package.
- The -devel package MUST require the base package using a fully versioned dependency: Requires: %{name}%{?_isa} = %{version}-%{release}. Note the "%{?_isa}" macro.
+ The package does NOT contain any .la libtool archives. 0 Not a GUI application (which means that it doesn't add some userspace graphical utilities which requires *.desktop file).
- The package can not own files or directories already owned by other packages. I really concerned about
* xorg-x11-xbitmaps who is the owner of the /usr/include/X11/bitmaps/ directory * xorg-x11-xinit, owner of the /etc/X11/xinit/xinitrc.d/
*If* these packages are picked up automatically by a dependency checker then it's ok. If not - you must add them as a Requires.
+ At the beginning of %install, the package runs rm -rf %{buildroot} (or $RPM_BUILD_ROOT). + All filenames in rpm packages are valid UTF-8.
Almost done.
* Please, fix devel sub-package dependency (requires %{?_isa} macro). * Ensure that xorg-x11-xbitmaps and xorg-x11-xinit are picked up automatically and inserted into dependency chain while installing Motif rpm. Otherwise please add them explicitly.
https://bugzilla.redhat.com/show_bug.cgi?id=870049
Peter Lemenkov lemenkov@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |jochen@herr-schmitt.de
--- Comment #8 from Peter Lemenkov lemenkov@gmail.com --- *** Bug 870215 has been marked as a duplicate of this bug. ***
https://bugzilla.redhat.com/show_bug.cgi?id=870049
--- Comment #9 from Thomas Woerner twoerner@redhat.com --- Ok, new spec and package in place:
http://twoerner.fedorapeople.org/Motif/motif.spec http://twoerner.fedorapeople.org/Motif/motif-2.3.4-2.fc17.src.rpm
https://bugzilla.redhat.com/show_bug.cgi?id=870049
Peter Lemenkov lemenkov@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags|fedora-review? | Flags| |fedora-review+
--- Comment #10 from Peter Lemenkov lemenkov@gmail.com --- (In reply to comment #9)
Ok, new spec and package in place:
http://twoerner.fedorapeople.org/Motif/motif.spec http://twoerner.fedorapeople.org/Motif/motif-2.3.4-2.fc17.src.rpm
I can't find any other issues so this package is
APPROVED.
https://bugzilla.redhat.com/show_bug.cgi?id=870049
Thomas Woerner twoerner@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags|fedora-review+ | Flags| |fedora-review? Flags| |fedora-cvs?
--- Comment #11 from Thomas Woerner twoerner@redhat.com --- New Package SCM Request ======================= Package Name: motif Short Description: Run-time libraries and programs Owners: twoerner siddharths Branches: f17 f18 InitialCC:
https://bugzilla.redhat.com/show_bug.cgi?id=870049
Peter Lemenkov lemenkov@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags|fedora-review? | Flags| |fedora-review+
https://bugzilla.redhat.com/show_bug.cgi?id=870049
--- Comment #12 from Jon Ciesla limburgher@gmail.com --- Git done (by process-git-requests).
https://bugzilla.redhat.com/show_bug.cgi?id=870049
Kevin Kofler kevin@tigcc.ticalc.org changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |kevin@tigcc.ticalc.org
--- Comment #13 from Kevin Kofler kevin@tigcc.ticalc.org --- So what will you do with lesstif, will this package Obsolete it? Or is the plan to keep it as a compatibility package for the old OpenMotif 2.1 ABI (libXm.so.2)?
https://bugzilla.redhat.com/show_bug.cgi?id=870049
Gilles J. Seguin segg2@videotron.ca changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |segg2@videotron.ca
--- Comment #14 from Gilles J. Seguin segg2@videotron.ca --- (In reply to comment #13)
So what will you do with lesstif, will this package Obsolete it? Or is the plan to keep it as a compatibility package for the old OpenMotif 2.1 ABI (libXm.so.2)?
IIRC lesstif is targeting compatibility with motif 1.2 which should be libXm.so.2 motif 2.1 is libXm.so.4
saying that the motif package provide openmotif, how the path to /usr/lib64/openmotif/libXm.so.4 should be interpreted ? ldconfig
https://bugzilla.redhat.com/show_bug.cgi?id=870049
Thomas Woerner twoerner@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution|--- |NEXTRELEASE Last Closed| |2012-11-02 10:54:48
--- Comment #15 from Thomas Woerner twoerner@redhat.com --- motif is not obsoleting lesstif right now, but it should do this in the near future.
Even with this latest Motif version old code (version 1.x and up) still can be compiled and used.
package-review@lists.fedoraproject.org