这是indexloc提供的服务,不要输入任何密码
Skip to content

kotlin-ci #638

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

Merged
merged 33 commits into from
Aug 19, 2024
Merged

kotlin-ci #638

merged 33 commits into from
Aug 19, 2024

Conversation

jcrist1
Copy link
Contributor

@jcrist1 jcrist1 commented Aug 16, 2024

Unfortunately I can't really debug the ci process other than trying to run them via PR, so I'm opening this up as a draft.

I've created a linux backed ci pipeline for kotlin that tests both functionalities. I've tried to keep the ci pipeline working locally for macos but I haven't set up ci pipeline for macos. One thing I don't really know anything about is how to test this on windows. Otherwise this feels pretty minimal. Another downside is that loading gradle takes a while. I wonder if we can cache spinning up the gradle daemon, but somehow that seems unlikely.

@jcrist1 jcrist1 marked this pull request as draft August 16, 2024 07:10
@jcrist1 jcrist1 marked this pull request as ready for review August 16, 2024 07:28
@jcrist1 jcrist1 marked this pull request as draft August 16, 2024 07:28
@jcrist1 jcrist1 marked this pull request as ready for review August 16, 2024 09:55
@jcrist1
Copy link
Contributor Author

jcrist1 commented Aug 16, 2024

@robertbastian @Manishearth I've added a ci pipeline for kotlin. Linux only for now. Should I create one for Mac too? Windows will be a bit of a challenge cause I don't really know how to work on it. But if necessary I can figure something out.

@jcrist1 jcrist1 changed the title kotlin-ci - try adding kotlin ci steps kotlin-ci Aug 16, 2024
@jcrist1
Copy link
Contributor Author

jcrist1 commented Aug 19, 2024

@Manishearth I'd like to keep this open a bit to see if I can get it to work on JVM 17

- name: Set up JDK 17 for x64
uses: actions/setup-java@v4
with:
java-version: '17'
Copy link
Contributor Author

Choose a reason for hiding this comment

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

As far as I can see @emarteca this now uses a JVM 17 version. I've also locally configured a temurin 17 JVM in my intelliJ and successfully ran the tests. So it should be good to go

Comment on lines -4 to -5
import java.lang.foreign.MemorySession
import java.lang.foreign.SegmentAllocator
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@emarteca the offending lines, if you want to try and remove these, maybe it'll work.

@jcrist1
Copy link
Contributor Author

jcrist1 commented Aug 19, 2024

@Manishearth I'm re-requesting review cause I made a few small changes to make this compatible with JVM 19

@jcrist1 jcrist1 requested a review from Manishearth August 19, 2024 18:21
@Manishearth Manishearth merged commit 266da0f into rust-diplomat:main Aug 19, 2024
16 checks passed
emarteca pushed a commit to emarteca/diplomat that referenced this pull request Aug 19, 2024
* kotlin-ci - try adding kotlin ci steps

* kotlin-ci - add cargo make

* kotlin-ci - comment out other ci temporarily

* korlin-ci - use so file instead of dylib

* kotlin-ci - add library path

* kotlin-ci - println debug

* kotlin-ci - more debug testing

* kotlin-ci - show stdout in gradle tests

* kotlin-ci - gradle debug

* kotlin-ci - commands need exec

* kotlin-ci - add native path

* kotlin-ci - can we add the library path as a jvm arg?

* kotlin-ci use env-variables

* kotlin-ci another try at env-vars

* kotlin-ci - another try at env-vars

* kotlin-ci - remove env-var cruft

* kotlin-ci - fix cargo make env

* kotlin-ci - remove debug code / add example

* kotlin-ci - remove debug code

* kotlin-ci - recomment

* kotlin-ci - dylib -> so for example

* kotlin-ci - cross-platform

* kotlin-ci - fix generated lib file extension

* kotlin-ci - remove debug

* kotlin-ci - uncomment other checks

* kotlin-ci focus on example

* kotlin-ci - debug link path

* kotlin-ci - test both example and feature / remove debug

* kotlin-ci - enable all ci

* kotlin-ci - remove plain whitespace from lines

* kotlin-ci - remove unused panama related imports

* kotlin-ci - example to JVM 17

* kotlin-ci - clean up some tests
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