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

Add more apple targets #41

Closed
wants to merge 1 commit into from
Closed

Add more apple targets #41

wants to merge 1 commit into from

Conversation

rsinukov
Copy link

@rsinukov rsinukov commented Apr 21, 2022

Fixed #40

@rsinukov rsinukov changed the title #40 Add more apple targets Add more apple targets Apr 21, 2022
@rsinukov
Copy link
Author

Hi @Him188,
I see that CI fails, but it also fails without my changes. Do you have any idea how to fix it?

@Him188
Copy link
Owner

Him188 commented Apr 25, 2022

@rsinukov Hi, I figured out that compile-native-platform.gradle is not used anymore. Yamllkt declares platforms here:

val nativeTargets = arrayOf(
"androidNativeArm32, androidNativeArm64, androidNativeX86, androidNativeX64",
"iosArm32, iosArm64, iosX64, iosSimulatorArm64",
"watchosArm32, watchosArm64, watchosX86, watchosX64, watchosSimulatorArm64",
"tvosArm64, tvosX64, tvosSimulatorArm64",
"macosX64, macosArm64",
"linuxArm64, linuxArm32Hfp, linuxMips32, linuxMipsel32, linuxX64",
"mingwX64, mingwX86",
"wasm32"
).flatMap { it.split(", ") }

So Yamlkt already supports all the platforms(but there isn't yet a release that includes all platforms). Maybe I forgot to delete that file in commit baca1d4... Sorry for that.

@rsinukov
Copy link
Author

Ok, thanks!
Do you have any estimates on when a new version will be released?

@rsinukov
Copy link
Author

Also, any plans on updating kotlin and serialization version?

@Him188
Copy link
Owner

Him188 commented Apr 25, 2022

@rsinukov Yes, I will publish a new version with updated Kotlin and serialization in recent 2-3 days.

@Him188 Him188 closed this Apr 26, 2022
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.

Add support for all Apple targets
2 participants