8000 sys.safe_path with Command line parameter -P and environment variable `PYTHONSAFEPATH` · Issue #4541 · RustPython/RustPython · GitHub
[go: up one dir, main page]
Skip to content

sys.safe_path with Command line parameter -P and environment variable PYTHONSAFEPATH #4541

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.

Sign up for GitHub

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

Jump to bottom
Open
1 of 4 tasks
Tracked by #5529
, #4537 ,  #4564 ,  #5104
...
youknowone opened this issue Feb 21, 2023 · 2 comments
Open
1 of 4 tasks
Tracked by #5529
, #4537 ,  #4564 ,  #5104
...

sys.safe_path with Command line parameter -P and environment variable PYTHONSAFEPATH #4541

youknowone opened this issue Feb 21, 2023 · 2 comments
Labels
C-compat A discrepancy between RustPython and CPython good first issue Good for newcomers

Comments

@youknowone
Copy link
Member
youknowone commented Feb 21, 2023 •
edited
Loading

https://docs.python.org/3/using/cmdline.html#cmdoption-P

https://github.com/RustPython/RustPython/pull/4537/files#diff-cb42fd26146ec61cb2242243494a9f314667a9a39153ad17e37465ff5a8a8000R536

  • add sys.flags.safe_path add sys.flags and sys.int_info #4568
  • add command line parameter -P to set flag
  • use environment variable PYTHONSAFEPATH to set flag
  • actual implementation of safe_path

I didn't check the actual implementation, but other 3 tasks are easy tasks
This was referenced Feb 21, 2023
Merged
Closed
@youknowone youknowone changed the title Command line parameter -P sys.safe_path with Command line parameter -P and environment variable PYTHONSAFEPATH Feb 25, 2023
@youknowone youknowone added good first issue Good for newcomers C-compat A discrepancy between RustPython and CPython labels Feb 25, 2023
@naonus
Copy link
Contributor
naonus commented Mar 2, 2023

I'll take this issue

@youknowone youknowone mentioned this issue Mar 2, 2023
Merged
@youknowone youknowone mentioned this issue Oct 23, 2023
Closed
@AgentK9
Copy link
AgentK9 commented Aug 3, 2024 •
edited
Loading

@youknowone - just FYI, tasks 2 and 3 are also completed - https://github.com/RustPython/RustPython/blob/main/src/settings.rs#L245-L249

After diving into the code, I don't see where it is implemented, but it seems like 4 is also completed after testing it out.

Could you please close this issue for future people like me?

Thanks

@youknowone youknowone mentioned this issue Feb 20, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C-compat A discrepancy between RustPython and CPython good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@youknowone @AgentK9 @naonus
0