Bug 220250 - [WebXR][WPE] Build fails without openxr installed
Summary: [WebXR][WPE] Build fails without openxr installed
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebXR (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Lauro Moura
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-01-04 11:59 PST by Lauro Moura
Modified: 2021-12-09 12:36 PST (History)
12 users (show)

See Also:


Attachments
Patch (3.69 KB, patch)
2021-01-04 12:10 PST, Lauro Moura
no flags Details | Formatted Diff | Diff
Patch for landing (3.75 KB, patch)
2021-01-04 14:17 PST, Lauro Moura
no flags Details | Formatted Diff | Diff
Patch (1.64 KB, patch)
2021-01-05 15:03 PST, Yury Semikhatsky
no flags Details | Formatted Diff | Diff
[fast-cq] Updated patch after rebase (1.18 KB, patch)
2021-12-09 12:10 PST, Lauro Moura
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Lauro Moura 2021-01-04 11:59:53 PST
The WPE Nightly Packaging bot is failing as WebXR is enabled (through ENABLE_EXPERIMENTAL_FEATURES) but openxr is not installed by jhbuild.

Also, instead of failing early, it fails late in the compilation after r270067 changed some defines.
Comment 1 Lauro Moura 2021-01-04 12:10:22 PST
Created attachment 416958 [details]
Patch
Comment 2 Carlos Alberto Lopez Perez 2021-01-04 12:25:24 PST
Comment on attachment 416958 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=416958&action=review

> Tools/jhbuild/jhbuild-minimal.modules:125
> +  <cmake id="openxr">

Please add here below a pkg-config line like this:
<pkg-config>openxr.pc</pkg-config>

This will tell JHBuild to only build openxr if the system doesn't have it already installed
Comment 3 Lauro Moura 2021-01-04 14:17:52 PST
Created attachment 416964 [details]
Patch for landing
Comment 4 EWS 2021-01-04 14:55:14 PST
Committed r271138: <https://trac.webkit.org/changeset/271138>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 416964 [details].
Comment 5 Yury Semikhatsky 2021-01-05 14:46:02 PST
Looks like this broke jhbuild-minimal, I get a bunch of errors like this on Ubuntu 20.04:

*** Building openxr *** [4/5]                                                                                                                                                                                                                                                     
ninja                                                                                                                                                                                                                                                                             
[3/18] Building C object src/loader/CMakeFiles/openxr_loader.dir/__/xr_generated_dispatch_table.c.o                                                                                                                                                                               
FAILED: src/loader/CMakeFiles/openxr_loader.dir/__/xr_generated_dispatch_table.c.o                                                                                                                                                                                                
/usr/lib/ccache/cc -DAPI_NAME=\"OpenXR\" -DDISABLE_STD_FILESYSTEM -DEXTRASYSCONFDIR=\"/etc\" -DFALLBACK_CONFIG_DIRS=\"/etc/xdg\" -DFALLBACK_DATA_DIRS=\"/usr/local/share:/usr/share\" -DOPENXR_HAVE_COMMON_CONFIG -DSYSCONFDIR=\"/home/yurys/playwright/browser_patches/webkit/che
ckout/WebKitBuild/WPE/DependenciesWPE/Root/etc\" -DXRAPI_DLL_EXPORT -DXR_OS_LINUX -DXR_USE_GRAPHICS_API_OPENGL -DXR_USE_PLATFORM_WAYLAND -DXR_USE_PLATFORM_XCB -DXR_USE_PLATFORM_XLIB -DXR_USE_TIMESPEC -Dopenxr_loader_EXPORTS -I/home/yurys/playwright/browser_patches/webkit/ch
eckout/WebKitBuild/WPE/DependenciesWPE/Source/openxr-1.0.11/src/../include -I/home/yurys/playwright/browser_patches/webkit/checkout/WebKitBuild/WPE/DependenciesWPE/Source/openxr-1.0.11/src -I/home/yurys/playwright/browser_patches/webkit/checkout/WebKitBuild/WPE/Dependencies
WPE/Source/openxr-1.0.11/src/external/jsoncpp/include -I/home/yurys/playwright/browser_patches/webkit/checkout/WebKitBuild/WPE/DependenciesWPE/Source/openxr-1.0.11/src/common -I/home/yurys/playwright/browser_patches/webkit/checkout/WebKitBuild/WPE/DependenciesWPE/Source/ope
nxr-1.0.11/include -Iinclude -I/home/yurys/playwright/browser_patches/webkit/checkout/WebKitBuild/WPE/DependenciesWPE/Source/openxr-1.0.11/src/loader/.. -Isrc/loader/.. -I/home/yurys/playwright/browser_patches/webkit/checkout/WebKitBuild/WPE/DependenciesWPE/Source/openxr-1.
0.11/src/loader -Isrc/loader -Wall -Werror=unused-parameter -Werror=unused-argument -Wpointer-arith -fPIC -fvisibility=hidden   -Wno-unused-parameter -Wextra -fno-strict-aliasing -fno-builtin-memcmp -ffunction-sections -fdata-sections -MD -MT src/loader/CMakeFiles/openxr_lo
ader.dir/__/xr_generated_dispatch_table.c.o -MF src/loader/CMakeFiles/openxr_loader.dir/__/xr_generated_dispatch_table.c.o.d -o src/loader/CMakeFiles/openxr_loader.dir/__/xr_generated_dispatch_table.c.o   -c /home/yurys/playwright/browser_patches/webkit/checkout/WebKitBuild
/WPE/DependenciesWPE/Source/openxr-1.0.11/src/xr_generated_dispatch_table.c
In file included from /home/yurys/playwright/browser_patches/webkit/checkout/WebKitBuild/WPE/DependenciesWPE/Source/openxr-1.0.11/src/xr_generated_dispatch_table.c:26:                                                                                                           
In file included from /home/yurys/playwright/browser_patches/webkit/checkout/WebKitBuild/WPE/DependenciesWPE/Source/openxr-1.0.11/src/xr_generated_dispatch_table.h:27:                                                                                                           
/home/yurys/playwright/browser_patches/webkit/checkout/WebKitBuild/WPE/DependenciesWPE/Source/openxr-1.0.11/src/common/xr_dependencies.h:83:10: fatal error: 'xcb/glx.h' file not found                                                                                           
#include <xcb/glx.h>                                                                                                                                                                                                                                                              
         ^~~~~~~~~~~                                                                                                                                                                                                                                                              
1 error generated.
Comment 6 Yury Semikhatsky 2021-01-05 15:03:44 PST
Created attachment 417039 [details]
Patch
Comment 7 Lauro Moura 2021-12-09 12:10:24 PST
Created attachment 446586 [details]
[fast-cq] Updated patch after rebase
Comment 8 EWS 2021-12-09 12:36:31 PST
Committed r286797 (245037@main): <https://commits.webkit.org/245037@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 446586 [details].