Currently all civilizations can build palisades, except for rome, presumably because they have siege walls. However, those are available only in the city phase, whereas palisades are buildable in the village phase.
This patch removes this exception, allowing rome to build palisades too. This means the siege walls are an additional wallset, similar to the other civ-specific structures, which don't replace default structures either.
Details
Details
- Reviewers
ValihrAnt Lionkanzen borg- - Group Reviewers
Restricted Owners Package (Owns No Changed Paths) - Commits
- rP23839: Allow Rome to build palisades.
Agree this is an improvement.
Diff Detail
Diff Detail
- Repository
- rP 0 A.D. Public Repository
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Event Timeline
Comment Actions
Successful build - Chance fights ever on the side of the prudent.
Link to build: https://jenkins.wildfiregames.com/job/docker-differential/2144/display/redirect
Comment Actions
I had always assumed Rome lacks palisade walls because of having the Siege walls, which does lead to some awkward situations. First of all, players are entirely incapable of building walls in Phase 1. Later on, it can still be very hard to wall as stone walls are very expensive and stone can be hard to obtain. Additionally giving Rome palisades won't take anything away from the Siege walls as one of the two can only be built in home territory and the other in neutral or opposing territory.
Comment Actions
Doesn't that make the factions more generic, and thus less interesting?
I don't think giving the Romans palisades will make them less interesting as they already have 2 unique things fortification wise: the Siege Walls and the Entrenched army camp, both of which can be built-in neutral and enemy territory.
Comment Actions
Successful build - Chance fights ever on the side of the prudent.
Link to build: https://jenkins.wildfiregames.com/job/docker-differential/2391/display/redirect