explicitly provided dependencies between different...
# developmentf
fast-nail-55400
09/08/2021, 7:41 PMexplicitly provided dependencies between different
java_libraryw
witty-crayon-22786
09/08/2021, 7:41 PMyea. only file level dependencies are immune to cycles
witty-crayon-22786
09/08/2021, 7:41 PMinference should emit file-level dependencies for that reason
f
fast-nail-55400
09/08/2021, 7:41 PMin same directory?
w
witty-crayon-22786
09/08/2021, 7:41 PMregardless of directory
f
fast-nail-55400
09/08/2021, 7:42 PMwell something to keep in mind for the dependency inference work
fast-nail-55400
09/08/2021, 7:42 PMcan you update the issue with the
only file level dependencies are immune to cyclesfast-nail-55400
09/08/2021, 7:43 PMand
inference should emit file-level dependencies for that reasonw
witty-crayon-22786
09/08/2021, 7:44 PMdone.
witty-crayon-22786
09/08/2021, 7:45 PMfile level dependencies can also be used explicitly in BUILD files to get around this. but would recommend tackling inference before working on real world projects… will save you a lot of time i think.
f
fast-nail-55400
09/08/2021, 7:49 PMI agree. I am triaging the current state of the JVM work on a real-world project (netty in this case) to learn what the current limits of the Pants plugin are.
fast-nail-55400
09/08/2021, 7:49 PMso this issue points at dep inference being pretty important
w
witty-crayon-22786
09/08/2021, 7:50 PMyea. cycle tolerance was a blocker for
tailorwitty-crayon-22786
09/08/2021, 7:51 PMbut manually creating deps at that level is also a pain in the butt, and inference fixes that
f
fast-nail-55400
09/08/2021, 7:52 PMwhich is why I’m stopping at providing anything more than deps on java_library targets
w
witty-crayon-22786
09/08/2021, 7:52 PMwhat do you mean
f
fast-nail-55400
09/08/2021, 7:52 PMstopping using netty as a triage project since file-level deps are too annoying to provide by hand
w
witty-crayon-22786
09/08/2021, 7:53 PMgot it