Skip to content

Conversation

@stephanmax
Copy link
Member

Changes

  • Introduce new method probeCommand to check for presence of critical tools pnpm and node
  • Introduce helper closure to stack notices in the status line of PDE
  • Make directory for builder optional

Fixes #14

@stephanmax stephanmax linked an issue Jan 28, 2026 that may be closed by this pull request
@Stefterv
Copy link
Collaborator

Stefterv commented Jan 29, 2026

Screenshot 2026-01-29 at 19 11 33

I suspect this doesn't work because my shell env first prints my lovely terminal art before running anything else

Screenshot 2026-01-29 at 19 13 11

@stephanmax
Copy link
Member Author

stephanmax commented Jan 29, 2026

That IS lovely! Interesting, I will check how I can distinguist command output and general shell output.

(Ah yes, I remember now, I am using a full-on interactive shell so we can have all the env variables, but that of course also triggers any user-defined shell startup code in .bashrc or equivalent.)

@catilac
Copy link

catilac commented Jan 29, 2026

@stephanmax this is working for me on gentoo/zsh/wayland /cc @lee2sman

@lee2sman
Copy link

how can i test? do i need to build from source? i don't have a kotlin or java environment set up for example.

@catilac
Copy link

catilac commented Jan 29, 2026

let us know what you're thinking for handling running shell commands @stephanmax

@lee2sman ooops nevermind. ill let you know when this is merged. i forgot you're not building

This ignores any user-defined output in an interactive shell that might
precede the output of the command.
@Stefterv
Copy link
Collaborator

Screenshot 2026-01-30 at 12 01 26

@stephanmax
Copy link
Member Author

I think the easiest workaround is only considering the last line of the command output. This is not as rigorous a solution as I’d like, but my other ideas fall short as well.

  • We could redirect the command output to std error and read that stream. User-defined output would still go to std out and we could dinstinguish them.
  • We could just consider the error code. But I like the debugging information that pnpm and Node version give us.
  • Come up with a more clever test for a certain binary (without relying on an interactive shell)?!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

need some help installing p5.js mode

5 participants