Bug 284502
| Summary: | REGRESSION(265294@main): [GTK][WPE] WTR is unable to find the fonts.conf file if the top-level compile-time and run-time directories are different | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Carlos Alberto Lopez Perez <clopez> |
| Component: | WebKitGTK | Assignee: | Carlos Alberto Lopez Perez <clopez> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | bugs-noreply, cgarcia, vitaly |
| Priority: | P2 | ||
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| See Also: | https://bugs.webkit.org/show_bug.cgi?id=258202 | ||
Carlos Alberto Lopez Perez
This has been detected on some downstream bots that I'm testing were we don't use flatpak.
It happens that we hardcode at build-time the path to the webkit top level dir as a compiler definition (TOP_LEVEL_DIR) and then we use that to find the fonts.conf file at TOP_LEVEL_DIR/Tools/WebKitTestRunner/glib/fonts/fonts.conf
The problem happens when we use that compilation on another checkout of webkit that has a different path.
Example:
bot1 -> Builds WebKit at /home/bot1/WebKit and hardcodes TOP_LEVEL_DIR as /home/bot1/WebKit
bot2 -> Downloads built-product from bot1 at /home/bot2/WebKit and tries to run layout tests. Then it will try to access font.conf file at /home/bot1/WebKit/Tools/WebKitTestRunner/glib/fonts/fonts.conf but that path is invalid on this bot.
This was working before 265294@main because we were using relative paths.
It is fine using absolute path, but we have to detect the path at run-time rather than rely only on the path that was used at compile-time.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Carlos Alberto Lopez Perez
With flatpak this is not an issue because inside the flatpak env everyone gets the same working dir at /app
Carlos Alberto Lopez Perez
Pull request: https://github.com/WebKit/WebKit/pull/37814
EWS
Committed 287735@main (d8bd0466a815): <https://commits.webkit.org/287735@main>
Reviewed commits have been landed. Closing PR #37814 and removing active labels.