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

Symbol's function definition is void: package-installed-p on startup when byte-compiled #33

Closed
rjww opened this issue Mar 2, 2017 · 7 comments

Comments

@rjww
Copy link
Contributor

rjww commented Mar 2, 2017

No description provided.

@hlissner
Copy link
Member

hlissner commented Mar 2, 2017

I think this was caused by #5b9a32 and fixed in #ca3628. I was experimenting with lexical-binding, which messes with doom-initialize-packages, which doom/recompile (and doom/reload-autoloads uses.

It snuck into that first commit without me realizing. If you could, please update and see if the issue is still around. Sorry about that!

hlissner added a commit that referenced this issue Mar 2, 2017
@hlissner
Copy link
Member

hlissner commented Mar 2, 2017

Another possibility: once byte compiled, package.el is not initialized, so none of the package-* functions will be available. If any part of a module (with def-package! calls) fails to compile during compilation for whatever reason, then you'll get this error.

So, I've added a check to def-package! just in case that ever happens.

@rjww
Copy link
Contributor Author

rjww commented Mar 2, 2017

Whatever it was, it looks like it's fine after the most recent commit. Thanks!

@hlissner hlissner closed this as completed Mar 2, 2017
@rjww
Copy link
Contributor Author

rjww commented Mar 2, 2017

Wait, sorry, now the modeline isn't showing up.

@hlissner hlissner reopened this Mar 2, 2017
@hlissner
Copy link
Member

hlissner commented Mar 2, 2017

Try reloading the autoloads: make autoloads or doom/reload-autoloads. I've changed some of the autoloaded functions doom-modeline depends on within the last 20 minutes.

@rjww
Copy link
Contributor Author

rjww commented Mar 2, 2017

Yep, I'm an idiot. Thanks again!

@hlissner
Copy link
Member

hlissner commented Mar 2, 2017

No problem! Sorry for all the breaking updates.

@hlissner hlissner closed this as completed Mar 2, 2017
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 1, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants