The “unlock shared dropsites” technology is now independent from the “unlock shared vision” technology (D2999/rP24541) and available at the civic centre (D3321/rP24605) in the village phase, to encourage cooperation at game start.
However, it still has a three traders requirement, which is problematic, since you need to be in the town phase to be able to build a market to train traders.
This patch removes that requirement.
Details
Agree.
Diff Detail
- Repository
- rP 0 A.D. Public Repository
- Branch
- /ps/trunk
- Lint
Lint OK - Unit
No Unit Test Coverage - Build Status
Buildable 15218 Build 33067: Vulcan Build Jenkins Build 33066: Vulcan Build (macOS) Jenkins Build 33065: Vulcan Build (Windows) Jenkins Build 33064: arc lint + arc unit
Event Timeline
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/2953/display/redirect for more details.
Er actually Freagarach convinced me that this is a fun requirement -> I'd just add that it requires phase_town to make things explicit.
To encourage team play from the beginning, we'd need a "Shared Allied Vision" option in game setup instead of moving the Shared Allied Vision tech to Village phase. Perhaps enabling the option could auto-research the tech.
"Standard" vision enabled would keep the tech and it would require Town Phase. I would require a Market though, instead of an arbitrary 3 traders. I don't know of any Diffs specifically for this tech/issue though, so I'm mentioning it here.
For this particular tech (Shared Dropsites), perhaps also require a Market, or make it supersede Shared Vision. So, Shared Vision + Market = Shared Dropsites?
Just some ideas.