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

setup.py: Separate description for tools and runtime #157

Merged
merged 1 commit into from
Dec 8, 2020

Conversation

abitrolly
Copy link

Another non-code fix.

I am not sure if the runtime is not included in tools package. Looks like it is packed as data, but not probably not installed as something to be imported from Python code. I am also not sure why runtime is packed for PyPI if it is only usable by generated Go code.

@alanjds alanjds merged commit 7b77540 into grumpyhome:master Dec 8, 2020
@alanjds
Copy link

alanjds commented Dec 8, 2020

You are mostly right. runtime is packed for PyPI because I wanted to let an easy way for people to install Grumpy. As Python is needed anyway, distribution using PyPI seemed as a nice idea. Also because I wanted not to mess with the Golang distribution tools.

grumpy-tools can transpile Python -> Go code
grumpy-runtime have the native modules that the transpilled ones need.

@abitrolly abitrolly deleted the package-desc branch December 8, 2020 15:36
@abitrolly
Copy link
Author

This runtime might need some explanation for the users of Python, but for now I couldn't think of anything better. Are there any chance to update PyPI packages?

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