Skip to content

Add a new ExecFactory extension point to support alternative processes #1763

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Mar 9, 2025

Conversation

laeubi
Copy link
Contributor

@laeubi laeubi commented Mar 9, 2025

Currently we already have a ProcessFactory that allows the creation of alternative IProcess for a launch, but something similar for creation of the native process is currently not possible.

This adds a new extension point to supply an ExecFactory to allow controlling the creation of native processes. The use case for this might be to launch inside a terminal session, but also using a chroot environment or containerization would be possible.

See:

Currently we already have a ProcessFactory that allows the creation of
alternative IProcess for a launch, but something similar for creation of
the native process is currently not possible.

This adds a new extension point to supply an ExecFactory to allow
controlling the creation of native processes. The use case for this
might be to launch inside a terminal session, but also using a chroot
environment or containerization would be possible.
@laeubi laeubi force-pushed the add_execfactory_ext_point branch from 8377a0f to 7af000d Compare March 9, 2025 15:21
Copy link
Contributor

github-actions bot commented Mar 9, 2025

Test Results

 1 600 files  + 47   1 600 suites  +47   1h 23m 28s ⏱️ - 5m 10s
 4 173 tests ±  0   4 149 ✅  -   1   23 💤 ±0  1 ❌ +1 
12 002 runs  +200  11 835 ✅ +198  166 💤 +1  1 ❌ +1 

For more details on these failures, see this check.

Results for commit 7af000d. ± Comparison against base commit 0d9f518.

@laeubi laeubi merged commit 67a55ce into eclipse-platform:master Mar 9, 2025
14 of 17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant