Hey < bitter ability 32190> you ve been fiddling with the `r Pants #development

Hey <@U02KAN6061E> you’ve been fiddling with the `...

curved-television-6568

10/20/2022, 2:29 PM

Hey @bitter-ability-32190 you’ve been fiddling with the

run

goal, right? So this I guess ought to be an easy fix?

Copy code

╰─❯ ./pants run build/scripts/makemigrations.sh
10:27:26.36 [INFO] Remote cache/execution options updated: reinitializing scheduler...
10:27:27.38 [INFO] Scheduler initialized.
10:27:28.06 [ERROR] 1 Exception encountered:

  NoApplicableTargetsException: No applicable files or targets matched. The `run` goal works with these target types:

  * experimental_run_shell_command
  * pex_binary
  * python_source
  * python_test

However, you only specified file arguments with these target types:

  * shell_source

Please specify relevant file and/or target arguments. Run `./pants --filter-target-type=experimental_run_shell_command,pex_binary,python_source,python_test list ::` to find all applicable targets in your project, or run `./pants --filter-target-type=experimental_run_shell_command,pex_binary,python_source,python_test filedeps ::` to find all applicable files.

Just putting it up for suggestion, I could take a look at it eventually.. 😉

curved-television-6568

10/20/2022, 2:30 PM

(there is an

experimental_run_shell_command

in there too that depends on this script..)

hundreds-father-404

10/20/2022, 3:12 PM

super easy to implement

run

support for `shell_source`: https://github.com/pantsbuild/example-plugin/blob/main/pants-plugins/examples/bash/run_binary.py

hundreds-father-404

10/20/2022, 3:13 PM

We didn't implement it because it seemed like there was no reason to. But I guess it's nice for uniform interface to always be able to

./pants run path/to/file.ext

bitter-ability-32190

10/20/2022, 3:21 PM

😤 https://github.com/pantsbuild/pants/issues/16615

🤪 1

bitter-ability-32190

10/20/2022, 3:22 PM

Also @hundreds-father-404 I'm thinking it might be useful to interact with codegen'd

dependencies

. Although, the in-repo CWD vs the sandbox CWD is a tough one (unlike Python, where namespace packages allow you to cheat)

👍 1

curved-television-6568

10/20/2022, 4:21 PM

Ah yeah, in this case I was kind of hoping to run the

experimental_run_shell_command

using it’s main dependency.. but that may not be universally possible to support in an unambiguous way I realize..

curved-television-6568

10/20/2022, 4:22 PM

just invoking the shell script stand alone would be no-bueno as there are other dependencies needed in the chroot (such as a packaged pex binary)

Open in Slack

Previous Next