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

regarding Arch #91

Closed
Shadowphoenix opened this issue May 28, 2024 · 10 comments
Closed

regarding Arch #91

Shadowphoenix opened this issue May 28, 2024 · 10 comments
Assignees
Labels
bug Something isn't working enhancement New feature or request
Milestone

Comments

@Shadowphoenix
Copy link

ik Arch is currently unsupported however i think i have found a way to still make it work.

https://aur.archlinux.org/packages/python311 can be used to get the python-venv up and running so i would find it reasonable to at least mention this in the Readme, however with the Caviart of saying that running anything from the AUR means you are on your own.

@goldyfruit
Copy link
Member

goldyfruit commented May 29, 2024

@goldyfruit goldyfruit self-assigned this May 29, 2024
@goldyfruit goldyfruit added this to the Quake milestone May 29, 2024
@JarbasAl
Copy link
Member

arch is now on python 3.12, i believe that is the problem here

@goldyfruit goldyfruit added bug Something isn't working enhancement New feature or request labels May 29, 2024
@Shadowphoenix
Copy link
Author

@Shadowphoenix Arch is a supported OS.

https://github.com/OpenVoiceOS/ovos-installer/blob/main/utils/common.sh#L235

yeah i seem to have had a bit of an Aneurysm skimming the Readme Lol. Yeah, Installer doesnt work with Python 3.12 because the script searches for 3.11 specificall. installing Python 3.11 for example via that AUR-Package is a workaround for the time being

@goldyfruit
Copy link
Member

I think it might be time to switch to Python 3.12, I gonna have to run more tests but so far so good.

goldyfruit added a commit that referenced this issue Jun 24, 2024
@goldyfruit
Copy link
Member

@Shadowphoenix do you mind to try this PR #101?

@Shadowphoenix
Copy link
Author

@Shadowphoenix do you mind to try this PR #101?

will do tmrw.

@Shadowphoenix
Copy link
Author

On another note i suggest rewriting the Readme a little.

Remember how i stumbled over the | Arch | N/A | you have written there?

Usually when i encounter N/A, it means "not available". This suggests that something is not supported in any way shape or form which was what lead me to think i cant run this on Arch properly.

https://en.wikipedia.org/wiki/N/A

While technically it is correct as there is no one specific version of Arch you need to run this, this is a bit misleading.
Since Arch, Manjaro and the other rolling releases dont have any specific "versions" that could be supported because they are rolling releases, why not replace the "not available" with "supported" or a simple "Yes"?

@goldyfruit
Copy link
Member

N/A works as well for Not Applicable but I will have a look on how to improve it.

@goldyfruit
Copy link
Member

Change got merged, please re-open if needed.

@goldyfruit
Copy link
Member

3c1a3d5

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants