Skip to content
New issue

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

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

Already on GitHub? # to your account

Give Kotlin jars an OSGi Manifest #18812

Closed
wants to merge 3 commits into from

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.

@deannagarcia deannagarcia requested review from a team as code owners October 14, 2024 18:42
@deannagarcia deannagarcia requested review from haberman and googleberg and removed request for a team and haberman October 14, 2024 18:42
@deannagarcia deannagarcia added the 🅰️ safe for tests Mark a commit as safe to run presubmits over label Oct 14, 2024
@github-actions github-actions bot removed the 🅰️ safe for tests Mark a commit as safe to run presubmits over label Oct 14, 2024
@deannagarcia deannagarcia added the 🅰️ safe for tests Mark a commit as safe to run presubmits over label Oct 14, 2024
@github-actions github-actions bot removed the 🅰️ safe for tests Mark a commit as safe to run presubmits over label Oct 14, 2024
@deannagarcia deannagarcia added the 🅰️ safe for tests Mark a commit as safe to run presubmits over label Oct 14, 2024
@github-actions github-actions bot removed the 🅰️ safe for tests Mark a commit as safe to run presubmits over label Oct 14, 2024
@zhangskz
Copy link
Member

Did you mean for this to be a draft PR? Or is this PR ready, and meant to land as a CL once approved?

@deannagarcia deannagarcia marked this pull request as draft October 15, 2024 19:50
copybara-service bot pushed a commit that referenced this pull request Oct 15, 2024
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 #18812

COPYBARA_INTEGRATE_REVIEW=#18812 from deannagarcia:kotlinOSGi 81bab06
FUTURE_COPYBARA_INTEGRATE_REVIEW=#18812 from deannagarcia:kotlinOSGi 81bab06
PiperOrigin-RevId: 686201830
deannagarcia added a commit to deannagarcia/protobuf that referenced this pull request Oct 16, 2024
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 added a commit to deannagarcia/protobuf that referenced this pull request Oct 16, 2024
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 added a commit to deannagarcia/protobuf that referenced this pull request Oct 16, 2024
Give Kotlin jars an OSGi Manifest (protocolbuffers#18812)

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
zhangskz pushed a commit that referenced this pull request Oct 16, 2024
* Give Kotlin jars an OSGi Manifest (#18812)

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 #18812

COPYBARA_INTEGRATE_REVIEW=#18812 from deannagarcia:kotlinOSGi 81bab06
PiperOrigin-RevId: 686220820

* Use old name of rules_kotlin
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants