Bug 125436 – [GTK] run-webkit-tests may DOS the system, specially in debug builds

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Gustavo Noronha (kov)]

[GTK] run-webkit-tests may DOS the system, specially in debug builds

[Gustavo Noronha (kov)]

Created attachment 218735 [details]
Patch for landing

[WebKit Commit Bot]

Comment on attachment 218735 [details]
Patch for landing

Clearing flags on attachment: 218735

Committed r160304: <http://trac.webkit.org/changeset/160304>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Carlos Garcia Campos]

This has caused other problems unfortunately. The other day I noticed that my ephy was using the webkit library in my source tree and I hadn't run make install. I've been investigating the problem and I've noticed that my source tree is hardcoded in the rpath of the web process and plugin process executables installed in my jhbuild:

$ objdump -p /home/cgarcia/gnome/libexec/WebKitWebProcess | grep RPATH
  RPATH                /home/cgarcia/src/git/gnome/WebKit/WebKitBuild/Release/.libs:/home/cgarcia/gnome/lib64

That explains why the non installed build sources are used by my ephy instead of the installed ones. The problem is that when using --no-fast-install, the binaries are not relinked when installed. I've tried using --disable-fast-install configure option, but then it fails when trying to relink binaries not using the --no-fast-install option. 

For now I've reverted this patch locally, but maybe we can use --no-fast-install for all binaries that are going to be installed and use --disable-fast-install configure option unconditionally. Or maybe there's a way to tell libtool to not do fast-install for binaries linked with --no-fast-install ld flag. I've tried manually adding fast_install=no to the WebKitWebProcess librool wrapper script and it worked, correctly relinking the command before installing it, but I don't know if that can be done automatically somehow.

Btw, the no-fast-install ld flag was already removed by Gustavo in r116205.

[Gustavo Noronha (kov)]

Maybe we can remove the rpath instead? It's what Debian does for all binaries.

[Carlos Garcia Campos]

(In reply to comment #5)
> Maybe we can remove the rpath instead? It's what Debian does for all binaries.

If that's enough and you don't need to relink. How can we do that while installing?

[Carlos Garcia Campos]

I've tried by disabling fast install globally and the build is a lot slower :-(

[Carlos Garcia Campos]

(In reply to comment #7)
> I've tried by disabling fast install globally and the build is a lot slower :-(

Was doing it wrongly, using LT_INIT, added AC_DISABLE_FAST_INSTALL to Source/autotools/SetupLibtool.m4 and it doesn't affect to the build times, as expected.

[Carlos Garcia Campos]

See https://bugs.webkit.org/show_bug.cgi?id=126381