-
-
Notifications
You must be signed in to change notification settings - Fork 79
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
Refactor Hyprland.sh #142
Refactor Hyprland.sh #142
Conversation
its not good to read OS release else wont be used to distros which are based to either ubuntu or debian atleast thats what I enountered.. thats why I didnt integrate that function and I have to separate the repo |
But anyway thanks I think you accidentally uploades install-logs? |
Oh sorry I was testing forgot to remove |
Just updating now |
not sure if I remember it correctly, Ive even got a message that he tested to another distro, which I didnt even know it exist. thats why it is safer to not read any from OS releases.. they just need to figure out either based on ubuntu or debian |
I'm testing on Zorin OS and ubuntu cause it is what is in my ventoy |
hmmm.. correct me if Im wrong... but it seems on your PR, will add the universe repo even if its debian... should be only on ubuntu instead? |
sorry the script is fine I just miss wrote the description sry and just edited |
Some minor bug fixes |
pls, DO NOT keep pinging me.... I already mentioned that I am busy and only at the end of this month that I will be free..... |
Sorry bro |
hey... Managed to review this.. However, wont be merging... as there are lots of adjustments..... I started already the process.. You will see the "changes" I am making |
Refactor Hyprland.sh
Description
This checks if the OS is a Ubuntu or Debain by reading the OS release file and then install hyprland and a dependence using the way the project normally uses, if the OS it is Ubuntu and same for Ubuntu like distros, the script enables the universe repos required by hyprland and runs sudo apt update.
Type of change
Please put an
x
in the boxes that apply:Checklist
Please put an
x
in the boxes that apply: