Skip to content

Place stumpish from stumpwm's contrib in bin/#1451

Closed
DerGuteMoritz wants to merge 5 commits intoNixOS:masterfrom
DerGuteMoritz:stumpwm-stumpish
Closed

Place stumpish from stumpwm's contrib in bin/#1451
DerGuteMoritz wants to merge 5 commits intoNixOS:masterfrom
DerGuteMoritz:stumpwm-stumpish

Conversation

@DerGuteMoritz
Copy link
Contributor

stumpish is a handy CLI RPC client for stumpwm (e.g. it can be used to
send code from Emacs to stumpwm for evaluation).

stumpish is a handy CLI RPC client for stumpwm (e.g. it can be used to
send code from Emacs to stumpwm for evaluation).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this script depend on a common lisp? If so you can use wrapProgram to create a wrapper which sets PATH correctly.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nope, it's a shell script. Cursory tests seemed to work fine. It calls xprop, though, which it requires to be in PATH. Do I need to anything special for that to be the case?

@7c6f434c
Copy link
Member

7c6f434c commented Jan 2, 2014

Well, xprop is a separate package which could be missing… I guess there is a choice whether to inject a line adding path to xprop with sed or use wrapProgram.

@DerGuteMoritz
Copy link
Contributor Author

Indeed, it seems that it needs a dependency on xprop. I for one didn't install that explicitly on my system but I guess it's pulled in by enabling X? Anyway, I will come up with a patch, thanks for the feedback!

@the-kenny
Copy link
Contributor

Merged in b10fe8a

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.

3 participants