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

do NOT suggest a command that skips confirmation of breaking changes and FIX the backup script #443

Open
Azazel-Woodwind opened this issue Nov 25, 2024 · 0 comments

Comments

@Azazel-Woodwind
Copy link

There are multiple things wrong:

  1. It is horrible behaviour that the wiki's command to run defaults to the web installer, which SKIPS the confirmation prompt that you will be overwriting all of your data, which happened to me. This should NEVER be the case. Given the gravity of these changes, which may not be done on a fresh system, and frankly probably won't be done on a fresh system most of the time, changes like this should NEVER be done without a confirmation prompt, as is general good practice.
  2. The backup script is flawed and doesn't work. First and foremost, it should backup the entire .config/hypr directory which could contain multiple .conf files instead of just .config/hypr/hyprland.conf. I have confirmed that it doesn't do this in the source code. But worst of all, it doesn't even copy successfully, as the backup is nowhere to be found on my system. I suspect that this is because you try to copy to a config directory which doesn't exist by default.
  3. Lastly, other configs such as the foot.ini get overriden with no backup

There is one main important takeaway: do NOT make system-breaking changes without confirmation beforehand, and when you do so, write working and tested backup scripts.

@Azazel-Woodwind Azazel-Woodwind changed the title do NOT run the web installer by default and FIX the backup script do NOT suggest a command that skips confirmation of breaking changes and FIX the backup script Nov 25, 2024
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

No branches or pull requests

1 participant