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

Change the downloader default GUI #66

Merged
merged 2 commits into from
Aug 13, 2024

Conversation

NeroReflex
Copy link
Contributor

The default mode being suitable for the installer (not showing download progress) is not good for using frzr via terminal: use whiptail GUI by default

The default mode being suitable for the installer (not showing download progress) is not good for using frzr via terminal: use whiptail GUI by default
@BoukeHaarsma23
Copy link
Contributor

I would not default to whiptail, why not the same as it was before? Just the wget output and the installing... messages?

@NeroReflex
Copy link
Contributor Author

I would not default to whiptail, why not the same as it was before? Just the wget output and the installing... messages?

Important question: do we want whiptail at all?

@pastaq
Copy link

pastaq commented Aug 4, 2024

The new installer won't use whiptail anymore, we're going to update state from ENV.

@NeroReflex
Copy link
Contributor Author

The new installer won't use whiptail anymore, we're going to update state from ENV.

I understand this. The current behaviour is assuming frzr is being run by the installer and this commit changes back to assuming an Hunan via CLI when not told otherwise to have the old GUI.

If whiptail is not needed anymore I will replace whiptail with textual output.

- introduce FRZR_INSTALLER to flag for the presence of the GUI installer
- use curl by default if frzr deploy is used by CLI
@NeroReflex NeroReflex changed the title Use whiptail by default Change the downloader default GUI Aug 13, 2024
@NeroReflex NeroReflex merged commit d57b04c into ChimeraOS:master Aug 13, 2024
1 check passed
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.

3 participants