iamgreaser / iceball

Open-source rewrite of the VOXLAP version of Ace of Spades.
http://iceball.build
GNU General Public License v3.0
113 stars 32 forks source link

Change <settingsfile>.json to <settingsfile>.json.example #224

Open rakiru opened 8 years ago

rakiru commented 8 years ago

Or something like that. It's a pain when you're messing with config files that are in version control. The game could look for whatever.json, then if that's not found look for whatever.json.example and copy that to whatever.json.

iamgreaser commented 8 years ago

I'm all for this. It'd probably have to take place in the launcher when first launched as the permissions are loosened on clsave/pub/... As for the server, we may have to just leave it at svsave/pub/server.json and suggest using the --server=svsave/pub/whatever/foo.json command-line argument... or we could loosen the path restrictions for the server files as well in launcher mode.

rakiru commented 8 years ago

Can't they create files in those folders anyway? I thought that was the point of pub/?

iamgreaser commented 8 years ago

pub/ is for reading, not writing - the reason being that if someone runs a dodgy mod it can clobber pub/. Hence why vol/ is there.

rakiru commented 8 years ago

Ah right, yeah, of course. It should be fine being in the launcher then. The build server could perhaps copy them across too, since it's only really an issue for people grabbing them from git.

As for the server, yeah, command-line arg is fine. We provide a few sample server configs anyway, rather than just one single base.