Bug 307720

Summary: flash-player does not require libexpat1-32bit
Product: [openSUSE] openSUSE 10.3 Reporter: Pavol Rusnak <prusnak>
Component: Update ProblemsAssignee: Pavol Rusnak <prusnak>
Status: RESOLVED FIXED QA Contact: Jiri Srain <jsrain>
Severity: Major    
Priority: P5 - None CC: dmueller, lars.vogdt, ro
Version: Beta 2   
Target Milestone: ---   
Hardware: x86-64   
OS: openSUSE 10.3   
Whiteboard:
Found By: Development Services Priority:
Business Priority: Blocker: ---
Marketing QA Status: --- IT Deployment: ---
Attachments: yast 2 logs

Description Pavol Rusnak 2007-09-05 11:43:13 UTC
I updated from Alpha to Beta2 and flash-player (in firefox using nspluginwrapper) stopped working.

To get it working again I had to do following steps:

a) install libexpat1-32bit (this should be automatically by Requires)
b) execute nspluginwrapper -i /usr/lib/browser-plugins/libflashplayer.so
Comment 1 Dirk Mueller 2007-09-05 12:14:42 UTC
please attach yast logs. nspluginwrapper has triggers which should fix this: 


%triggerin -- RealPlayer acroread acroread_ja flash-player mplayerplug-in
%{_bindir}/%{name} -a -u



a) is obviously an update bug. 
Comment 2 Pavol Rusnak 2007-09-05 13:27:29 UTC
Created attachment 162020 [details]
yast 2 logs
Comment 3 Jiri Srain 2007-09-06 06:23:47 UTC
Checked the package dependencies of flash-player and nspluginwrapper, neither of them has libexpat* in its Requires.
Comment 4 Pavol Rusnak 2007-09-06 07:56:50 UTC
(In reply to comment #3 from Jiri Srain)
> Checked the package dependencies of flash-player and nspluginwrapper, neither
> of them has libexpat* in its Requires.

[prusnak@spectra ~] ldd /usr/bin/nspluginwrapper | grep expat

[prusnak@spectra ~] ldd /usr/lib/browser-plugins/libflashplayer.so | grep expat
libexpat.so.1 => /lib/libexpat.so.1 (0xf6f70000)

Flash-player should Require libexpat1-32bit on x86_64, and libexpat1 on x86. But these requires should be added automagically, aren't they?
Comment 5 Dirk Mueller 2007-09-06 09:38:00 UTC
flash player doesn't require libexpat. flash player requires libfontconfig, which happens to require libexpat. 

thats the dependency chain. 

Comment 6 Andreas Jaeger 2007-09-06 16:04:23 UTC
And flash-player does indeed require libfontconfig.so.1

So, it might be a libzypp bug.  Pavol, can you add the y2logs?
Comment 7 Dirk Mueller 2007-09-06 16:22:49 UTC
the logs are already attached
Comment 8 Stefan Schubert 2007-09-10 15:10:36 UTC
Back to the root:
Error : libexpat1-32bit will not be installed.

libexpat1-32bit provides
-libexpat.so.1
-libexpat1-32bit == 2.0.1-21

libexpat1-32bit == 2.0.1-21 will not be required by anyone.

libexpat.so.1 will be required by many other packages BUT
libexpat.so.1 will be provided by expat-32bit too which is already installed.

So the requiremnt is already fulfilled. I cannot see a libzypp problem here.

I do not know which package is wrong. I try the maintainer of the firstone:-)
Comment 9 Pavol Rusnak 2007-09-11 08:45:55 UTC
Added "Obsoletes: expat < 2.0.1" to libexpat1 subpackage. Hope that solves the problem. Closing.
Comment 10 Dirk Mueller 2007-09-11 12:21:02 UTC
how does "obsoletes expat " obsolete the package "expat-32bit" ?

Comment 11 Pavol Rusnak 2007-09-11 13:02:17 UTC
I thought, that if I add "Obsoletes: expat < 2.0.1" to libexpat1 it solves all 3 cases:

a) libexpat1 obsoletes expat on 32-bit archs
b) libexpat1 obsoletes expat on 64-bit archs
c) libexpat1-32bit obsoletes expat-32bit on 64-bit archs

I discussed this issue with Michal Marek and he enlightened me that case c) must be done somehow internally in autobuild.

Adding Autobuild guys to CC, maybe they will explain further.
Comment 12 Ruediger Oertel 2007-09-11 13:26:41 UTC
adding "Obsoletes: expat" to libexpat1 is wrong as long as the package
expat still exists (and it does, containing docu files).

even if it would be correct:
 no there is no automatic for (c) ...

but: since there is no expat-32bit anymore, we can add the obsoletes
from (c) to the baselibs.conf file so this can be fixed.

fixed in config, will be fixed with next rebuild of expat package

Comment 13 Pavol Rusnak 2007-09-11 13:33:45 UTC
Ah OK, removed last submission. Thanks for adding fix to autobuild config.