Page MenuHomeWildfire Games

Correct sword prop for three units
Needs ReviewPublic

Authored by Nescio on Tue, Jul 2, 12:31 AM.

Details

Reviewers
None
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Summary

This patch:

  • changes the weapon of cart and gaul infantry swordmen to (Celtic) csword, because all other Celtic swordsmen (cavalry, champions, heroes, ptol mercenaries) use it already and the spatha is a Roman cavalry sword;
  • changes the weapon of athen cavalry swordsmen to falcata (kopis/makhaira), a chopping sword, because ancient sources (e.g. Xenophon) emphasize cavalry used that, not a xiphos (similar to gladius or akinakes), a short stabbing sword, unsuitable for horsemen
Test Plan

Ought to be unproblematic.

Diff Detail

Repository
rP 0 A.D. Public Repository
Branch
/ps/trunk
Lint
Lint OK
Unit
No Unit Test Coverage
Build Status
Buildable 8155
Build 13281: Vulcan BuildJenkins
Build 13280: arc lint + arc unit

Event Timeline

Nescio created this revision.Tue, Jul 2, 12:31 AM
Owners added a subscriber: Restricted Owners Package.Tue, Jul 2, 12:31 AM

Successful build - Chance fights ever on the side of the prudent.

Link to build: https://jenkins.wildfiregames.com/job/differential/1873/display/redirect

Nescio added a comment.EditedTue, Jul 2, 12:34 AM

Separatedly, someone with more knowledge and experience of the art/ files should correct the following:

  • "akin" [sic] into "akinakes"
  • "gladus" [sic] into "gladius"
  • "pillum" [sic] into "pilum"
  • "tessalian" [sic] into "thessalian"

(involves file moves etc.).

Nescio edited reviewers, added: Restricted Owners Package; removed: Stan.Sat, Jul 6, 4:26 PM