Skip to content

[Build] Apply _spawn_patch.py to grpcio_tools#37074

Closed
veblush wants to merge 1 commit intogrpc:masterfrom
veblush:grpcio_build
Closed

[Build] Apply _spawn_patch.py to grpcio_tools#37074
veblush wants to merge 1 commit intogrpc:masterfrom
veblush:grpcio_build

Conversation

@veblush
Copy link
Copy Markdown
Contributor

@veblush veblush commented Jun 27, 2024

grpcio_tool now needs _spawn_patch.py to workaround the length limit of commandline on Windows. This is a prerequisite for #36753.

@veblush veblush added lang/Python area/build release notes: no Indicates if PR should not be in release notes labels Jun 27, 2024
@veblush veblush requested a review from XuanWang-Amos June 27, 2024 17:47
@veblush veblush changed the title [Build] Appy _spawn_patch.py to grpcio_tools [Build] Apply _spawn_patch.py to grpcio_tools Jun 27, 2024
Copy link
Copy Markdown
Contributor

@XuanWang-Amos XuanWang-Amos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks for working on this!

alto-ruby added a commit to alto-ruby/grpc that referenced this pull request Jul 11, 2024
paulosjca pushed a commit to paulosjca/grpc that referenced this pull request Nov 25, 2024
grpcio_tool now needs `_spawn_patch.py` to workaround the length limit of commandline on Windows. This is a prerequisite for grpc#36753.

Closes grpc#37074

COPYBARA_INTEGRATE_REVIEW=grpc#37074 from veblush:grpcio_build 9ba7b44
PiperOrigin-RevId: 647405460
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.

2 participants