-
-
Notifications
You must be signed in to change notification settings - Fork 32.3k
gh-127146: Report uid in Emscripten + node as native uid #136509
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
base: main
Are you sure you want to change the base?
Conversation
Fixes `test_netrc.NetrcTestCase.test_security`, which was getting mad that we reported the uid of a file correctly but reported the user's uid as 0.
1fc5975
to
f2a7202
Compare
!buildbot emscripten |
🤖 New build scheduled with the buildbot fleet by @freakboy3742 for commit f2a7202 🤖 Results will be shown at: https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F136509%2Fmerge The command will test the builders whose names match following regular expression: The builders matched are:
|
The buildbot isn't happy with the new C file... |
Indeed, there is a missing |
How do the |
!buildbot emscripten |
🤖 New build scheduled with the buildbot fleet by @freakboy3742 for commit 789681e 🤖 Results will be shown at: https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F136509%2Fmerge The command will test the builders whose names match following regular expression: The builders matched are:
|
I'm not sure if the buildbot issue is a permissions thing, or if the buildbot directive has to be a comment on its own. I've asked on Discord for clarification; in the meantime, I've kicked off a build for you. |
Fixes
test_netrc.NetrcTestCase.test_security
, which was getting mad that we reported the uid of a file correctly but reported the user's uid as 0.cc @freakboy3742.