Page MenuHomeWildfire Games

Regrouped special technologies
Needs ReviewPublic

Authored by Nescio on Sep 8 2017, 12:41 PM.
This revision needs review, but there are no reviewers specified.

Details

Reviewers
None
Summary

D1472 groups all civbonus technologies under technologies/civbonuses/
D1775 deletes unused technologies that were deprecated years ago

Special technologies are technologies that have a "civ": or "notciv": requirement, i.e. they're available to one or more factions, but not all. Currently some are grouped alongside regular technologies, others are arbitrarily distibuted over the carthaginians/, hellenes/, kushites/, mauryas/, persians/, romans/, and successors/ subdirectories.
This patch:

  • merges identical technologies (archery tradition, naval architects)
  • moves special technologies that are used out of those subdirectories, occassionally implementing more consistent naming
  • updates specific structure template production queues accordingly
Test Plan

Check if everything still works (I encountered some svn:mime-type errors when running arc diff --preview).

Diff Detail

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

Event Timeline

Nescio created this revision.Sep 8 2017, 12:41 PM
Owners added a subscriber: Restricted Owners Package.Sep 8 2017, 12:41 PM
Nescio updated this revision to Diff 3579.Sep 8 2017, 4:24 PM
Nescio retitled this revision from Grouped all civbonuses under /technologies/civbonuses/ to Regrouped civbonuses and special technologies.
Nescio edited the summary of this revision. (Show Details)
Nescio edited the test plan for this revision. (Show Details)
Nescio added a reviewer: fatherbushido.

Regrouped all civbonuses under /technologies/civbonuses/ *and* all special technologies under /technologies/special/

Reason: I was informed it was more important that diffs are complete than that they're small.

Nescio edited the summary of this revision. (Show Details)Sep 8 2017, 4:25 PM
Nescio added a reviewer: bb.Sep 8 2017, 11:42 PM
fatherbushido resigned from this revision.EditedSep 9 2017, 6:58 AM

I have a lot of respect for your work. I'd say also that your mod really shows your ability to manage such things (even if I am basically not fond of those hard bonuses). So, if I was the guy who gives congratulations, I'd congratulate you.
I also share your observation/analysis.
For information there were such a suggestion: (staff only) https://wildfiregames.com/forum/index.php?/topic/21563-subfolders-in-simulationdatatechnologies/
But I won't review your diff, sorry.

Nescio updated this revision to Diff 3603.Sep 9 2017, 8:50 AM
Nescio edited the summary of this revision. (Show Details)

Minor correction

Vulcan added a subscriber: Vulcan.Sep 9 2017, 9:37 AM

Build is green

Updating workspaces.
Build (release)...
Build (debug)...
Running release tests...
Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK!
Running debug tests...
Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK!
Checking XML files...

http://jenkins-master:8080/job/phabricator/2002/ for more details.

Nescio edited the summary of this revision. (Show Details)Sep 9 2017, 1:19 PM
Nescio updated this revision to Diff 7485.Feb 16 2019, 10:29 AM
Nescio edited the summary of this revision. (Show Details)
Nescio edited the test plan for this revision. (Show Details)
Nescio removed reviewers: fatherbushido, bb.
Nescio edited the summary of this revision. (Show Details)
Nescio edited the test plan for this revision. (Show Details)
Nescio added subscribers: Stan, fatherbushido.

Build failure - The Moirai have given mortals hearts that can endure.

Link to build: https://jenkins.wildfiregames.com/job/differential/1057/

Nescio edited the summary of this revision. (Show Details)Feb 16 2019, 10:31 AM
Nescio retitled this revision from Regrouped civbonuses and special technologies to Regrouped special technologies.Feb 16 2019, 10:35 AM