Hi,
I am kinda new to `pants`, and had a question ...
# generals
straight-farmer-24902
04/27/2023, 12:57 AMHi,
I am kinda new to
pantspantspantsGo Copy code
./pants run <path to python file> --args1 args1e
enough-analyst-54434
04/27/2023, 1:05 AMDo you want to turn off Go support completely for CI?
enough-analyst-54434
04/27/2023, 1:08 AMIf so you could have a CI pants.toml override file you activate by having CI export `PANTS_CONFIG_FILES=pants.no-go.toml`:
Copy code
[GLOBAL]
backend_packages.remove = ["pants.backend.experimental.go"]s
straight-farmer-24902
04/27/2023, 1:08 AMJust for that run.
e
enough-analyst-54434
04/27/2023, 1:09 AMOk, you can do that then as a fairly ugly hack, but easy to implement and try real quick.
c
curved-television-6568
04/27/2023, 1:09 AMwon’t that make pants choke on go targets then?
e
enough-analyst-54434
04/27/2023, 1:09 AMDamn Pants
enough-analyst-54434
04/27/2023, 1:09 AMI guess
c
curved-television-6568
04/27/2023, 1:10 AMhad the same idea but no feasible work around.. would be nice perhaps with exposing a
--ignore-unknown-targetse
enough-analyst-54434
04/27/2023, 1:11 AMI'd rather Pants didn't nanny quite so much, but since it does - yeah - maybe that.
enough-analyst-54434
04/27/2023, 1:12 AMThe real fix here though is that Pants shouldn't need Go or any binaries if it doesn't actually use them. That seems like a bug.
c
curved-television-6568
04/27/2023, 1:13 AMyea
s
straight-farmer-24902
04/27/2023, 1:14 AMAgreed, I have to unnecessary blot the environment.
e
enough-analyst-54434
04/27/2023, 1:15 AMMaybe Pants does really use Go binaries as part of parsing BUILD somehow.
c
curved-television-6568
04/27/2023, 1:16 AMI think it is during parsing of dependencies or some such..
e
enough-analyst-54434
04/27/2023, 1:16 AMDep inference
enough-analyst-54434
04/27/2023, 1:16 AMYeah 😕
c
curved-television-6568
04/27/2023, 1:17 AMso there’s likely some too eager
AllTargetse
enough-analyst-54434
04/27/2023, 1:18 AMThis is apparently totally ad-hoc, not present for Go fwict, but: https://www.pantsbuild.org/docs/reference-scala-infer#imports
enough-analyst-54434
04/27/2023, 1:19 AMSo if that sort of thing were around for Go, turning it off might work.
h
happy-kitchen-89482
04/29/2023, 12:49 AMYeah, I think this is due to needing to use go to parse .go files for dep inference, because as far as Pants knows, the python file might depend on a go file. But that is clearly silly, and we should have a way to not do this.
2 Views