[gameplay] enable cart basic infantry javelineer
Needs ReviewPublic
Actions

Authored by Nescio on Jan 15 2021, 10:58 PM.

Details

Reviewers

None

Group Reviewers

Summary

The ancient Libyans had a reputation for their nimble skirmisher fighting with javelins, so if there is one civ in 0 A.D. that should have an infantry javelineer, it's Carthage.
There is actually a javelineer actor, it's been there for years:

It's not used, presumably because Carthage could already get a javelineer from its Iberian embassy. However, it has also two types of infantry swordsman (Celtic and Italic) and two types of cavalry swordsmen (Celtic and Iberian).
Now mercenaries can no longer gather (D3269/rP24635), Carthage could use another basic citizen.

This patch:

adds templates for the existing actors;
makes the javelineer the default village phase ranged unit for Carthage.
postpones the Mauretanian archer to the town phase.

The barracks is also less empty as a consequence.

Test Plan

Agree this is an improvement.

Diff Detail

Repository

rP 0 A.D. Public Repository

Branch

/ps/trunk

Lint

Lint OK

Unit

No Unit Test Coverage

Build Status

Buildable 15095
Build 32697: Vulcan Build	Jenkins
Build 32696: Vulcan Build (macOS)	Jenkins
Build 32695: Vulcan Build (Windows)	Jenkins
Build 32694: arc lint + arc unit

Event Timeline

Nescio created this revision.Jan 15 2021, 10:58 PM

Owners added a subscriber: Restricted Owners Package.Jan 15 2021, 10:58 PM

Build is green

builderr-debug-macos.txt
ld: warning: text-based stub file /System/Library/Frameworks//CoreAudio.framework/CoreAudio.tbd and library file /System/Library/Frameworks//CoreAudio.framework/CoreAudio are out of sync. Falling back to library file for linking.
ld: warning: text-based stub file /System/Library/Frameworks//AudioToolbox.framework/AudioToolbox.tbd and library file /System/Library/Frameworks//AudioToolbox.framework/AudioToolbox are out of sync. Falling back to library file for linking.
ld: warning: text-based stub file /System/Library/Frameworks//ForceFeedback.framework/ForceFeedback.tbd and library file /System/Library/Frameworks//ForceFeedback.framework/ForceFeedback are out of sync. Falling back to library file for linking.
ld: warning: text-based stub file /System/Library/Frameworks//CoreVideo.framework/CoreVideo.tbd and library file /System/Library/Frameworks//CoreVideo.framework/CoreVideo are out of sync. Falling back to library file for linking.
ld: warning: text-based stu

See https://jenkins.wildfiregames.com/job/macos-differential/2887/display/redirect for more details.

Harbormaster completed remote builds in B15095: Diff 15361.Jan 15 2021, 11:03 PM

Nescio requested review of this revision.Jan 15 2021, 11:03 PM

I don't have a strong opinion here. With the change in mercenaries, I think it might be interesting to have one more ranged unit collecting resources.

Nescio mentioned this in D3390: [gameplay] new cart market civ bonus.Jan 16 2021, 10:15 PM

wowgetoffyourcellphone added a subscriber: wowgetoffyourcellphone.Jan 17 2021, 2:22 AM

If you'd like, you can take inspiration for their look from Delenda Est:

https://i.imgur.com/bj57p5b.jpg

Lionkanzen awarded a token.Mar 24 2021, 6:34 AM

Lionkanzen added a subscriber: Lionkanzen.

I dont see any problem. good change for early.