Skip to content

Quick-install script updated#2355

Closed
mrx23dot wants to merge 1 commit intomicro-editor:masterfrom
mrx23dot:master
Closed

Quick-install script updated#2355
mrx23dot wants to merge 1 commit intomicro-editor:masterfrom
mrx23dot:master

Conversation

@mrx23dot
Copy link

@mrx23dot mrx23dot commented Feb 7, 2022

Original install method gives privileged error see benweissmann/getmic.ro#33

@benweissmann
Copy link

Hey @mrx23dot -- I think we can keep these instructions as-is; the getmic.ro script will print instructions for users who want to use the update-alternatives integration. I don't think it's a good idea to recommend that users run this script as root by default; there's lots of setups where that's not needed.

@mrx23dot
Copy link
Author

mrx23dot commented Feb 7, 2022

Yeah, but it doesn't work out of the box with the current instructions (gives error),
nor does apt install the latest version.
So entry of barrier is put really high.

We could suggest in readme:
curl https://getmic.ro | sh && sudo mv micro /usr/bin
that only gives sudo to mv cmd, but .sh script needs to be cleaned up, so it should end with ./micro available.

Also I would like to get it working with ansible automation, for that I would need silent .sh argument. (to prevent asking for user input)

@benweissmann
Copy link

There's ongoing discussion about improving error messaging around the update-alternatives feature; I think we can keep that discussion over on the getmic.ro repo before we make any requests on this repo to update the official docs.

The getmic.ro script does not prompt for input when running without a TTY, so you should be fine running it as part of an Ansible playbook. There's docs on using the GETMICRO_REGISTER env var to control this feature in a non-interactive environment.

@mrx23dot mrx23dot closed this Feb 11, 2025
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