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

apt waits for user input when installing dependencies #439

Closed
Touratica opened this issue Jul 24, 2022 · 1 comment
Closed

apt waits for user input when installing dependencies #439

Touratica opened this issue Jul 24, 2022 · 1 comment

Comments

@Touratica
Copy link
Contributor

When installing on a debian based system, apt asks if the user wants to install certain packages. I believe this shouldn't be the case, the script should ask no further input after those first four questions.

Touratica added a commit to Touratica/msm that referenced this issue Jul 24, 2022
Fixes: apt waits for user input when installing dependencies msmhq#439
Touratica added a commit to Touratica/msm that referenced this issue Jul 24, 2022
Fixes: apt waits for user input when installing dependencies msmhq#439
Touratica added a commit to Touratica/msm that referenced this issue Jul 25, 2022
…do and wget dependencies (#1)

* Set dependency installation as non-interactive

Fixes: apt waits for user input when installing dependencies msmhq#439

* Add sudo and wget as dependencies

Fixes: Installer assumes system has sudo and wget installed msmhq#440

* Add openjdk-17-jre-headless as dependency

The script will install openjdk-17-jre-headless if java isn't installed already. The choice for version 17 was motivated by the fact it is the latest LTS version and also that it is required for running Minecraft 1.18 and above.

Fixes: Make java an MSM installation dependency? msmhq#295

* Add software-properties-common as dependency

The Debian script now installs software-properties-common if not installed, so it can use add-apt-repository.

Fixes: Installer assumes software-properties-common is installed on debian system msmhq#441

* Fix broken whitespace
@endofcake
Copy link
Contributor

Fix merged in #442 .

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

2 participants