http://bugzilla.novell.com/show_bug.cgi?id=509945
Summary: Enforce packaging of static libraries by rpmlint check Classification: openSUSE Product: openSUSE 11.2 Version: Factory Platform: Other OS/Version: Other Status: NEW Severity: Normal Priority: P5 - None Component: Basesystem AssignedTo: dmueller@novell.com ReportedBy: prusnak@novell.com QAContact: qa@suse.de CC: ro@novell.com, mrueckert@novell.com, crrodriguez@novell.com Found By: Development
We are trying to adopt the static libraries packaging scheme described on http://en.opensuse.org/Packaging/Guidelines#Packaging_Static_Libraries
By adding rpmlint check we will enforce the correct packaging.
rpmlint should check whether the package contains *.{a,la} files and perform these steps:
a) package contains *.la files, does not contain *.a files -> throw error suggesting the packager to remove *.la files in %install phase
b) package contains only *.la and *.a files -> check if the name ends with "-devel-static", throw error otherwise, suggesting the rename
c) package contains *.la, *.a files AND *.so files (and probably some includes) -> check if the name ends with "-devel" AND package also provides "-devel-static" symbol, throw error otherwise, suggesting the addition of symbol -> also point out that packager has to be sure that static libraries are absolutely needed in the system. If not the package should be built with --disable-static and *.la files removed in %install phase
http://bugzilla.novell.com/show_bug.cgi?id=509945
User crrodriguez@novell.com added comment http://bugzilla.novell.com/show_bug.cgi?id=509945#c1
--- Comment #1 from Cristian Rodríguez crrodriguez@novell.com 2009-06-04 08:39:33 MDT --- one suggestion: -devel-static packages should not be accepted unless reviewed by autobuild team or something similar.
the check must also ensure all static libraries "down the dependency chain" are available, otherwise throw a warning.
Overall, Im against the idea of having a lot of -devel-static packages, it makes no sense at all.
http://bugzilla.novell.com/show_bug.cgi?id=509945
User mrueckert@novell.com added comment http://bugzilla.novell.com/show_bug.cgi?id=509945#c2
--- Comment #2 from Marcus Rückert mrueckert@novell.com 2009-06-04 08:49:21 MDT --- yes those should be an exception. if the check stops all packages with static libraries many people will first start thinking whether they really need the static library or not.
http://bugzilla.novell.com/show_bug.cgi?id=509945
User jsmeix@novell.com added comment http://bugzilla.novell.com/show_bug.cgi?id=509945#c3
Johannes Meixner jsmeix@novell.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |jsmeix@novell.com
--- Comment #3 from Johannes Meixner jsmeix@novell.com 2009-06-10 03:37:12 MDT --- FYI: Cristian Rodriguez already changed CUPS (Many Thanks for it!) and right now I filed submitreq 11749 for CUPS -> openSUSE:Factory "no longer provide static libraries in cups-devel to enforce detection of other software which might be built with static libraries".
http://bugzilla.novell.com/show_bug.cgi?id=509945
Dirk Mueller dmueller@novell.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Priority|P5 - None |P4 - Low
http://bugzilla.novell.com/show_bug.cgi?id=509945
User crrodriguez@novell.com added comment http://bugzilla.novell.com/show_bug.cgi?id=509945#c4
--- Comment #4 from Cristian Rodríguez crrodriguez@novell.com 2009-10-13 05:39:08 MDT --- Dirk: If this check is added, tell people in the error message to email crrodriguez@opensuse.org (or better opensuse-packaging ;) ) to request help or review.
http://bugzilla.novell.com/show_bug.cgi?id=509945
http://bugzilla.novell.com/show_bug.cgi?id=509945#c
Dirk Mueller dmueller@novell.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED Component|Basesystem |Basesystem Product|openSUSE 11.2 |openSUSE 11.3
http://bugzilla.novell.com/show_bug.cgi?id=509945 http://bugzilla.novell.com/show_bug.cgi?id=509945#c5
--- Comment #5 from Bernhard Wiedemann bwiedemann@suse.com --- This is an autogenerated message for OBS integration: This bug (509945) was mentioned in https://build.opensuse.org/request/show/11566 Factory / xz https://build.opensuse.org/request/show/11589 Factory / bzip2 https://build.opensuse.org/request/show/11611 Factory / mpc https://build.opensuse.org/request/show/11749 Factory / cups
http://bugzilla.novell.com/show_bug.cgi?id=509945
SMASH SMASH smash_bz@suse.de changed:
What |Removed |Added ---------------------------------------------------------------------------- Whiteboard| | maint:planned:update