Skip to content
This repository has been archived by the owner on Jul 15, 2024. It is now read-only.

fix(java): exclude unused dependencies from gax-grpc #721

Merged
merged 2 commits into from
Aug 31, 2022
Merged

Conversation

mpeddada1
Copy link
Contributor

@mpeddada1 mpeddada1 commented Aug 29, 2022

Gax-grpc contains some native-image configurations that are needed for native-image compilation to be successful (see #688). However, adding the gax-grpc dependency also transitively brings in it's own dependencies which are not required for either standard Java or native image users. This can result in some side effects including the one seen in googleapis/gax-java#1781. This PR excludes those dependencies to ensure that only the necessary code is brought in.

Reproducer: https://github.com/mpeddada1/test-gax/tree/main

@product-auto-label product-auto-label bot added size: m Pull request size is medium. api: compute Issues related to the googleapis/java-compute API. labels Aug 29, 2022
@mpeddada1 mpeddada1 marked this pull request as ready for review August 30, 2022 05:59
@mpeddada1 mpeddada1 requested a review from a team as a code owner August 30, 2022 05:59
@mpeddada1 mpeddada1 requested review from suztomo and meltsufin August 30, 2022 05:59
@mpeddada1 mpeddada1 added the owlbot:run Add this label to trigger the Owlbot post processor. label Aug 30, 2022
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Aug 30, 2022
@mpeddada1 mpeddada1 merged commit 4da7a1f into main Aug 31, 2022
@mpeddada1 mpeddada1 deleted the unused-deps branch August 31, 2022 14:14
gcf-merge-on-green bot pushed a commit that referenced this pull request Sep 16, 2022
🤖 I have created a release *beep* *boop*
---


## [1.12.1](v1.12.0...v1.12.1) (2022-09-15)


### Bug Fixes

* **java:** Exclude unused dependencies from gax-grpc ([#721](#721)) ([4da7a1f](4da7a1f))


### Dependencies

* Update dependency com.google.cloud:google-cloud-shared-dependencies to v3.0.2 ([#724](#724)) ([2b41e83](2b41e83))
* Update dependency com.google.cloud:google-cloud-shared-dependencies to v3.0.3 ([#727](#727)) ([63bfcce](63bfcce))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
github-actions bot pushed a commit to meltsufin/monorepo-experiments that referenced this pull request Sep 17, 2022
🤖 I have created a release *beep* *boop*
---


## [1.12.1](googleapis/java-compute@v1.12.0...v1.12.1) (2022-09-15)


### Bug Fixes

* **java:** Exclude unused dependencies from gax-grpc ([#721](googleapis/java-compute#721)) ([b6b4e15](googleapis/java-compute@b6b4e15))


### Dependencies

* Update dependency com.google.cloud:google-cloud-shared-dependencies to v3.0.2 ([#724](googleapis/java-compute#724)) ([374a16b](googleapis/java-compute@374a16b))
* Update dependency com.google.cloud:google-cloud-shared-dependencies to v3.0.3 ([#727](googleapis/java-compute#727)) ([fc748b7](googleapis/java-compute@fc748b7))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
api: compute Issues related to the googleapis/java-compute API. size: m Pull request size is medium.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants