-
Notifications
You must be signed in to change notification settings - Fork 59
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
design: add in results of python discussion #56
Conversation
shipping python. For `#4` we determined there was not a significant | ||
number of security issues to make shipping python prohibitive. We can | ||
achieve the goals for `#1` and `#3` by shipping a *system python* that | ||
is only accessible to operating system tools and not to end users. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It'd be useful to spell out in more detail how this might be implemented.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't disagree, but I think that we haven't fully fleshed out the best options there so it might be a bit premature to add it to the design doc. The first few comments in #32 touched on it a bit, but I think it's worth its own discussion.
Also that discussion may warrant us figuring out if we even need to do it or not (i.e. if we can manage to get python out then no need).
WDYT?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure, works for me.
rebased on top of latest master |
@bgilbert @cgwalters @ajeddeloh can I get a review so we can merge? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
LGTM |
🚂 |
#32