Skip to content

Fix C# protoc plugin argument parsing#26896

Merged
apolcyn merged 1 commit intogrpc:masterfrom
apolcyn:fix_csharp_generator
Aug 6, 2021
Merged

Fix C# protoc plugin argument parsing#26896
apolcyn merged 1 commit intogrpc:masterfrom
apolcyn:fix_csharp_generator

Conversation

@apolcyn
Copy link
Copy Markdown
Contributor

@apolcyn apolcyn commented Aug 5, 2021

Fixes #26819

#26162 caused a regression by adding the check for "file_suffix" in an if rather than else if, causing the else block to always be hit in the case that a non-"file_suffix" option is passed in.

@apolcyn
Copy link
Copy Markdown
Contributor Author

apolcyn commented Aug 6, 2021

RBE windows C++ failure hit an out of disk space error which I believe should be fixed by #26895

PHP macos: #26924

python windows: #26926

interop cloud to prod: #26652

@apolcyn apolcyn merged commit 2ea6aef into grpc:master Aug 6, 2021
@jtattermusch
Copy link
Copy Markdown
Contributor

ooops, sorry for introducing the issue & thanks for the fix.

Vignesh2208 pushed a commit to Vignesh2208/grpc that referenced this pull request Aug 20, 2021
jtattermusch pushed a commit to jtattermusch/grpc that referenced this pull request Aug 30, 2021
jtattermusch added a commit that referenced this pull request Aug 31, 2021
lidizheng pushed a commit to lidizheng/grpc that referenced this pull request Sep 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lang/C# release notes: no Indicates if PR should not be in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[C#] Grpc.Tools 2.39.0-pre1 error

3 participants