Persistent-Kingdoms / persistent-factions

Game repo for the Persistent Factions Minetest server
4 stars 2 forks source link

Separate codebases #9

Open appgurueu opened 4 years ago

appgurueu commented 4 years ago

This repo is a collection of different codebases, and as such, poorly manageable. Bugs stack up and we need to port every fix for MTG / factions etc here manually. So I propose that we:

This will mean quite some work at first, but will be extremely useful in the long term. We are running an outdated MTG with tons of outdated mods with many tweaks and fixes by us poured in, making it hard to merge.

The benefits will be great: Straightforward licensing, less work, easy updates, merging, new features, and finally being able to concentrate on our neatly separated PK code.

archfan7411 commented 4 years ago

Overall agreed, the prospect of separating custom(ized) mods into other PK repos is especially alluring. Unsure what you mean by "Separate this codebase from MTG" though, could you elaborate?

appgurueu commented 4 years ago

kingdoms_game extends minetest_game, so we should separate it

archfan7411 commented 4 years ago

How would that be accomplished?

appgurueu commented 4 years ago

By having the MTG mods separated from this mods. You would always get the freshest ones by pulling them. Texmex is one example of a provider of freshly chopped MTG mods

appgurueu commented 4 years ago

I'm not sure about the details, though. We need a way to manage a collection of mods as one game. I'd most likely go with submodules. But apparently there have been issues with that...

appgurueu commented 4 years ago

Other members: what do you think? Pings in no specific order:

luk3yx commented 4 years ago

I suggest something similar to https://github.com/BlockySurvival/bls_mods/ (including the instructions on how to clone and update the repository).

appgurueu commented 4 years ago

Indeed, that's simply awesome. Conclusions:

appgurueu commented 4 years ago

In general, BlockySurvival is a great example for how the entire organization could work. Amazing.

ghost commented 4 years ago

Hola, I am late to the party.

ghost commented 4 years ago

Hmmmmm.

appgurueu commented 4 years ago

First step done ! See our brand new Minetest Game Modpack :tada:

appgurueu commented 4 years ago

Second step done ! Kingdoms Game repo made (uses submodules as well)

appgurueu commented 4 years ago

Ideally, the final step, as I imagine it (probably by the end of the year), will be swapping out Persistent Factions by the Kingdoms Game, keeping the world. @archfan7411

archfan7411 commented 4 years ago

Awesome! Just one minor thing: the games are minetest_game and kingdoms_game but you used dashes in place of underscores. Obviously very minor, it just bugs my OCD. Optional if you want to change it.

appgurueu commented 4 years ago

Yeah, the thing is, GitHub convention pretty much uses hyphens, as perceived by me. So I just adhere to it and use a-b-c instead of a_b_c. Different naming conventions. I would prefer the GH one as it's more visually balanced, the hyphen being in the middle of the line instead of at the bottom (it just looks neater, doesn't it?). At least no lowerCamelCase.

archfan7411 commented 4 years ago

At the risk of bikeshedding, minetest_game is "correct" because that's the actual name of the game :P This is getting offtopic though, so I'll just drop it. Thanks again for going through all that work to make the repos!

appgurueu commented 4 years ago

Change made.