Merge lp:~widelands-dev/widelands/bug_1480961_configure_economy into lp:widelands

Proposed by kaputtnik
Status: Merged
Merged at revision: 7959
Proposed branch: lp:~widelands-dev/widelands/bug_1480961_configure_economy
Merge into: lp:widelands
Diff against target: 126 lines (+8/-31)
8 files modified
data/tribes/wares/blackroot/init.lua (+1/-3)
data/tribes/wares/corn/init.lua (+1/-3)
data/tribes/wares/fish/init.lua (+1/-5)
data/tribes/wares/grape/init.lua (+1/-3)
data/tribes/wares/log/init.lua (+1/-5)
data/tribes/wares/thatch_reed/init.lua (+1/-3)
data/tribes/wares/water/init.lua (+1/-5)
data/tribes/wares/wheat/init.lua (+1/-4)
To merge this branch: bzr merge lp:~widelands-dev/widelands/bug_1480961_configure_economy
Reviewer Review Type Date Requested Status
GunChleoc Approve
Review via email: mp+291669@code.launchpad.net

Commit message

Hide wares from configure economy which does not need prerequisites and therefor are produced 'endless'

Description of the change

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

Not tested yet, but code LGTM.

@Tibor: does this affect the AI in any way?

Revision history for this message
bunnybot (widelandsofficial) wrote :

Continuous integration builds have changed state:

Travis build 1004. State: passed. Details: https://travis-ci.org/widelands/widelands/builds/122593739.
Appveyor build 837. State: failed. Details: https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_bug_1480961_configure_economy-837.

Revision history for this message
TiborB (tiborb95) wrote :

I would say it is safe for AI

However I am not sure that it is good for granite and marble - they can be mined also in mine (correct?) and a mine can decide that no further mining is needed...

Revision history for this message
GunChleoc (gunchleoc) wrote :

Good point - granite and marble need to stay in. I have tested this now, so once you have made the change, this can be merged.

review: Approve
Revision history for this message
kaputtnik (franku) wrote :

Thanks Tibor and GunChleoc :-)

Hopefully i didn't forgot any other ware.

@bunnybot merge

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'data/tribes/wares/blackroot/init.lua'
2--- data/tribes/wares/blackroot/init.lua 2015-12-11 16:54:00 +0000
3+++ data/tribes/wares/blackroot/init.lua 2016-04-13 15:50:42 +0000
4@@ -7,9 +7,7 @@
5 descname = pgettext("ware", "Blackroot"),
6 helptext_script = dirname .. "helptexts.lua",
7 icon = dirname .. "menu.png",
8- default_target_quantity = {
9- atlanteans = 20
10- },
11+ default_target_quantity = {},
12 preciousness = {
13 atlanteans = 10
14 },
15
16=== modified file 'data/tribes/wares/corn/init.lua'
17--- data/tribes/wares/corn/init.lua 2015-12-11 16:54:00 +0000
18+++ data/tribes/wares/corn/init.lua 2016-04-13 15:50:42 +0000
19@@ -7,9 +7,7 @@
20 descname = pgettext("ware", "Corn"),
21 helptext_script = dirname .. "helptexts.lua",
22 icon = dirname .. "menu.png",
23- default_target_quantity = {
24- atlanteans = 25
25- },
26+ default_target_quantity = {},
27 preciousness = {
28 atlanteans = 12
29 },
30
31=== modified file 'data/tribes/wares/fish/init.lua'
32--- data/tribes/wares/fish/init.lua 2015-12-11 16:54:00 +0000
33+++ data/tribes/wares/fish/init.lua 2016-04-13 15:50:42 +0000
34@@ -7,11 +7,7 @@
35 descname = pgettext("ware", "Fish"),
36 helptext_script = dirname .. "helptexts.lua",
37 icon = dirname .. "menu.png",
38- default_target_quantity = {
39- atlanteans = 20,
40- barbarians = 20,
41- empire = 10
42- },
43+ default_target_quantity = {},
44 preciousness = {
45 atlanteans = 4,
46 barbarians = 3,
47
48=== modified file 'data/tribes/wares/grape/init.lua'
49--- data/tribes/wares/grape/init.lua 2015-12-11 16:54:00 +0000
50+++ data/tribes/wares/grape/init.lua 2016-04-13 15:50:42 +0000
51@@ -7,9 +7,7 @@
52 descname = pgettext("ware", "Grape"),
53 helptext_script = dirname .. "helptexts.lua",
54 icon = dirname .. "menu.png",
55- default_target_quantity = {
56- empire = 20
57- },
58+ default_target_quantity = {},
59 preciousness = {
60 empire = 10
61 },
62
63=== modified file 'data/tribes/wares/log/init.lua'
64--- data/tribes/wares/log/init.lua 2015-12-11 16:54:00 +0000
65+++ data/tribes/wares/log/init.lua 2016-04-13 15:50:42 +0000
66@@ -7,11 +7,7 @@
67 descname = pgettext("ware", "Log"),
68 helptext_script = dirname .. "helptexts.lua",
69 icon = dirname .. "menu.png",
70- default_target_quantity = {
71- atlanteans = 40,
72- barbarians = 40,
73- empire = 40
74- },
75+ default_target_quantity = {},
76 preciousness = {
77 atlanteans = 14,
78 barbarians = 14,
79
80=== modified file 'data/tribes/wares/thatch_reed/init.lua'
81--- data/tribes/wares/thatch_reed/init.lua 2015-12-11 16:54:00 +0000
82+++ data/tribes/wares/thatch_reed/init.lua 2016-04-13 15:50:42 +0000
83@@ -7,9 +7,7 @@
84 descname = pgettext("ware", "Thatch Reed"),
85 helptext_script = dirname .. "helptexts.lua",
86 icon = dirname .. "menu.png",
87- default_target_quantity = {
88- barbarians = 10
89- },
90+ default_target_quantity = {},
91 preciousness = {
92 barbarians = 5
93 },
94
95=== modified file 'data/tribes/wares/water/init.lua'
96--- data/tribes/wares/water/init.lua 2015-12-11 16:54:00 +0000
97+++ data/tribes/wares/water/init.lua 2016-04-13 15:50:42 +0000
98@@ -7,11 +7,7 @@
99 descname = pgettext("ware", "Water"),
100 helptext_script = dirname .. "helptexts.lua",
101 icon = dirname .. "menu.png",
102- default_target_quantity = {
103- atlanteans = 25,
104- barbarians = 25,
105- empire = 25
106- },
107+ default_target_quantity = {},
108 preciousness = {
109 atlanteans = 7,
110 barbarians = 8,
111
112=== modified file 'data/tribes/wares/wheat/init.lua'
113--- data/tribes/wares/wheat/init.lua 2015-12-11 16:54:00 +0000
114+++ data/tribes/wares/wheat/init.lua 2016-04-13 15:50:42 +0000
115@@ -7,10 +7,7 @@
116 descname = pgettext("ware", "Wheat"),
117 helptext_script = dirname .. "helptexts.lua",
118 icon = dirname .. "menu.png",
119- default_target_quantity = {
120- barbarians = 25,
121- empire = 25
122- },
123+ default_target_quantity = {},
124 preciousness = {
125 barbarians = 12,
126 empire = 12

Subscribers

People subscribed via source and target branches

to status/vote changes: