https://bugzilla.novell.com/show_bug.cgi?id=421322 User sndirsch@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=421322#c1 Stefan Dirsch changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|sndirsch@novell.com |crrodriguez@novell.com --- Comment #1 from Stefan Dirsch 2008-08-29 02:53:28 MDT --- I didn't break it. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=421322 User sndirsch@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=421322#c3 Stefan Dirsch changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |NEEDINFO Info Provider| |bwalle@novell.com --- Comment #3 from Stefan Dirsch 2008-08-29 03:08:04 MDT --- Where can I find this policy? Why isn't forced by letting the build fail, if it's that important? -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=421322 User sndirsch@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=421322#c5 Stefan Dirsch changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|sndirsch@novell.com |crrodriguez@novell.com --- Comment #5 from Stefan Dirsch 2008-08-29 03:24:04 MDT --- Thanks. So did Christian asked Mike, if the static lib + .la file of fontconfig might still be useful before he changed this on 2008-08-26? I seriously doubt this. Mike is currently on vacation. I suggest to revert this change until Mike is back from vacation. I'm *not* kidding! It's a bad idea in general to change packages of others without asking them beforehand. Especially when they are currently on vacation. Christian, please consider to learn this lesson. Thanks. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=421322 User sndirsch@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=421322#c6 Stefan Dirsch changed: What |Removed |Added ---------------------------------------------------------------------------- CC|sndirsch@novell.com |mfabian@novell.com Component|X.Org |X11 Applications QAContact|xorg-maintainer-bugs@forge.provo.novell.com |sndirsch@novell.com --- Comment #6 from Stefan Dirsch 2008-08-29 03:35:38 MDT --- Mike will be back on Mon 2008-09-08. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=421322 User sndirsch@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=421322#c8 --- Comment #8 from Stefan Dirsch 2008-08-29 04:32:11 MDT --- If you would revert your fontconfig change, we wouldn't need to change/fix any other packages. After Mike is back from vacation, we can discuss with him if it is fine to remove the fontconfig static libs. And change/fix other packages if necessary. This is all I asked you for, i.e. postpone this possible change. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=421322 User crrodriguez@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=421322#c10 --- Comment #10 from Cristian Rodriguez 2008-08-29 04:42:27 MDT --- (In reply to comment #8 from Stefan Dirsch)

If you would revert your fontconfig change, we wouldn't need to change/fix any other packages.

but you dont need to fix anything due to removal of "la" files !! you just have to wait the complete rebuild to finish !! -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=421322 User rodrigo@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=421322#c12 Rodrigo Moya changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |rodrigo@novell.com --- Comment #12 from Rodrigo Moya 2008-08-29 05:23:08 MDT --- I'm having problems building some GNOME packages because of this (libXft still linking to libfontconfig.la). Should this be really fixed once everything rebuilds? I can revert the change if someone gives me permission if rebuilding does not fix it -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=421322 User crrodriguez@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=421322#c14 --- Comment #14 from Cristian Rodriguez 2008-08-29 05:41:24 MDT --- To make it more clear. once xorg-x11-libs is fixed to buildRequire libexpat-devel , and the pacakae rebuilt libXft.la will no longer reference libfontconfig.la and all dependant packages will succeed to build. if Stefan does not want to remove libXft.la it will work anyway, of course that everything will break when he removal of "la" files starts to be enforced. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=421322 User crrodriguez@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=421322#c16 --- Comment #16 from Cristian Rodriguez 2008-08-29 05:56:03 MDT --- (In reply to comment #15 from Stefan Dirsch)

So you broke more stuff

No, that packages ARE BROKEN, they expect WRONG -devel packages to be present in the build root -.. can you please make a bit of analysis before blaming me ?

not mentioning this change in fontconfig.changes.

Yes, I missed that, this is the only point when you are right, sorry. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=421322 User matz@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=421322#c18 Michael Matz changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |matz@novell.com --- Comment #18 from Michael Matz 2008-08-29 06:17:50 MDT --- Stefan: There is nothing forbidding other maintainers to touch packages they don't maintain (it's considered good style to tell the maintainer of course). Especially for janitory work its actually considered nice to just do the work instead of annoying the maintainer. Cristians cleanups make the distribution strictly better. OTOH Cristian: The removal of libexpat-devel from fontconfig-devel is not a 100% useful change. If you need fontconfig-devel, then most probably because you want to link against libfontconfig.so. That in turn is linked against libexpat.so.X, so having also installed libexpat.so seems reasonable. It's correct that there's no strict requirement to have it (or the headers, because the fontconfig headers don't refer to the expat headers), but it seems nice. But there's not point in reverting this now. So, if Cristian tells package maintainers about his changes in the future and Stefan makes xorg-x11-libs buildrequire libexpat-devel, can we all go back to work and live happily ever after? :-) -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=421322 User crrodriguez@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=421322#c20 --- Comment #20 from Cristian Rodriguez 2008-08-29 08:11:40 MDT --- (In reply to comment #19 from Stefan Dirsch)

done + libXft.la file removed for openSUSE > 11.0.

Hint: if you remove just libXft.la ensure that other "la" files in the same package are not referencing it, or just remove all "la" files, they serve no purpose whatsoever. ;-) -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=421322 User sndirsch@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=421322#c22 Stefan Dirsch changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution| |FIXED --- Comment #22 from Stefan Dirsch 2008-08-29 09:15:35 MDT --- Packages are submitted now. Christian, I really would prefer it that you contact me beforehand, if you plan to change any of my packages. Thanks. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=421322 User crrodriguez@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=421322#c24 --- Comment #24 from Cristian Rodriguez 2008-08-29 18:27:55 MDT --- (In reply to comment #23 from Marcus Rückert)

just curious: why not remove all .la files?:)

just curious, why not just ban them all together eh ? ;-) -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=421322 User sndirsch@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=421322#c26 --- Comment #26 from Stefan Dirsch 2008-08-29 20:52:27 MDT --- xorg-x11-libs.changes ------------------------------------------------------------------- Sat Aug 30 04:39:04 CEST 2008 - sndirsch@suse.de - remove also libXft.a (now that libXft.la is gone) -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=421322 User matz@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=421322#c28 --- Comment #28 from Michael Matz 2008-09-01 06:29:58 MDT --- Cristian, as much as I like your cleanups, please be more careful with spreading misleading information. .la files _do_ provide vital information for static linking that isn't found anywhere else, namely dependencies. For ELF DSOs the dependencies are also in the DT_NEEDED entries, that's why (on ELF!) .la files are redundant for them. Static libs don't have such means, so the dependencies need to be written somewhere else, either in the mind of the developers (so to spell them out explicitely in Makefiles), that's fragile, or in pkg-config files (not everyone uses them, and they aren't much better than .la files anyway), or in the .la files. Sometimes the dependencies of static libs will already be fulfilled trivially, or at least in all circumstances in which they happen to be currently used, in that case the .la files are then useless. But not in the general case. Having said all that, of the two fixes for this problem, either retaining the .la file or removing the .a file the latter is the better option. Currently we have about 760 .a files in the system on i386, and I guess at least half of them are useless. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=421322 User bwalle@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=421322#c30 --- Comment #30 from Bernhard Walle 2008-09-01 06:33:33 MDT --- But instead of removing static libraries on a random basis, in my opinion we should first define which static libraries we distribute and then drop the others. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.