https://pantsbuild.org/ logo
w

witty-crayon-22786

01/10/2022, 9:00 PM
@curved-television-6568: it looks like there is an un-published page about `experimental_shell_command`: https://dash.readme.com/project/pants/v2.9/docs/run-shell-commands … is that waiting on something?
c

curved-television-6568

01/10/2022, 9:43 PM
It's a draft. Would be good to do some proof reading, then it's probably ok to publish, if it looks ok overall.
w

witty-crayon-22786

01/10/2022, 9:45 PM
looks good to me! … want me to make it visible for 2.9?
c

curved-television-6568

01/10/2022, 9:46 PM
Ok by me :)
w

witty-crayon-22786

01/10/2022, 9:46 PM
cool: now visible!
👍 1
c

curved-television-6568

01/10/2022, 9:49 PM
I'll take a look at filling in the missing section at the end of that page later this week..
👍 1
w

witty-crayon-22786

01/13/2022, 6:24 PM
i just filled in a few more details here, and reorganized the page a bit
❤️ 1
c

curved-television-6568

01/13/2022, 6:50 PM
This is not clear to me:
it must be a root target in your build graph
w

witty-crayon-22786

01/13/2022, 6:50 PM
um, yea… that’s a fuzzy concept, i suppose.
but it’s not packageable, right? so there is no use in depending on it from any other target
./pants run
works, but
./pants package
cannot/should-not
1
c

curved-television-6568

01/13/2022, 6:51 PM
Yea, no point in depending on it.
w

witty-crayon-22786

01/13/2022, 6:52 PM
hence, root. it’s a pretty informal concept though… because we’ve called
pex_binary
a root in the past (no other python code can/should depend on it)… but a
docker_image
can depend on it
(…and even
python_tests
can now, iirc)
c

curved-television-6568

01/13/2022, 6:55 PM
Ah, right.. that makes sense, but the word
root
in my mind associates more with a directory than the dependency graph of targets (still!)
so I tried to grok why it had to be in the build root 😛 (feeling I had the wrong idea..)
w

witty-crayon-22786

01/13/2022, 6:56 PM
ah, yes… true.
i’d really, really love to formalize the concept of a build graph root. but the changes around
pex_binary
demonstrate that that is hard
c

curved-television-6568

01/13/2022, 6:57 PM
so until then, perhaps add a few more words, clarifying the effects of being a root target? 🙂
w

witty-crayon-22786

01/13/2022, 6:59 PM
i think that it amounts to:
Copy code
it must be a root target in your build graph (i.e. it may not be consumed by other targets).
👍 1
will edit.
c

curved-television-6568

01/13/2022, 7:03 PM
yep, that does it. thanks 🙂