Currently Pants’ dependency tracking has file-leve...
# generall
loud-lizard-27679
08/14/2023, 6:03 PMCurrently Pants’ dependency tracking has file-level granularity. I wonder if it's possible to make it even more granular than that...
h
happy-kitchen-89482
08/14/2023, 6:11 PMIt could be done specifically for Scala, but it's not trivial. It requires deeper integration into the compiler than we currently have.
l
loud-lizard-27679
08/14/2023, 6:12 PMWhy?
loud-lizard-27679
08/14/2023, 6:13 PMCurrently there is no scalac integration, right? Only the dependency inference stuff based on Scalameta
loud-lizard-27679
08/14/2023, 6:15 PMIf you just use Scalameta to write a program that extracts every top level definition of every file into its own file (copying the imports of the original file), then that should be enough. Or am I missing something here?
h
happy-kitchen-89482
08/14/2023, 6:23 PMIf scalameta has all the information then possibly it could be done without a compiler plugin
happy-kitchen-89482
08/14/2023, 6:23 PMThe dependencies would need to be annotated with the symbols that cause the dependency, or something
happy-kitchen-89482
08/14/2023, 6:24 PMWell, it's more than that, I'm not sure how this would work with cache keys
l
loud-lizard-27679
08/14/2023, 6:28 PMHmm, now that I think about it it's probably not worth it. Every change to the imports in a file can potentially change the meaning of the code due to implicits. And I would say that most changes also involve changes to imports
loud-lizard-27679
08/14/2023, 6:33 PMSo yeah, you'd probably need a compiler plugin to make it worthwhile
h
happy-kitchen-89482
08/14/2023, 7:49 PMzinchappy-kitchen-89482
08/14/2023, 7:49 PMAnd it was horrifically complicated
happy-kitchen-89482
08/14/2023, 7:49 PMI hacked on that code at one point
l
loud-lizard-27679
08/14/2023, 7:50 PMI actually suspect that Pants’ dependency inference is much more broken than we realize right now. I'm looking into that right now.
h
happy-kitchen-89482
08/14/2023, 7:58 PMThat is certainly possible!
l
loud-lizard-27679
08/15/2023, 8:42 AMI've made a thread about it in #C0D7TNJHL, would you like to comment?
2 Views