Bug 1505747

(In reply to Tomas Pelka from comment #0)
> rebase gjs to 1.47

We should do rather 1.50 as 3.26 gnome-shell wont work with 1.47.

Created attachment 1345347 [details]
journal.log

Ok this is not good even with new cogl, clutter clutter-gtk.

Attaching journactl and will attach screenshot.

Created attachment 1345350 [details]
screenshot

Just point out that even gdm is not staring, it starts when I update cogl/clutter/clutter-gtk/gjs from COPR.

After gdm start and one trie to start session, than see the screenshot

Finally found config that works

cogl-1.22.2-6.1.el7_4.x86_64
clutter-1.26.2-3.1.el7_4.x86_64
clutter-gtk-1.8.4-1.el7_4.x86_64
gjs-1.50.0-1.el7_4.x86_64
mozjs52-52.3.0-1.el7_4.x86_64

Other pkgs are from RHEL (Brew)

I'm afraid we would need 1.50 gjs, Kalev should be on it now.

1.50.1 build with bundled mozjs52 is under way: https://brewweb.engineering.redhat.com/brew/taskinfo?taskID=14424097

Well with 1.50 it was still broken, which was weird. So Vlad (thanks vbenes!) had an idea that due to following error

 Oct 30 09:33:02 localhost.localdomain gnome-shell[15854]: JS ERROR: Error: Could not find definition of virtual function observe_channels

still appears in journactl we might need to rebuild telepathy-glib against new glib2-2.54 that we introduced as dep for mutter might help. And guess what it helped.

So I'm going to ask richi to rebuild telepathy and also I'm going to try downgrade to gjs-1.47 to avoid bundling mozjs52 into newer gjs. Unless you think it is beneficial to have new gjs in rhel7.

-Tom

I'd feel much more confident shipping 1.50 than 1.47, as 1.47.0 is a development snapshot and not a final stable release. Also, gjs 1.50 together with gnome-shell 3.26 is the combination we shipped in F27 and known to work well.

(In reply to Tomas Pelka from comment #13)
> Well with 1.50 it was still broken, which was weird. So Vlad (thanks
> vbenes!) had an idea that due to following error
> 
>  Oct 30 09:33:02 localhost.localdomain gnome-shell[15854]: JS ERROR: Error:
> Could not find definition of virtual function observe_channels
> 
> still appears in journactl we might need to rebuild telepathy-glib against
> new glib2-2.54 that we introduced as dep for mutter might help. And guess
> what it helped.
> 
> So I'm going to ask richi to rebuild telepathy and also I'm going to try
> downgrade to gjs-1.47 to avoid bundling mozjs52 into newer gjs. Unless you
> think it is beneficial to have new gjs in rhel7.
> 
> -Tom

OK with gjs-1.47 gdm wont start, so your work Kalev was not useless. So I guess we will keep 1.50 version. Regarding "unbundling" mozjs52 I would propose to open new bugs for including it as standalone rpm in 7.6 and also updating gjs to use this system version instead of the bundled on in 7.6 too. What do you think?

Sure, I think that's a good plan, but I'd wait a little bit with 7.6 bugs until we know what version of gnome-shell/gjs we'll be shipping there. It could be that we'll end up shipping a newer gjs version that needs even newer mozjs than mozjs52.

But if it's at all possible at this point to get mozjs52 into 7.5 I'd like that very much to clean things up a bit.

Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2018:0770