Add check for snapshot, check for release instead of minecraft

[dbergl]

This is a tweak to #169

Instead of checking for 'minecraft' it now checks for 'release' or 'snapshot' and grabs the latest released version as specified in the json.

[IronTetsubo]

Hold off on this for just a sec - I was going to submit a pull request
related to this. I assume this is due to the fact that the default jargroup
isn't working right now that we added? I was going to tweak said jargroup
so the target matched "minecraft", which should address this in a simpler
fashion. Or, if not, what are you solving for here?

On Mon, Sep 23, 2013 at 12:03 PM, dbergl [email protected] wrote:

This is a tweak to #169#169

Instead of checking for 'minecraft' it now checks for 'release' or

'snapshot' and grabs the latest released version as specified in the json.

You can merge this Pull Request by running

git pull https://github.com/dbergl/minecraft-server-manager getlatest

Or view, comment on, or merge it at:

#214
Commit Summary

• check for release or snapshot and get latest appropriate version

File Changes

• M init/msmhttps://github.com/Add check for snapshot, check for release instead of minecraft #214/files#diff-0(7)

Patch Links:

https://github.com/marcuswhybrow/minecraft-server-manager/pull/214.patch

https://github.com/marcuswhybrow/minecraft-server-manager/pull/214.diff

Dave Fallon

[dbergl]

Current implementation doesn't allow for updating a server running the weekly snapshots without updating the URL each week. This makes it easy to update the jar for a snapshot server too.

Checking for minecraft was fine but changing the check to release or snapshot just simplified the code.

[kbrock]

👍 This is great. Noticed it documented in article

[renderorange]

I still have a couple of ideas to address with this, but am working to get this one out.

[renderorange]

I've updated the installation and init scripts to check for this, so default installs will now pull the most recent "release" builds. This brings the installation current, so no "hacking" will need to take place to get things working. More things to catch the project up on, but this was the most pressing, IMO.