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

init-growl.el has a bug with windows #52

Closed
zhangriyueming opened this issue Jan 9, 2013 · 4 comments
Closed

init-growl.el has a bug with windows #52

zhangriyueming opened this issue Jan 9, 2013 · 4 comments

Comments

@zhangriyueming
Copy link

it causes "cannot save customizations- init file not fully loaded" and "Spawning child process: invalid argument" on windows.
and remove
(require 'init-growl) in init.el
it works fine.
thanks for your work.

@purcell purcell closed this as completed in 8fefff9 Jan 9, 2013
@purcell
Copy link
Owner

purcell commented Jan 9, 2013

Ah, I see. It looks like todochiku doesn't check to see if the snarl helper program is available at "C:/Program Files/full phat/Snarl/exrras/sncmd/snarl_command.exe".

I've committed a fix. Thanks for the report!

-Steve

@zhangriyueming
Copy link
Author

i'm sorry your solution doesn't work for me.
it seems the todochiku is too old to work with the new snarl.
i edited the file "todochiku.el" located on .emacs.d\elpa\todochiku-20120202.2005 with replacing
"C:/Program Files/full phat/Snarl/extras/sncmd/snarl_command.exe" to
"C:/Program Files (x86)/full phat/Snarl/tools/heysnarl.exe"
then delete the todochiku.elc
and it works.
i hope it's helpful.
thanks for fast response.

@purcell
Copy link
Owner

purcell commented Jan 9, 2013

Yeah, the fix I committed wouldn't make the newer version of Snarl work, but it should prevent errors when the old version is not installed, which I think was what was happening for you originally.

If you want to use the newer Snarl, you should be able to use the Emacs customization system to set todochiku-command to "C:/Program Files (x86)/full phat/Snarl/tools/heysnarl.exe" -- it shouldn't be necessary to edit the source file directly.

@zhangriyueming
Copy link
Author

i'm totally newbie for emacs.
thanks for your great zest, sorry for my poor English. : )

qioixiy pushed a commit to qioixiy/emacs.d that referenced this issue Aug 5, 2017
* lispy.el (lispy--slurp-forward): Improve.

* lispy-test.el (lispy-slurp): Add test.

Fixes purcell#52.
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

No branches or pull requests

2 participants