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
- 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
- Repository
- rP 0 A.D. Public Repository
- Branch
- /ps/trunk
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 12041 Build 22660: Vulcan Build Jenkins Build 22659: Vulcan Build (Windows) Jenkins Build 22658: arc lint + arc unit
Event Timeline
Successful build - Chance fights ever on the side of the prudent.
Link to build: https://jenkins.wildfiregames.com/job/docker-differential/2144/display/redirect
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.
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.
Successful build - Chance fights ever on the side of the prudent.
Link to build: https://jenkins.wildfiregames.com/job/docker-differential/2391/display/redirect