Page MenuHomeSolus

Initial Kotlin build. Inclusion of Kotlin fixes T5345.
ClosedPublic

Authored by serebit on Mar 3 2018, 3:15 PM.
Tags
None
Referenced Files
F11033054: D2427.id.diff
Wed, Aug 9, 3:09 PM
F11033053: D2427.id6061.diff
Wed, Aug 9, 3:09 PM
F11033052: D2427.id6516.diff
Wed, Aug 9, 3:09 PM
F11033051: D2427.id6057.diff
Wed, Aug 9, 3:09 PM
F11033050: D2427.id5867.diff
Wed, Aug 9, 3:08 PM
F11033049: D2427.id6501.diff
Wed, Aug 9, 3:08 PM
F11026003: D2427.diff
Wed, Aug 9, 1:40 AM
F10983437: D2427.diff
Sun, Jul 23, 2:21 PM
Tokens
"Grey Medal" token, awarded by Snuggle.

Diff Detail

Branch
arcpatch-D2427
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

serebit retitled this revision from Initial build. Inclusion of kotlin fixes T5345. to Initial Kotlin build. Inclusion of Kotlin fixes T5345..Mar 3 2018, 3:19 PM

Also important to note that Kotlin is also packaged in snap form by JetBrains themselves

This revision is now accepted and ready to land.Mar 11 2018, 2:58 PM

Missing Makefile guessing I should also be seeing abi_* reports. Did you actually use solbuild for this?

This revision now requires changes to proceed.Mar 11 2018, 3:30 PM

Added the Makefile. I did use solbuild, I didn't realize I needed to include the makefile as well as the pspec and the package.yml.

Were there no abi_ files? Those need to be included too.

Were there no abi_ files? Those need to be included too.

Nope, no abi_ files were in the directory after a successful make run.

Removed apache-ant from runtime dependencies.

Were there no abi_ files? Those need to be included too.

Kotlin doesn't provide Linux libraries, just like https://dev.solus-project.com/source/apache-ant/

This revision is now accepted and ready to land.Mar 17 2018, 1:44 PM
$ kotlin -version
error: no build.txt was found at home=/usr/share/kotlin

Needs to be fixed. It'll either break tools or anger users. Thanks!

This revision now requires changes to proceed.Mar 25 2018, 4:32 PM

Kotlin was recently updated to 1.2.31, should I fix the missing build.txt in a new package request for 1.2.31 or just fix it in this one?

Might as well update to the newer version, if you don't mind. Same us the update post merge. If that Package Request is behind the current version that's no big deal. The build.txt file definitely needs to go in though. Thanks!

You got it! Fixing it now. I'll modify the diff once I get it working.

Added build.txt to /usr/share/kotlin target, and updated to Kotlin 1.2.31.

LGTM. Thanks! I'll land this when the builder is freed up.

This revision is now accepted and ready to land.Mar 25 2018, 8:20 PM
This revision was automatically updated to reflect the committed changes.