feat(launchdoctor): detect missing libraries for dlopen
#3202
Conversation
WebKit WPE assumes `libglesv2.so` is available on the host system and uses `dlopen` to open it. This patch starts using `ldconfig -p` to check if the library exists on the system. References microsoft#2745
| LD_LIBRARY_PATH: process.env.LD_LIBRARY_PATH ? `${process.env.LD_LIBRARY_PATH}:${dirname}` : dirname, | ||
| }, | ||
| }); | ||
| if (code !== 0) |
There was a problem hiding this comment.
We should throw if it failed otherwise it's indistinguishable from the case when all dependencies are good.
There was a problem hiding this comment.
ldd fails if we feed in a non-ELF file (and we might occasionally do so).
We should throw if it failed otherwise it's indistinguishable from the case when all dependencies are good.
If our spawn fails for some reason, i'd still try launching to avoid false negatives.
There was a problem hiding this comment.
That's unfortunate, perhaps we could run this checks after launch failed? Don't remember what was the rationale for running the checks before launch.
There was a problem hiding this comment.
The reason is that there might be no browser crash at all. In case of missing libglesv2 the browsers won't crash - instead, renderer will either misbehave (on ubuntu 18.04) or crash (on ubuntu 20.04)
| if (!libraries.length) | ||
| return []; | ||
| const {stdout, code} = await spawnAsync('ldconfig', ['-p'], {}); | ||
| if (code !== 0) |
2 participants
WebKit WPE assumes
libglesv2.sois available on the host systemand uses
dlopento open it.This patch starts using
ldconfig -pto check if the libraryexists on the system.
References #2745