Skip to content
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

optim: replace usage of os.sysconf for Windows #104

Open
eddiebergman opened this issue Jan 9, 2025 · 2 comments
Open

optim: replace usage of os.sysconf for Windows #104

eddiebergman opened this issue Jan 9, 2025 · 2 comments
Labels
bug Something isn't working

Comments

@eddiebergman
Copy link
Collaborator

See issue #100 and the workaround with #103

os.sysconf only exists on Unix systems (which I think includes Mac).

Need a more windows specific way to do this. Hopefully you don't need to go down the route of using the win32 lib

@LennartPurucker
Copy link
Collaborator

Solved in #103

It will be part of the next hotfix in Pip.

@eddiebergman
Copy link
Collaborator Author

eddiebergman commented Jan 9, 2025

Not fully solved, this issue is meant to indicate that a real fix is needed. Here's some little research:

https://chatgpt.com/share/677f9cbe-7ce8-8009-90c6-ed3b9a0d5801

I don't want to just add a new dependancy (although psutil is high quality and pretty regular), and the ctypes solution I would want to test properly on a windows machine but I do not have one readily available.

@eddiebergman eddiebergman reopened this Jan 9, 2025
@noahho noahho added the bug Something isn't working label Jan 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants