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
JVM build time improvements via tracking unused deps or classes [Kotlin part] #849
base: master
Are you sure you want to change the base?
JVM build time improvements via tracking unused deps or classes [Kotlin part] #849
Conversation
@@ -25,6 +25,7 @@ kt_bootstrap_library( | |||
"//src/main/kotlin/io/bazel/kotlin/builder/utils/jars", | |||
"//src/main/protobuf:deps_java_proto", | |||
"@com_github_jetbrains_kotlin//:kotlin-compiler", | |||
"@kotlin_rules_maven//:com_google_guava_guava", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How much does this increase the release binary size of rules_kotlin?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Final rules_kotlin_release.tgz
went 19Mb->21.8Mb
"experimental_track_class_usage": attr.string( | ||
doc = "Whether to track used classes", | ||
default = "off", | ||
values = [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why not use booleans here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah.. I should change that. a boolean should be fine, not planning to use a third value here anymore..
This is the Kotlin part of deps/classes tracking to achieve compilation avoidance.
Refer to Java part (TODO) for more info.