#development
Title
a
ancient-vegetable-10556
09/17/2021, 8:01 PMDo I need to do anything in particular to make a new subsystem show up after I add one?
h
hundreds-father-404
09/17/2021, 8:05 PMyeah, it needs to be used by a rule that is registered in a register.py
a
ancient-vegetable-10556
09/17/2021, 8:05 PM(It’s not immediately clear how they get picked up by Pants, even if the type is consumed by a rule)
OK, so if there’s no register.py, then it’s unlikely to be picked up?
h
hundreds-father-404
09/17/2021, 8:07 PMyes. And not only would you need the register.py, it would need to be activated in pants.toml
a
ancient-vegetable-10556
09/17/2021, 8:07 PMOK, that makes sense
h
hundreds-father-404
09/17/2021, 8:08 PMyour tests can activate it via RuleRunner though. Their rule graph != the rule graph you use when running
./pantspantsbuild/pantsf
fast-nail-55400
09/17/2021, 8:10 PM@ancient-vegetable-10556: the register is under src/python/pants/backend/experimental/java
note the “experimental”
a
ancient-vegetable-10556
09/17/2021, 8:10 PM@fast-nail-55400 that’d explain why I couldn’t find it 🙂
f
fast-nail-55400
09/17/2021, 8:10 PMthe thought was to preserve commit history as much as possible by not having to refactor out the
experimentala
ancient-vegetable-10556
09/17/2021, 8:11 PMsure thing
f
fast-nail-55400
09/17/2021, 8:11 PMso only the register.py’s will have their history funky
a
ancient-vegetable-10556
09/17/2021, 8:12 PMHey, enabling that experimental package made my subsystem show up!
Thanks!
w
witty-crayon-22786
09/20/2021, 6:33 PMyea, the only reason it’s not enabled right now is https://github.com/pantsbuild/pants/issues/12889 … i think that i found a solution to that on Friday: will be focused on implementing it today