Merge lp:~wl-zocker/widelands/conf_files_changes into lp:widelands

Proposed by wl-zocker
Status: Merged
Merged at revision: 7068
Proposed branch: lp:~wl-zocker/widelands/conf_files_changes
Merge into: lp:widelands
Diff against target: 112 lines (+10/-11)
9 files modified
tribes/atlanteans/farm/conf (+1/-0)
tribes/atlanteans/horse/conf (+1/-1)
tribes/atlanteans/horsefarm/conf (+1/-2)
tribes/barbarians/cattlefarm/conf (+1/-2)
tribes/barbarians/cloth/conf (+2/-1)
tribes/barbarians/ox/conf (+1/-1)
tribes/barbarians/weaving-mill/conf (+1/-1)
tribes/empire/donkey/conf (+1/-1)
tribes/empire/donkeyfarm/conf (+1/-2)
To merge this branch: bzr merge lp:~wl-zocker/widelands/conf_files_changes
Reviewer Review Type Date Requested Status
SirVer Approve
Review via email: mp+226384@code.launchpad.net

Description of the change

This merge request includes three issues:
- Transportation animals have been produced when they were needed, no matter if the economy had enough wares (wheat/corn and water) or not. But they were also produced when there was no need for them, only because the economy had the wares to do so. This is probably the reason why the AI always has so many animals in its warehouses at the end of the game. I removed the last case; the animals are only produced when they are needed. I increased their default_target_quantity a bit - just to be sure.
- The same as above for the cloth of the Barbarians. The weaving-mill used to consume all thatch reed. This was not no big problem, but it caused road traffic. The default_target_quantity is enough for two ships (4 cloth each).
- Last but not least, I fixed bug 1332455. Seemingly, when no return was defined, this was interpreted as success.

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

Good changes! You become quite adept at those conf files tweaks :)

Cannot merge right now, but I think this should go in.

review: Approve
Revision history for this message
wl-zocker (wl-zocker) wrote :

Thanks. Are there other things that come to your mind and that only require relatively small changes in the conf files (bigger projects may lay around for some months, and I do not want to delay things others could do, too)? Otherwise, I will just search through the bug reports more or less randomly.

Revision history for this message
SirVer (sirver) wrote :

I have nothing in particular right now. It would be great if you can pick up more bugs from the bug tracker of course :)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'tribes/atlanteans/farm/conf'
2--- tribes/atlanteans/farm/conf 2014-03-17 17:23:26 +0000
3+++ tribes/atlanteans/farm/conf 2014-07-10 21:16:32 +0000
4@@ -37,6 +37,7 @@
5 call=plant_corn
6 call=harvest_corn
7 playFX=../../../sound/farm/farm_animal 192
8+return=skipped
9
10 [idle]
11 pics=farm_i_??.png # ???
12
13=== modified file 'tribes/atlanteans/horse/conf'
14--- tribes/atlanteans/horse/conf 2014-03-14 12:38:09 +0000
15+++ tribes/atlanteans/horse/conf 2014-07-10 21:16:32 +0000
16@@ -1,5 +1,5 @@
17 help=_Horses help to carry items along busy roads. They are reared in a horse farm.
18-default_target_quantity=5
19+default_target_quantity=10
20 ware_hotspot=-2 12
21
22 [idle]
23
24=== modified file 'tribes/atlanteans/horsefarm/conf'
25--- tribes/atlanteans/horsefarm/conf 2014-07-03 20:06:38 +0000
26+++ tribes/atlanteans/horsefarm/conf 2014-07-10 21:16:32 +0000
27@@ -26,8 +26,7 @@
28
29 [work]
30 sleep=15000
31-return=skipped unless economy needs horse or not economy needs water
32-return=skipped unless economy needs horse or not economy needs corn
33+return=skipped unless economy needs horse
34 consume=corn water
35 playFX=../../../sound/farm/horse 192
36 animate=working 15000 # feeding cute little foals ;)
37
38=== modified file 'tribes/barbarians/cattlefarm/conf'
39--- tribes/barbarians/cattlefarm/conf 2014-07-03 20:06:38 +0000
40+++ tribes/barbarians/cattlefarm/conf 2014-07-10 21:16:32 +0000
41@@ -26,8 +26,7 @@
42
43 [work]
44 sleep=15000
45-return=skipped unless economy needs ox or not economy needs wheat
46-return=skipped unless economy needs ox or not economy needs water
47+return=skipped unless economy needs ox
48 consume=wheat water
49 playFX=../../../sound/farm/ox 192
50 animate=working 15000 # animation of feeding the cattles
51
52=== modified file 'tribes/barbarians/cloth/conf'
53--- tribes/barbarians/cloth/conf 2013-07-23 19:04:12 +0000
54+++ tribes/barbarians/cloth/conf 2014-07-10 21:16:32 +0000
55@@ -1,6 +1,7 @@
56-# TODO: this is currently not shown in warehouses as it is not used.
57 help=_Cloth is needed for barbarian ships. It is produced out of thatch reed.
58
59+default_target_quantity=10
60+
61 [idle]
62 pics=idle.png
63 hotspot=6 9
64
65=== modified file 'tribes/barbarians/ox/conf'
66--- tribes/barbarians/ox/conf 2014-03-14 12:38:09 +0000
67+++ tribes/barbarians/ox/conf 2014-07-10 21:16:32 +0000
68@@ -1,5 +1,5 @@
69 help=_Oxen help to carry items along busy roads. They are reared in a cattle farm.
70-default_target_quantity=5
71+default_target_quantity=10
72 ware_hotspot=-2 13
73
74 [idle]
75
76=== modified file 'tribes/barbarians/weaving-mill/conf'
77--- tribes/barbarians/weaving-mill/conf 2014-07-03 20:06:38 +0000
78+++ tribes/barbarians/weaving-mill/conf 2014-07-10 21:16:32 +0000
79@@ -24,7 +24,7 @@
80
81 [work]
82 sleep=20000
83-return=skipped unless economy needs cloth or not economy needs thatchreed
84+return=skipped unless economy needs cloth
85 consume=thatchreed
86 animate=working 25000
87 produce=cloth
88
89=== modified file 'tribes/empire/donkey/conf'
90--- tribes/empire/donkey/conf 2014-03-14 12:38:09 +0000
91+++ tribes/empire/donkey/conf 2014-07-10 21:16:32 +0000
92@@ -1,5 +1,5 @@
93 help=_Donkeys help to carry items along busy roads. They are reared in a donkey farm.
94-default_target_quantity=5
95+default_target_quantity=10
96 ware_hotspot=-2 8
97
98 [idle]
99
100=== modified file 'tribes/empire/donkeyfarm/conf'
101--- tribes/empire/donkeyfarm/conf 2014-07-03 20:06:38 +0000
102+++ tribes/empire/donkeyfarm/conf 2014-07-10 21:16:32 +0000
103@@ -26,8 +26,7 @@
104
105 [work]
106 sleep=15000
107-return=skipped unless economy needs donkey or not economy needs wheat
108-return=skipped unless economy needs donkey or not economy needs water
109+return=skipped unless economy needs donkey
110 consume=wheat water
111 playFX=../../../sound/farm/donkey 192
112 animate=working 15000 # feeding cute little baby donkeys ;)

Subscribers

People subscribed via source and target branches

to status/vote changes: