-
Notifications
You must be signed in to change notification settings - Fork 166
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
Create Python-on-test-machines.md #1802
Conversation
@cclauss Output of your scripts, is this what you wanted checked?
|
doc/Python-on-test-machines.md
Outdated
#!/bin/sh | ||
|
||
echo "$0: $1" | ||
ssh $1 which python ; python --version || true |
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.
These don't work, they run python on the host machine because they aren't correctly quoted. Should be
ssh $1 "which python ; python --version" || true
I found a problem with the test scripts, and updated the output above after patching locally. |
On AIX, I think that we should just export PATH=/opt/bin:$PATH as my bet is that Python will find its site-packages just fine. Other than that, your output seems to match the table above. |
@rvagg @thefourtheye @sam-github @refack Could I please get your review on adding this table to our docs? Without it we will have difficulties trying to gather current status across multiple pull requests. |
Well, I guess it matches what @thefourtheye reported in #1775. It does seem more like something that belongs in a tracking issue rather than in the repo though, how are you imagining it to be more helpful in the repo? That freebsd11 machine is back up btw. |
My sense is that several of you are making Python updates that need to go into a single table. Like #1807 for instance. It might be easier to add table edits to those pull requests so that the shared table is updated as the PRs are merged. Alternatively, if the scripts at the bottom can be made to work for those with access the we can have a test script that lists those machines that are still not compliant. |
Closing as stale, but @cclauss , if you think we really need this, ping me, and I'll work with you to get it landed, or finished, or whatever it needs :-). |
@sam-github @nodejs/python