You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Kotlin keeps additional package metadata in .kotlin_module files; however, these files are not adjusted automatically when relocate function of shadow plugin is used. This does not prevent proper usage of the resulting libraries at runtime, but can cause issues if you try to compile Kotlin code against libraries with such relocated packages, as Kotlin compiler uses this metadata to filter out packages.
There is a kotlinx-metadata-jvm library that can manipulate (read/write) Kotlin metadata, including .kotlin_module files. I can make a PR to support automatic package relocation in Kotlin metadata when relocate is used. Will you accept this sort of contribution?
The text was updated successfully, but these errors were encountered:
@elizarov I'd be happy to take a look at a PR.
My gut is telling me that based on how complex that support is, that I might want to take this time to see if there is a way to have Shadow be extending in a way that makes adding this type of language specific support be added externally instead of in the main library. Would you be willing to work through that process? (My time is pretty small to work on this, so it could drag on, but if you're willing, we can try).
Kotlin keeps additional package metadata in
.kotlin_module
files; however, these files are not adjusted automatically whenrelocate
function of shadow plugin is used. This does not prevent proper usage of the resulting libraries at runtime, but can cause issues if you try to compile Kotlin code against libraries with such relocated packages, as Kotlin compiler uses this metadata to filter out packages.There is a
kotlinx-metadata-jvm
library that can manipulate (read/write) Kotlin metadata, including.kotlin_module
files. I can make a PR to support automatic package relocation in Kotlin metadata whenrelocate
is used. Will you accept this sort of contribution?The text was updated successfully, but these errors were encountered: