Finally got around to upgrading to 1.6.20 Took a ...
# javascript
d
Finally got around to upgrading to 1.6.20 Took a little bit to fix some of the JsExports, but completely worth it for interface and enum support. Also tried out the “kotlin.incremental.js.ir=true” flag, and it did seem to cut build times in half. Decided not to use it because source maps were not generating properly, but still super excited to see effort being made on this front. Thank you so much, these improvements really hit the hammer on the nail when it comes to improving developer experience. Keep up the great work ❤️
🙏 2
5
b
Thank you for the feedback! It would very helpful if could share more information about broken sourcemaps (cc @anton.bannykh)
d
@bashor absolutely can do, when setting
Copy code
<http://kotlin.incremental.js.ir|kotlin.incremental.js.ir>=true
And running the BrowserDevelopmentWebPack task, the resulting .js.map file has an empty array in the sources:
Copy code
{"version":3,"sources":[],"sourcesContent":[],"names":[],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;
...
Whereas disabling the flag results in the expected output:
Copy code
{"version":3,"sources":["../../../../../src/commonMain/kotlin/mesa/entity/Stats.kt","../../../../../Maps.kt","../../../../../src/commonMain/kotlin/mesa/model/Access.kt"
...
kotlin browser task: ${moduleName}BrowserDevelopmentWebPack ${moduleName}.js.map file location: build -> js -> packages -> $moduleName -> kotlin
b
Thanks, we will take a look at it
a
So, I reproduced the issue and also find the root of the problem, and it's, actually, already fixed in
master
branch of kotlin. So, seems like the issue will be fixed in 1.7.20. Am I right @bashor?
👍 1
b
Depends on when the fix was pushed — before or after brunching 1.7.0
👍 1