0ADMods / terra_magna

Terra Magna - an 0 A.D. extension.
http://www.wildfiregames.com/forum
GNU General Public License v2.0
29 stars 17 forks source link

How do we keep Han and TM in sync? #67

Closed udeved closed 4 years ago

udeved commented 5 years ago

@StanleySweet

I made a little testing branch with the new buildings and added couple of techs. Feel free to test and backport what you deem useful.

The question arises, how do we keep the two mods in sync?

In order to achieve that, I propose some simple points we follow. For example, all xml have a default indent of 4 characters, no tabs. This should already reduce changes of mismatches.

StanleySweet commented 5 years ago

Game standard is two spaces for XML files. I'll have a look at the branch if I have some time.

Keeping it in sync is I believe always gonna be tricky :/

udeved commented 5 years ago

Might be possible to add a git remote to TM and Han serving as upstream repo for Han specific changes.

Or, we consider git-subrepo

We use it in our git infra already successfully.

StanleySweet commented 5 years ago

Yeah but git subrepo works fine when it's a subfolder, here it's not. It's mixed everywhere. If you find a solution though let me know.

udeved commented 5 years ago

Sure, subrepo would require TM to be reorganized, and encapsulate each civ as much as possible. Alternatively, subrepo is folder based, but subrepo supports other nested subrepos, question would be more usuability with a n-depth level of subrepos. But overall, subrepo would be the kind of thing that solved the sync problem best.

StanleySweet commented 5 years ago

You can't really reorganize it. It's like this because of the game.

On Sun, Feb 17, 2019, 5:32 PM artoo <notifications@github.com wrote:

Sure, subrepo would required TM to be reorganized, and encapsulate each civ as much as possible. Alternatively, subrepo is folder based, but subrepo supports other nested subrepos, question would be more usuability with a n-depth level of subrepos. But overall, subrepo would be the kind of thing that solved the sync problem best.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/0ADMods/terra_magna/issues/67#issuecomment-464478002, or mute the thread https://github.com/notifications/unsubscribe-auth/AGhUmuZMptYoUECd2i7lhSARWXWbNh5wks5vOYQkgaJpZM4a_B-f .

StanleySweet commented 4 years ago

Let's remove the chinese faction totally from this mod, and add it as dependency instead.

udeved commented 4 years ago

I would also recommend to split out the xiongnu. The Han military is heavily geared towards a nomadic cavalry enemy, perhaps Han should depend on Xiongnu? TM would become more like a meta mod that pulls factions.

StanleySweet commented 4 years ago

Split out the Xiongnu

Sure they can have their own mod. I dont't have the time to do that though.

Han china depending on Xiongnu

I dont think that's a good idea. Han_china is supposed to be standalone not dependant else this whole enterprise makes no sense

TM meta mod

Yeah I agree. The goal is mod like de do the same.