Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Build Target Reorganization Part 2 #30570

Merged
merged 14 commits into from
Aug 30, 2022

Conversation

ralphchung
Copy link
Contributor

Given that we are removing codegen related files, we need to clean up the build targets in the current build file. There will be a series of PRs for this.

The new build targets should roughly follow this dependency graph.

┌───────┐┌───────────────┐        
│grpc++ ││grpc++_unsecure│        
└┬─────┬┘└┬─────────┬────┘        
┌▽───┐┌▽──▽───────┐┌▽────────────┐
│grpc││grpc++_base││grpc_unsecure│
└┬───┘└┬──────────┘└┬────────────┘
┌▽─────▽────────────▽┐            
│grpc_base           │            
└────────────────────┘            

Detailed steps is the following.

  • (Covered in Build Target Reorganization Part 1 #30518 ) gpr_base is merged into gpr. The gpr here itself is nothing but depending on gpr_base. I don't think we need to separate them.
  • grpc++_internals and grpc++_internal_hdrs_only are merged into grpc++. The reason is similar. grpc++ does not contain much stuff but grpc++_internals.
  • (Covered in Build Target Reorganization Part 1 #30518 ) gpr_codegen is merged to gpr since the files in gpr_codegen should be removed in the future and be moved to gpr.
  • (Covered in this PR) grpc_codegen is merged to grpc_base with the similar reason as above.
  • grpc++_codegen_base and grpc++_codegen_base_src are merged to grpc++_base with the similar reason as above.
  • (Covered in this PR) grpc_secure is merged to grpc since grpc is by default secure. We do not need to separate them.
  • grpc++_base_unsecure is merged either into grpc++_base or into grpc++_unsecure.

This PR merges grpc_secure and grpc_codegen to grpc without removing grpc_codegen for temporary compatibility. It should be removed in the following PRs.

@@ -468,7 +474,6 @@ grpc_cc_library(
"//conditions:default": GRPC_XDS_TARGETS,
},
],
standalone = True,
Copy link
Member

Choose a reason for hiding this comment

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

what does this change?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@ralphchung ralphchung merged commit 0c46726 into grpc:master Aug 30, 2022
@ralphchung ralphchung deleted the build-target-reorganization branch August 30, 2022 20:13
@copybara-service copybara-service bot added the imported Specifies if the PR has been imported to the internal repository label Aug 30, 2022
@ralphchung ralphchung mentioned this pull request Sep 13, 2022
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/core 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.

None yet

2 participants