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

Windows installation: KeyError os.environ["HOME"] #83

Open
ahalota opened this issue Apr 8, 2021 · 5 comments
Open

Windows installation: KeyError os.environ["HOME"] #83

ahalota opened this issue Apr 8, 2021 · 5 comments

Comments

@ahalota
Copy link

ahalota commented Apr 8, 2021

I'm failing several of the tests on a Windows install, because Windows uses "USERPROFILE" instead of "HOME".

I'm not sure if this is designed to run with Windows or not, for now I am going to replace the code on my end with this:

import os.path
home_folder = os.path.expanduser('~')

(src: https://stackoverflow.com/questions/14742064/python-os-environhome-works-on-idle-but-not-in-a-script/33935163)

@ahalota
Copy link
Author

ahalota commented Apr 8, 2021

I'm not familiar with your process and how much Windows support you have planned in, I am happy to push any updates I make as I run through this on my Windows machine, in case anything else comes up.

@ahalota
Copy link
Author

ahalota commented Apr 8, 2021

I see now that actinia's installation instructions say it works for linux only, so this issue is mostly irrelevant (unless they change this on their end eventually). Either way it can't hurt to make it os neutral.

@mmacata
Copy link
Collaborator

mmacata commented Apr 8, 2021

Hi ahalota, we only use it on linux environments, mostly inside a docker container, so no current plans for Windows support. But I agree with you - it can't hurt to make it os neutral. If you are willing to push your updates as you said, it would help!
As a running actinia instance is independant from this backend, they can be installed on different operating systems.

@jonas-eberle
Copy link

os.environ["HOME"] seems only be used in the config.py. So this relates to #76. Such a config file would solve this issue here.

@metzm
Copy link
Collaborator

metzm commented Dec 16, 2022

TODO: Use os.path.expanduser('~') in config.py

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants