Skip to content

Revert "[CSHARP] Add base_namespace experimental option to C# plugin"#32957

Merged
veblush merged 1 commit intomasterfrom
revert-32636-grpc_base_namespace
Apr 27, 2023
Merged

Revert "[CSHARP] Add base_namespace experimental option to C# plugin"#32957
veblush merged 1 commit intomasterfrom
revert-32636-grpc_base_namespace

Conversation

@drfloob
Copy link
Copy Markdown
Member

@drfloob drfloob commented Apr 27, 2023

Reverts #32636

src/compiler/csharp_generator_helpers.h:25:7: error: no member named 'compiler' in namespace ...
src/compiler/csharp_generator_helpers.h:25:25: error: no member named 'csharp' in namespace 'compiler' ...

@apolcyn
Copy link
Copy Markdown
Contributor

apolcyn commented Apr 27, 2023

cc @tonydnewell FYI

@drfloob drfloob added the release notes: no Indicates if PR should not be in release notes label Apr 27, 2023
@drfloob drfloob enabled auto-merge (squash) April 27, 2023 19:52
@veblush veblush disabled auto-merge April 27, 2023 19:52
@veblush veblush merged commit d63f8d4 into master Apr 27, 2023
@copybara-service copybara-service bot added the imported Specifies if the PR has been imported to the internal repository label Apr 28, 2023
XuanWang-Amos pushed a commit to XuanWang-Amos/grpc that referenced this pull request May 1, 2023
…grpc#32957)

Reverts grpc#32636

```
src/compiler/csharp_generator_helpers.h:25:7: error: no member named 'compiler' in namespace ...
src/compiler/csharp_generator_helpers.h:25:25: error: no member named 'csharp' in namespace 'compiler' ...
```
paulosjca pushed a commit to paulosjca/grpc that referenced this pull request May 4, 2023
…grpc#32957)

Reverts grpc#32636

```
src/compiler/csharp_generator_helpers.h:25:7: error: no member named 'compiler' in namespace ...
src/compiler/csharp_generator_helpers.h:25:25: error: no member named 'csharp' in namespace 'compiler' ...
```
wanlin31 pushed a commit that referenced this pull request May 18, 2023
…#32957)

Reverts #32636

```
src/compiler/csharp_generator_helpers.h:25:7: error: no member named 'compiler' in namespace ...
src/compiler/csharp_generator_helpers.h:25:25: error: no member named 'csharp' in namespace 'compiler' ...
```
tonydnewell added a commit to tonydnewell/grpc that referenced this pull request Jun 1, 2023
jtattermusch pushed a commit to jtattermusch/grpc that referenced this pull request Jun 23, 2023
jtattermusch pushed a commit to jtattermusch/grpc that referenced this pull request Jun 26, 2023
jtattermusch added a commit that referenced this pull request Jun 26, 2023
… patch) (#33535)

Reintroduces patched version of #32636
(which was reverted in #32957).

Together with cl/542843305, the internal build should work fine.

Supersedes #33310 (since a patch is
also needed).

---------

Co-authored-by: tony <tony.newell@pobox.com>
mario-vimal pushed a commit to mario-vimal/grpc that referenced this pull request Jul 13, 2023
… patch) (grpc#33535)

Reintroduces patched version of grpc#32636
(which was reverted in grpc#32957).

Together with cl/542843305, the internal build should work fine.

Supersedes grpc#33310 (since a patch is
also needed).

---------

Co-authored-by: tony <tony.newell@pobox.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bloat/none imported Specifies if the PR has been imported to the internal repository lang/C# per-call-memory/neutral per-channel-memory/neutral 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.

3 participants