Skip to content

Fix unzip with sandboxing disabled - #570

Closed
keith wants to merge 1 commit into
bazelbuild:mainfrom
keith:ks/fix-unzip-with-sandboxing-disabled
Closed

keith wants to merge 1 commit into
bazelbuild:mainfrom
keith:ks/fix-unzip-with-sandboxing-disabled

Conversation

@keith

@keith keith commented Oct 6, 2026

Copy link
Copy Markdown
Member

On Windows where sandboxing is off by default I saw:

ERROR:
D:/...//src/tools/java/com/google/devtools/build/android/proto:Configuration_proto_file
...
replace Configuration.proto? [y]es, [n]o, [A]ll, [N]one, [r]ename:  NULL
(EOF or read error, treating as "[N]one" ...)

Instead we now put the output directly into the output file

@ted-xie

ted-xie commented Oct 21, 2026

Copy link
Copy Markdown
Contributor

Would you mind rebasing so we can get a fresh presubmit run?

On Windows where sandboxing is off by default I saw:

```
ERROR:
D:/...//src/tools/java/com/google/devtools/build/android/proto:Configuration_proto_file
...
replace Configuration.proto? [y]es, [n]o, [A]ll, [N]one, [r]ename:  NULL
(EOF or read error, treating as "[N]one" ...)
```

Instead we now put the output directly into the output file
@keith
keith force-pushed the ks/fix-unzip-with-sandboxing-disabled branch from 43ea6df to 75a0738 Compare October 22, 2026 07:45
@keith

keith commented Oct 22, 2026

Copy link
Copy Markdown
Member Author

rebased!

@copybara-service copybara-service Bot closed this in 600cc93 Nov 4, 2026
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.

2 participants