Skidamek / AutoModpack

Enjoy a seamless modpack installation process and effortless updates with a user-friendly solution that simplifies management, making your gaming experience a breeze.
https://modrinth.com/mod/automodpack
GNU Lesser General Public License v3.0
92 stars 29 forks source link

Configuration settings for auto-update behavior #78

Closed limejet closed 1 year ago

limejet commented 1 year ago

Is your feature request related to a problem? Please describe. I run a headless (as in openjdk-17-jre-headless) server on a machine that I own. This server has no graphical environment available and is configured to automatically restart on failure.

This interacts with automodpack in the following ways:

Describe the solution you'd like I'd like two config options, at least on the server side:

Describe alternatives you've considered The second issue can be worked around by letting the server crash and having my recovery configuration kick in to restart it, but that doesn't address the first issue.

Additional context If it were me, I would probably remove the remote updating functionality altogether.

I am kind of against auto-update on servers as a whole. I get why it exists, and i get why you'd want it for AutoModpack specifically considering the way it works what with protocol changes and all that, but I don't think it's a good idea to implement it inside the mod. There are multiple reasons for that:

This mod is already very clever. I tend to think that limiting cleverness to where it is necessary makes for better code.

Skidamek commented 1 year ago

Done in 3.0.0 https://github.com/Skidamek/AutoModpack/releases/tag/v3.0.0-beta