Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dotool support #92

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Dotool support #92

wants to merge 6 commits into from

Conversation

msin32
Copy link
Contributor

@msin32 msin32 commented May 14, 2024

Added support for dotool which doesn't require a service and should work everywhere.

@msin32 msin32 mentioned this pull request Jun 27, 2024
@tecosaur
Copy link
Owner

tecosaur commented Jul 9, 2024

Hmmm, this looks a bit so-so for me.

  1. It doesn't seem worth pulling in anaphora as a dependency just for the usage here
  2. The use of temp files seems last-resort ugly to me
  3. dotool itself looks good

@msin32
Copy link
Contributor Author

msin32 commented Aug 23, 2024

I think it works now, I tried to get everything handled using call-process-region but it wouldn't work with dotool for some reason, though shell-command-on-region does.

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.

2 participants