ancient-vegetable-10556
01/06/2023, 4:55 PMancient-vegetable-10556
01/06/2023, 5:04 PMworkdir= with a common parent directory, and then have the process itself chdir into the working directory it actually wants to work from.
The manual workaround is possible for experimental_shell_command, because it’s possible to call cd as part of your command string. It’s less obvious that you can do it from experimental_run_in_sandbox , though most languages can let you do it.
I’m wondering if there’s scope for an output_root_directory on Process ? This would solve the problem here by allowing the output_root_directory to be the build root, and the workdir to be the BUILD file directory.
(ping @witty-crayon-22786)witty-crayon-22786
01/06/2023, 5:14 PMwitty-crayon-22786
01/06/2023, 5:14 PMwitty-crayon-22786
01/06/2023, 5:15 PMancient-vegetable-10556
01/06/2023, 5:17 PMProcess with workdir=None and it calls bash -c "chdir {NEW_WORKDIR} && {shlex.join(original_argv)}ancient-vegetable-10556
01/06/2023, 5:17 PMoutput_dir correct after the process completes)witty-crayon-22786
01/06/2023, 5:21 PMworkdir argument be clear? it seems like maybe some wouldn’t be legal (or sensical at least)…ancient-vegetable-10556
01/06/2023, 5:25 PMoutput_files to .. -ascend higher than `workdir`; in this case, the Process implementation would notice if you ..-ascended higher than the buildrootancient-vegetable-10556
01/06/2023, 5:26 PMesc and erisancient-vegetable-10556
01/06/2023, 7:05 PM