-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Extend the os module #1175
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
Comments
Ok, I'm working on get/set uid, guid etc. stuff |
I'm going to take a stab at Also, as a general question, is the current goal to match CPython functionality here? Currently it doesn't seem that the os module is implemented in C for CPython. Is the goal here to implement the Python os module in Rust as, for example, the sys module in CPython is implemented in C and not in Python? Or am I completely missing something here? |
Because Rust's cross-platform support is so much better than C's, we decided to implement a single |
Ah, gotcha..so for |
Yep, and you can use the |
cool cool, thanks! |
PR #1524 has an implementation of |
This comment has been minimized.
This comment has been minimized.
Hi, new here. I was just wondering how to start working on one of these outstanding implementations without having any overlap with others? Should we open a new issue for what we want to tackle and then have it assigned to us? Thanks! |
I'm so sorry for not responding earlier @nicholasraphael, if you're still interested you can feel free to just open a PR for it; there are a lot of functions and hopefully what you choose won't conflict if anyone else is working on something. You could also just comment what you're working on in this thread. |
It's always possible to just do |
Hey, I wanted to work on some of these issue. But how can I create tests for this ? Basically after I implement a method, how can I test if it's working as expected? Sorry quite new here. |
hi @coolreader18 is the list here #1175 (comment) updated? i'd like to start working on one of the API's thanks! |
@AthulMuralidhar I think of that list,
still need to be implemented in os.rs |
thanks for the quick reply @coolreader18 i'll try to work on |
|
Hi, let me work on |
Let me work on |
I made a analysis to detemine the next important os module based on the real usage in GitHub public code. This is based on the master branch of this repo at the time of writing. Feel free to use and modify it. |
Implementing os.listxattr |
Hi, is there something left to work on this? |
Run |
I added a note to the issue body to check |
Added a few macos constants: #4110 |
I'd like to take on os.pathconf_names - and have crated #4494 to track |
Edit: Looks like its already implemented in |
If you're referring to what's on the website, the website gets updated every Friday (at least, in my timezone). |
Currently, the python
os
module is missing many methods. Try adding a method to this module by editing the filevm/src/stdlib/os.rs
.This is an issue for new contributors, so please do not implement everything at once, so other people have a chance to go for it (or, actually, please implement all! >:)! ).
Note: Before working on platform-specific functions, check if they are members of
posix
ornt
. Their members are reexported toos
.The text was updated successfully, but these errors were encountered: