Skip to content

fix(action-plugin): connect helpers from different directories - #16385

Open
rgrinberg wants to merge 1 commit into
ocaml:mainfrom
rgrinberg:push-nttlvsztyvso
Open

rgrinberg wants to merge 1 commit into
ocaml:mainfrom
rgrinberg:push-nttlvsztyvso

Conversation

@rgrinberg

@rgrinberg rgrinberg commented Sep 12, 2026

Copy link
Copy Markdown
Member

A dynamic action can launch a subprocess in another working directory. That subprocess inherits DUNE_RPC, but an address relative to the parent’s launch directory no longer locates the server.

Advertise an absolute RPC endpoint, then shorten Unix socket addresses relative to each client’s cwd at discovery time to preserve the socket path-length budget. Document that clients must connect before changing directories again, and flip the existing helper regression from failure to success.

Follow-up to #16384. Related to #5681.

Advertise an absolute RPC endpoint so subprocesses started in another
working directory can connect using the inherited DUNE_RPC environment.
Relativize Unix socket addresses at client discovery time to preserve
short socket paths, and document the requirement to connect before
changing directories again.

Flip the existing helper regression from failure to success.

Signed-off-by: Rudi Grinberg <me@rgrinberg.com>
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.

1 participant