boom ```joshuacannon CEPHANDRIUS ~ work pants$ set RUST BAC Pants #development

:boom: ```joshuacannon@CEPHANDRIUS:~/work/pants$ s...

bitter-ability-32190

01/05/2022, 12:36 PM

💥

Copy code

joshuacannon@CEPHANDRIUS:~/work/pants$ set RUST_BACKTRACE=1
joshuacannon@CEPHANDRIUS:~/work/pants$ ./pants peek src/python/pants/backend/python/dependency_inference/scripts/BUILD
06:36:15.38 [ERROR] panic at 'called `Result::unwrap()` on an `Err` value: PyErr { type: <class 'TypeError'>, value: TypeError("'int' object cannot be converted to 'PyString'"), traceback: None }', src/externs/mod.rs:145
06:36:15.38 [ERROR] Please set RUST_BACKTRACE=1, re-run, and then file a bug at <https://github.com/pantsbuild/pants/issues>.

bitter-ability-32190

01/05/2022, 12:36 PM

Is the backtrace dumped somewhere?

bitter-ability-32190

01/05/2022, 12:40 PM

The

BUILD

file that seemingly makes this unhappy is:

Copy code

resources(name="resources", sources=["*.py"])

# Also expose scripts as `python_sources` so they get formatted/linted/checked
python_sources()

bitter-ability-32190

01/05/2022, 12:40 PM

Also:

0887ee2cd4adbfce6a488382fbd0580cbc7e81f0

bitter-ability-32190

01/05/2022, 12:41 PM

BUILD

file might be a red herring

bitter-ability-32190

01/05/2022, 12:42 PM

Ah yes, it's related to my changes, but I'd need a dev to explain why

bitter-ability-32190

01/05/2022, 12:45 PM

Ah OK solved. If you create a

Process

and set an

env

value to an int, this will happen.

bitter-ability-32190

01/05/2022, 12:47 PM

I suppose

mypy

would've told me, but the dogfooding made it impossible to run

mypy

without first hitting this error 😂

🙈 1

bitter-ability-32190

01/05/2022, 2:19 PM

Perhaps the code should still map to string so there's hope. I'll open a PR

witty-crayon-22786

01/05/2022, 5:13 PM

it is possible to actually bootstrap a PEX and then run from that… i’ve only done it once or twice, but it can be really handy for this case (“i’m going to break a lot of things, so let’s prepare”)

witty-crayon-22786

01/05/2022, 5:13 PM

would be interesting to add (back, probably) support for that to the

pants

script in the repo

👍 1

bitter-ability-32190

01/05/2022, 5:15 PM

@witty-crayon-22786 for the record I didn't know I was going to break a lot of things 😂

🤣 1

curved-television-6568

01/05/2022, 5:43 PM

but it would be easy enough to stash your changes to prepare the pex, then pop them back and run.. 😉

witty-crayon-22786

01/05/2022, 5:44 PM

yep

witty-crayon-22786

01/05/2022, 5:44 PM

./pants package src/python/pants/bin:pants

will build the pex, and then you can run it with

./dist/*.pex

🔖 2

👀 1

busy-vase-39202

01/05/2022, 7:20 PM

This seems like a useful addition for somewhere in the docs. Maybe the "Tips and Debugging" page? https://www.pantsbuild.org/docs/rules-api-tips

witty-crayon-22786

01/05/2022, 7:21 PM

yea, probably. it’s only useful for developing pants itself… some probably somewhere under https://www.pantsbuild.org/v2.9/docs/development

👍 1

Open in Slack

Previous Next