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

Populate Kotlin Manifest Files #18859

Merged
merged 2 commits into from
Oct 16, 2024
Merged

Conversation

deannagarcia
Copy link
Member

Extend our Java OSGi library to have a version that works for Kotlin. Add a protobuf_versioned_kt_jvm_library that creates a bundle with the OSGi manifest and call that instead of kt_jvm_library for all our kotlin maven release targets.

Cherrypick of 27ddb9c.

Extend our Java OSGi library to have a version that works for Kotlin. Add a `protobuf_versioned_kt_jvm_library` that creates a bundle with the OSGi manifest and call that instead of `kt_jvm_library` for all our kotlin maven release targets.

Closes protocolbuffers#18812

COPYBARA_INTEGRATE_REVIEW=protocolbuffers#18812 from deannagarcia:kotlinOSGi 81bab06
PiperOrigin-RevId: 686220820
@deannagarcia deannagarcia requested review from a team as code owners October 16, 2024 17:36
@deannagarcia deannagarcia requested review from mkruskal-google, googleberg and zhangskz and removed request for a team, mkruskal-google and googleberg October 16, 2024 17:36
@deannagarcia deannagarcia added the 🅰️ safe for tests Mark a commit as safe to run presubmits over label Oct 16, 2024
@github-actions github-actions bot removed the 🅰️ safe for tests Mark a commit as safe to run presubmits over label Oct 16, 2024
@deannagarcia deannagarcia added the 🅰️ safe for tests Mark a commit as safe to run presubmits over label Oct 16, 2024
@github-actions github-actions bot removed the 🅰️ safe for tests Mark a commit as safe to run presubmits over label Oct 16, 2024
@zhangskz zhangskz merged commit d82fb47 into protocolbuffers:28.x Oct 16, 2024
163 of 164 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants