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

Fix supported distro checking and handling #12

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

askiiart
Copy link

@askiiart askiiart commented Nov 7, 2024

Previously, distros were checked in both blend and user, which led to user's distro list being forgotten about and severely out of date, not to mention incorrect in the first place; it's a blessing that the code didn't work as it probably should have - that is, by exiting if the distro was unsupported.

With this, supported distro checking is handled by blend only, and it now has an exit code of 1 if the distro is unsupported, and user returns that exit code.

@askiiart
Copy link
Author

askiiart commented Nov 7, 2024

I also threw in a fix so that user's default distro is arch-linux, which is what blend actually uses, not arch.

Anyways, it just needs to verified that none of this will break the frontend, and then we should be good to go.

@askiiart
Copy link
Author

askiiart commented Nov 7, 2024

Turns out no distro being specified is already handled by blend, so I updated it so that user doesn't add a distro in the case that one isn't specified.

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

Successfully merging this pull request may close these issues.

1 participant