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

esup fails to run on macOS High Sierra / Emacs 25.3 #51

Closed
deybhayden opened this issue Mar 28, 2018 · 19 comments
Closed

esup fails to run on macOS High Sierra / Emacs 25.3 #51

deybhayden opened this issue Mar 28, 2018 · 19 comments

Comments

@deybhayden
Copy link

Hey there, thanks for this great plugin! I recently started having some problems running M-x esup

screen shot 2018-03-28 at 4 45 00 pm

The *Messages* buffer has this:

For information about GNU Emacs and the GNU system, type C-h C-a.
Error: (void-function vc-git-root)
esup-child-send-log: Process *esup-child-connection* not running

Any ideas (this probably worked about a 2-3 weeks ago)?

@zaript
Copy link

zaript commented Apr 9, 2018

I also have an issue in MacOS, not sure if it's related or completely different:

Starting esup...
esup process started on port 50709
at 1
error in process sentinel: eieio-oref: Wrong type argument: (or eieio-object class), nil, obj
error in process sentinel: Wrong type argument: (or eieio-object class), nil, obj

Emacs version:

GNU Emacs 25.3.1 (x86_64-apple-darwin17.4.0, NS appkit-1561.20 Version 10.13.3 (Build 17D102)) of 2018-03-08

@casouri
Copy link
Contributor

casouri commented Apr 28, 2018

+1 on Emacs 26, High Sierra.

GNU Emacs 26.0.91 (build 1, x86_64-apple-darwin17.4.0, NS appkit-1561.20 Version 10.13.3 (Build 17D102)) of 2018-02-26

@CeleritasCelery
Copy link

+1 for Emacs 25.1 on SUSE Linux. Seeing Error: (void-function vc-git-root)

@gmjain
Copy link

gmjain commented May 3, 2018

+1 for emacs 25.3.
Seeing the same eieio-oref issue as @zaript.

Will help in anyway I can. :-)

jschaf added a commit that referenced this issue May 6, 2018
@jschaf
Copy link
Owner

jschaf commented May 6, 2018

Hey all, as a hot-fix I'll reduce the reduce profile depth to see if that addresses the issue.

@jschaf
Copy link
Owner

jschaf commented May 6, 2018

I think it has to do with how esup intercepts require calls. I switched it to use advice instead of parsing the elisp. The advice is much more robust, but has the unintended side-effect of intercepting require calls that don't appear in dotfiles.

vc-git.el is loaded indirectly when loading a vc-controlled file and esup happily starts profiling. It's not useful to profile vc-git so esup should skip it.

To fix this, esup should only intercept require calls that appear directly in dotfiles.

@gmjain
Copy link

gmjain commented May 7, 2018

@jschaf, Thanks for the update! :-) I tried the latest version. The eieio issue seems unaffected by the fix.
I still see this:

Starting esup...
esup process started on port 34141
You can run the command ‘esup’ with M-x esu RET
at 1
error in process sentinel: eieio-oref: Wrong type argument: (or eieio-object class), nil, obj
error in process sentinel: Wrong type argument: (or eieio-object class), nil, obj
esup process started on port 34141

I guess it's the same issue in #52 and #54 within esup-fontify-results.
I can help you with any debugging information you might need. :-)

@CeleritasCelery
Copy link

CeleritasCelery commented May 7, 2018

@jschaf, that fixed the original issue for me (the error that this issue was created for: (void-function vc-git-root)). Thanks!

@zaript
Copy link

zaript commented May 28, 2018

Seems to be fixed with latest version of esup and Emacs 26.1.

@jeffbowman
Copy link

Still seeing this with Emacs 26.1:
Starting esup...
esup process started on port 39165
at 1
error in process sentinel: eieio-oref: Wrong type argument: (or eieio-object class), nil, obj
error in process sentinel: Wrong type argument: (or eieio-object class), nil, obj

for reference, my configuration is at https://bitbucket.org/jeffbowman/dotfiles

@zaript
Copy link

zaript commented May 29, 2018

@jeffbowman Occasionally I manage to fix things by deleting all *.elc files in .emacs.d. Things start working after recompilation.

@matthew-piziak
Copy link

Still seeing this on Emacs 26.1 with today's version of esup and all elc files freshly recompiled.

@ckeschnat
Copy link

I also stumbled upon this after trying to switch to straight.el which compiles everything. After reverting to package.el (and not having any *.elc) esup worked again.

GNU Emacs 26.1 (build 1, x86_64-w64-mingw32) of 2018-05-30
Windows 10
esup-20180727.342

@matthew-piziak
Copy link

Oh is that what it is? I also use straight.el.

@CeleritasCelery
Copy link

I am also seeing this again after switching to straight.el. Not sure which package is at fault.

@glepnir
Copy link

glepnir commented Feb 22, 2020

@jschaf got same error with straight on emacs 28

@AtomicNess123
Copy link

Same error ) esup-child-send-log: Process esup-child-connection not running". Any ideas?

@casouri
Copy link
Contributor

casouri commented Apr 21, 2020

Maybe you can try this: https://github.com/dholm/benchmark-init-el

@AtomicNess123
Copy link

Thanks, I will try.

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