Merge lp://staging/~fakeuserxxxx-deactivatedaccount/widelands/farm_sounds into lp://staging/widelands

Proposed by Fake Name
Status: Merged
Merged at revision: not available
Proposed branch: lp://staging/~fakeuserxxxx-deactivatedaccount/widelands/farm_sounds
Merge into: lp://staging/widelands
Diff against target: 59 lines (+6/-0)
3 files modified
tribes/atlanteans/horsefarm/conf (+2/-0)
tribes/barbarians/cattlefarm/conf (+2/-0)
tribes/empire/donkeyfarm/conf (+2/-0)
To merge this branch: bzr merge lp://staging/~fakeuserxxxx-deactivatedaccount/widelands/farm_sounds
Reviewer Review Type Date Requested Status
SirVer Needs Information
Review via email: mp+22318@code.staging.launchpad.net

Description of the change

Adds a few sounds for cattle/donkey/horse farms to make it more alive.

To post a comment you must log in.
Revision history for this message
SirVer (sirver) wrote :

Soeb, there seem to be some functional changes in this:
3 -return=skipped when economy needs water and not economy needs horse
24 -return=skipped when economy needs corn and not economy needs horse

was this intended? This can't go to build15 then.

review: Needs Information
Revision history for this message
Fake Name (fakeuserxxxx-deactivatedaccount) wrote :

Sorry, no. I was checking how it fits and must've forgot to revert this. I pushed a new revision fixing this.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to status/vote changes: