https://pantsbuild.org/ logo
a

ancient-vegetable-10556

09/17/2021, 8:01 PM
Do 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 PM
yeah, 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 PM
yes. 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 PM
OK, that makes sense
h

hundreds-father-404

09/17/2021, 8:08 PM
your tests can activate it via RuleRunner though. Their rule graph != the rule graph you use when running
./pants
in
pantsbuild/pants
f

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 PM
the thought was to preserve commit history as much as possible by not having to refactor out the
experimental
a

ancient-vegetable-10556

09/17/2021, 8:11 PM
sure thing
f

fast-nail-55400

09/17/2021, 8:11 PM
so only the register.py’s will have their history funky
a

ancient-vegetable-10556

09/17/2021, 8:12 PM
Hey, enabling that experimental package made my subsystem show up!
Thanks!
w

witty-crayon-22786

09/20/2021, 6:33 PM
yea, 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