Merge lp:~widelands-dev/widelands/bug-1751440-smugglers-desync into lp:widelands
Proposed by
GunChleoc
Status: | Work in progress | ||||
---|---|---|---|---|---|
Proposed branch: | lp:~widelands-dev/widelands/bug-1751440-smugglers-desync | ||||
Merge into: | lp:widelands | ||||
Diff against target: |
96 lines (+48/-20) 1 file modified
data/maps/MP_Scenarios/Smugglers.wmf/scripting/smuggling.lua (+48/-20) |
||||
To merge this branch: | bzr merge lp:~widelands-dev/widelands/bug-1751440-smugglers-desync | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Widelands Developers | Pending | ||
Review via email: mp+340735@code.launchpad.net |
Commit message
To address desync problems in Smugglers, going round robin rather than smuggling a random ware. Also, allow smuggling if players are playing different tribes - only wares that both tribes have will be smuggled.
Description of the change
I'm not 100% sure if this branch actually fixes the problem, by I was able to play a full game with 2 players having 2 smuggling routes and the 2 other players idle.
Also, we can now expand the scenario in the future to allow players to choose their tribes, but we need to finish the game loading screen UI first before we can start working on that.
ETA: Just got another desync, so it's not fixed.
To post a comment you must log in.