Bug 219354

This look strange anyhow.  Stanislav, any ideas why it asks for mozilla-xulrunner (Without version!)?

This is hardcoded in /usr/share/YaST2/clients/inst_suse_register.ycp.

It's a pure YaST problem and does not have anything to do with the xulrunner packaging (except that it doesn't follow the packaging changes which have been done when xulrunner was changed to allow multiple installs).

Just for clarity, this is not related to the recent xulrunner packaging changes, the problem is there since xulrunner{180,181} exist because YaST needs to know the exact package name. Provides is not sufficient.

/usr/share/YaST2/clients/inst_suse_register.ycp does:

Package::InstallAllMsg(["mozilla-xulrunner"], nil);

This needs to be the exact package name, i.e. either mozilla-xulrunner180 or mozilla-xulrunner181.

Note that the packages *do* provide mozilla-xulrunner without version, but that is not sufficient because Package::InstallAllMsg takes the exact package name only.

You need it, if you enable the "registration" checkbox. This is to register for installation support. 

IIRC I did uncheck all boxes at installation and it still wanted to start a browser... ?

Ok, please attache the logs. 

/var/log/YaST2/*
/var/log/messages
/root/.suse_register.log

Created attachment 104485 [details]
The requested log files

Ok, the Problem of Comment 5 is fixed and submitted to stable. 
The other (initial) bug is still valid.

All related gnome packages currently use:

%define xulrunner_version 181

%if %suse_version < 1000
BuildRequires:  mozilla-devel
%define mozilla_ver %(rpm -q --qf '%%{version}' mozilla)
Requires:       mozilla = %{mozilla_ver}
%endif
%if %suse_version > 1000 && %suse_version < 1010
BuildRequires:  mozilla-xulrunner-devel
%define mozilla_ver %(rpm -q --qf '%%{version}' mozilla-xulrunner)
Requires:       mozilla-xulrunner = %{mozilla_ver}
%endif
%if %suse_version > 1010
BuildRequires:  mozilla-xulrunner%{xulrunner_version}-devel
%define mozilla_ver %(rpm -q --qf '%%{version}' mozilla-xulrunner%{xulrunner_version})
Requires:       mozilla-xulrunner%{xulrunner_version} = %{mozilla_ver}
%endif

or followins with less strict version check:

%if %suse_version < 1000
BuildRequires:  gecko-sdk
Requires:       mozilla
%endif
%if %suse_version > 1000 && %suse_version < 1010
BuildRequires:  mozilla-xulrunner-devel
Requires:       mozilla-xulrunner
%endif
%if %suse_version > 1010
BuildRequires:  mozilla-xulrunner%{xulrunner_version}-devel
Requires:       mozilla-xulrunner%{xulrunner_version}
%endif

Still in Beta 2 Plus: It pops up when I want to click the URL referring to the privacy policy.

IMO it is a bug that Package::InstallAllMsg(["mozilla-xulrunner"], nil) only works on package names and not on provides (or we should create a new binding for that, whatever).

Workaround: click cancel. (Then it does not work because of an unrelated bug)

(In reply to comment #10)
> (Then it does not work because of an unrelated bug)

Bug 221227 reported.

Sorry for disturbing. I don't know what's planned for final.
Looks like there will be mozilla-xulrunner180 and mozilla-xulrunner181 on 10.2?
If so, please note that the browser of the registration module _needs_ mozilla-xulrunner180. The current XUL doesn't work with 1.8.1.
And if somebody is interested in future having a version for 1.8.1 I think I made one when I was still employed and still have it on some DVD here ;-)

Part of GNOME was using 180, part was using 181, part was using 180 but required 181, part was using 181 but required 180. Now all require 181.

Most of mentioned packages use only gecko (or even inherit dependency from another package), the rest should have 181 support (e. g. epiphany).

It woul be nice to have 181 for everything and don't require 180.

sbrabec@chip:~> whatdependson mozilla-xulrunner180
- no dependent package found
sbrabec@chip:~> whatdependson mozilla-xulrunner181
- OpenOffice_org (pmladek@suse.cz)
- beagle (gnome-maintainers@suse.de)
- beagle-index (gnome-maintainers@suse.de)
- blam (gnome-maintainers@suse.de)
- classpath (dbornkessel@suse.de)
- collect-desktop-files (coolo@suse.de)
- devhelp (gnome-maintainers@suse.de)
- epiphany (gnome-maintainers@suse.de)
- epiphany-extensions (gnome-maintainers@suse.de)
- galago-gtk-sharp (gnome-maintainers@suse.de)
- gecko-sharp (mono-maintainers@suse.de)
- gecko-sharp2 (mono-maintainers@suse.de)
- gecko-sharp2-docs (mono-maintainers@suse.de)
- gedit (gnome-maintainers@suse.de)
- gnome-python-desktop (gnome-maintainers@suse.de)
- gnome-web-photo (gnome-maintainers@suse.de)
- gtksourceview-sharp2 (mono-maintainers@suse.de)
- gwget (lmichnovic@suse.cz)
- ifolder3 (srinidhi@novell.com)
- ipod-sharp (gnome-maintainers@suse.de)
- liferea (gnome-maintainers@suse.de)
- mono-tools (mono-maintainers@suse.de)
- monodevelop (mono-maintainers@suse.de)
- mozilla-bonobo (gnome-maintainers@suse.de)
- mplayerplug-in (cthiel@suse.de)
- nautilus-ifolder3 (jhargadon@novell.com)
- njb-sharp (gnome-maintainers@suse.de)
- novell-iprint-xclient (mjcowley@novell.com)
- python-gnome-extras (gnome-maintainers@suse.de)
- rhythmbox (gnome-maintainers@suse.de)
- totem (gnome-maintainers@suse.de)
- yelp (gnome-maintainers@suse.de)
  maintainers: coolo@suse.de, cthiel@suse.de, dbornkessel@suse.de, gnome-maintainers@suse.de, jhargadon@novell.com, lmichnovic@suse.cz, mjcowley@novell.com, mono-maintainers@suse.de, pmladek@suse.cz, srinidhi@novell.com

(In reply to comment #13)
> sbrabec@chip:~> whatdependson mozilla-xulrunner180
> - no dependent package found

As Wolfgang said (thanks!), yast now needs 180. It is not visible to you because the dependency is not in the spec file (Requires) but in YCP code for on-the-fly installation (comment 3).

Perhaps we should make a policy to put these dynamic dependencies to .spec as Suggests.

> MAs Wolfgang said (thanks!), yast now needs 180
Martin: Not 180 - 181, note the 1.

(In reply to comment #15)
> Martin: Not 180 - 181, note the 1.
 
If you require 181 please make sure the browser works.
As stated above if it's still the old version I wrote it wouldn't work with 1.8.1.
If you want to use 1.8.1 I can attach a version which should work.

I thought we would use 181 everywhere and drop 181.  Yes, it should be tested and then would need a fixed version.

Created attachment 105564 [details]
current browser app

This is the version I had in my archive and IIRC it worked. To give you a chance to test ASAP I attach this one currently untested by me. If it doesn't work please open a new bug about it and assign it to me if needed.

(In reply to comment #18)
> To give you a
> chance to test ASAP I attach this one currently untested by me.

It works great - thanks a lot Wolfgang!

*** Bug 221227 has been marked as a duplicate of this bug. ***

fixed in yast2-registration 2.14.1 and submitted

The yastbrowser now works with mozilla-xulrunner181 !