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

Isolate proc-macro dependency to the proc macro #77

Merged
merged 1 commit into from
Apr 12, 2018
Merged

Isolate proc-macro dependency to the proc macro #77

merged 1 commit into from
Apr 12, 2018

Conversation

dtolnay
Copy link
Contributor

@dtolnay dtolnay commented Apr 12, 2018

The dependency of askama on askama_shared on syn on proc_macro used to mean libproc_macro would be dynamically linked into any crate using askama. We want only askama_derive to have a runtime dependency on proc macro. This commit moves all proc macro code from askama_shared into askama_derive so that the askama crate no longer dynamically links libproc_macro.

Fixes #73. Closes dtolnay/syn#400.

The dependency of askama on askama_shared on syn on libproc_macro used
to mean libproc_macro would be dynamically linked into any crate using
askama. We want only askama_derive to have a runtime dependency on proc
macro. This commit moves all proc macro code from askama_shared into
askama_derive so that the askama crate no longer dynamically links
libproc_macro.
@dtolnay
Copy link
Contributor Author

dtolnay commented Apr 12, 2018

(This is a breaking change for askama_shared.)

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