Skip to content

[builtins] Allow using an already unpacked openui5 runtime distribution - #23910

Open
guitargeek wants to merge 1 commit into
root-project:masterfrom
guitargeek:openui_src
Open

guitargeek wants to merge 1 commit into
root-project:masterfrom
guitargeek:openui_src

Conversation

@guitargeek

Copy link
Copy Markdown
Contributor

Add an OPENUI5_SOURCE_DIR CMake variable, analogous to CLAD_SOURCE_DIR, that points to an already unpacked openui5 runtime distribution. With it, ROOT can be built with builtin_openui5=ON without network access, as the download via ExternalProject and the connectivity check are skipped.

FYI @linev and @ferdymercury

This is part of the effort to make sure a fully-featured ROOT can still be built in a connection-less sandbox.

Add an OPENUI5_SOURCE_DIR CMake variable, analogous to CLAD_SOURCE_DIR,
that points to an already unpacked openui5 runtime distribution. With
it, ROOT can be built with builtin_openui5=ON without network access, as
the download via ExternalProject and the connectivity check are skipped.
@linev

linev commented Oct 30, 2026

Copy link
Copy Markdown
Member

This is also resolved in #23877 - it allows to use local openui5 installation
via OPENUI5DIR shell variable or just configure ROOT to always use external openui5 - issuing warning for it

@guitargeek

Copy link
Copy Markdown
Contributor Author

Nice!

@github-actions

Copy link
Copy Markdown

Test Results

    23 files      23 suites   3d 15h 58m 42s ⏱️
 3 883 tests  3 883 ✅ 0 💤 0 ❌
80 760 runs  80 760 ✅ 0 💤 0 ❌

Results for commit 14337a1.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants