Skip to content

Standalone binary preparation#2134

Merged
iamdefinitelyahuman merged 3 commits into
vyperlang:masterfrom
iamdefinitelyahuman:cli/single-entry-point
Aug 21, 2020
Merged

Standalone binary preparation#2134
iamdefinitelyahuman merged 3 commits into
vyperlang:masterfrom
iamdefinitelyahuman:cli/single-entry-point

Conversation

@iamdefinitelyahuman

Copy link
Copy Markdown
Contributor

What I did

  • Adjust vyper_compile to offer a single entry-point for both normal and JSON compiling.
  • Add pyinstaller as a dev dependency and freeze as a makefile command.

This sets the stage for #1953

How I did it

  • Access to vyper-json is provided via vyper with the --standard-json flag.
  • vyper/cli/vyper_compile.py has if __name__ == "__main__" logic to allow it to be run directly.

Cute Animal Picture

image

@lgtm-com

lgtm-com Bot commented Aug 21, 2020

Copy link
Copy Markdown
Contributor

This pull request introduces 2 alerts when merging 5a8be5f into 0879886 - view on LGTM.com

new alerts:

  • 1 for Use of the return value of a procedure
  • 1 for Module-level cyclic import

Comment thread .gitignore Outdated
Comment thread Makefile Outdated
@iamdefinitelyahuman iamdefinitelyahuman merged commit 0dcd148 into vyperlang:master Aug 21, 2020
@iamdefinitelyahuman iamdefinitelyahuman deleted the cli/single-entry-point branch August 21, 2020 22:57
@charles-cooper charles-cooper mentioned this pull request Apr 4, 2022
3 tasks
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