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

core: byte-compile the "core/libs/" folder when dotspacemacs-byte-compile is t #15696

Merged
merged 1 commit into from
Aug 11, 2022

Conversation

sunlin7
Copy link
Contributor

@sunlin7 sunlin7 commented Aug 10, 2022

Hi,

Currently dotspacemacs-byte-compile is t will only compile serveral files in core/lib/*.
After the PR #15692, all files in core/ will be able byte-compile/native-compile.

So we'll byte compile the core/ directory when dotspacemacs-byte-compile is t.

@lebensterben
Copy link
Collaborator

I prefer not to make haste on byte compiling everything for now.

@sunlin7
Copy link
Contributor Author

sunlin7 commented Aug 10, 2022

Hi @lebensterben

What about byte-compiling the directory core/libs/? I'll change it to work in core/libs/ directory.

@lebensterben
Copy link
Collaborator

it's okay to byte compile core/libs. but don't modify any of those files because they are 3rd party packages.

@sunlin7 sunlin7 changed the title core: byte-compile the "core/" folder when dotspacemacs-byte-compile is t core: byte-compile the "core/libs/" folder when dotspacemacs-byte-compile is t Aug 11, 2022
@sunlin7
Copy link
Contributor Author

sunlin7 commented Aug 11, 2022

Hi @lebensterben

I had modified the code to byte-compile the core/libs/ folder when the dotspacemacs-byte-compile is t.

Please help review and approve this change. Thanks.

@sunlin7 sunlin7 marked this pull request as ready for review August 11, 2022 03:29
@smile13241324 smile13241324 merged commit c5c69ce into syl20bnr:develop Aug 11, 2022
@sunlin7
Copy link
Contributor Author

sunlin7 commented Aug 11, 2022

Thank you all !

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