Nick Mills-Barrett

We can probably add a new fact to check for this before passing to the shared package install method, and call update if needed.

Thank you for the detailed issue, @dfaerch, definitely not quite right. The debug output seems to be different again - 1,2,3,4 all present but no 5. I'm fairly sure the...

Augh I thought 3.4.1 fixed this 😩

Figured it out - it's due to searching for Python code matching the target using [`find_spec`](https://docs.python.org/3/library/importlib.html#importlib.abc.MetaPathFinder.find_spec) which turns out _might_ import the code: > If the name is for submodule...

Does it also get stuck if you run the pyinfra command outside of the IDE? Need to understand if the IDE is specifically the problem here or something else.

Hm. The `_sudo` argument should be applied before any `_chdir` resulting in a command like this: ```sh sudo -H -n sh -c 'cd /tmp && echo hi' ``` Which should...

Hm this appears to be failing at the prior command: ``` [hlf.bevjam.com] >>> sh -c 'cd /var/services/homes/something && rm -f /volume1/tfr/tmp/pyinfra-6b6c971ce9dd9aaea30991dbaa63ef13f3cc0e4c' ``` The askpass removal succeeds here: ``` --> Disconnecting...

I believe this should be fixed by 3.4.1 (fix: https://github.com/pyinfra-dev/pyinfra/commit/8e2335415d6a48ce05dbbe086f932cec4648c61f)

Found it! It's due to the change in 3.4 to allow facts to call each other, but the context was never correctly set in the facts API, https://github.com/pyinfra-dev/pyinfra/commit/0c48515f324b1b73b5c5fd6df5291732be989baf will fix.

Completely agree this should be documented and the option added to do the checks recursively. Flagging as bug until the documentation bit is done as this will cause confusion.