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
=== modified file 'data/tribes/wares/blackroot/init.lua'
--- data/tribes/wares/blackroot/init.lua 2015-12-11 16:54:00 +0000
+++ data/tribes/wares/blackroot/init.lua 2016-04-13 15:50:42 +0000
@@ -7,9 +7,7 @@
7 descname = pgettext("ware", "Blackroot"),7 descname = pgettext("ware", "Blackroot"),
8 helptext_script = dirname .. "helptexts.lua",8 helptext_script = dirname .. "helptexts.lua",
9 icon = dirname .. "menu.png",9 icon = dirname .. "menu.png",
10 default_target_quantity = {10 default_target_quantity = {},
11 atlanteans = 20
12 },
13 preciousness = {11 preciousness = {
14 atlanteans = 1012 atlanteans = 10
15 },13 },
1614
=== modified file 'data/tribes/wares/corn/init.lua'
--- data/tribes/wares/corn/init.lua 2015-12-11 16:54:00 +0000
+++ data/tribes/wares/corn/init.lua 2016-04-13 15:50:42 +0000
@@ -7,9 +7,7 @@
7 descname = pgettext("ware", "Corn"),7 descname = pgettext("ware", "Corn"),
8 helptext_script = dirname .. "helptexts.lua",8 helptext_script = dirname .. "helptexts.lua",
9 icon = dirname .. "menu.png",9 icon = dirname .. "menu.png",
10 default_target_quantity = {10 default_target_quantity = {},
11 atlanteans = 25
12 },
13 preciousness = {11 preciousness = {
14 atlanteans = 1212 atlanteans = 12
15 },13 },
1614
=== modified file 'data/tribes/wares/fish/init.lua'
--- data/tribes/wares/fish/init.lua 2015-12-11 16:54:00 +0000
+++ data/tribes/wares/fish/init.lua 2016-04-13 15:50:42 +0000
@@ -7,11 +7,7 @@
7 descname = pgettext("ware", "Fish"),7 descname = pgettext("ware", "Fish"),
8 helptext_script = dirname .. "helptexts.lua",8 helptext_script = dirname .. "helptexts.lua",
9 icon = dirname .. "menu.png",9 icon = dirname .. "menu.png",
10 default_target_quantity = {10 default_target_quantity = {},
11 atlanteans = 20,
12 barbarians = 20,
13 empire = 10
14 },
15 preciousness = {11 preciousness = {
16 atlanteans = 4,12 atlanteans = 4,
17 barbarians = 3,13 barbarians = 3,
1814
=== modified file 'data/tribes/wares/grape/init.lua'
--- data/tribes/wares/grape/init.lua 2015-12-11 16:54:00 +0000
+++ data/tribes/wares/grape/init.lua 2016-04-13 15:50:42 +0000
@@ -7,9 +7,7 @@
7 descname = pgettext("ware", "Grape"),7 descname = pgettext("ware", "Grape"),
8 helptext_script = dirname .. "helptexts.lua",8 helptext_script = dirname .. "helptexts.lua",
9 icon = dirname .. "menu.png",9 icon = dirname .. "menu.png",
10 default_target_quantity = {10 default_target_quantity = {},
11 empire = 20
12 },
13 preciousness = {11 preciousness = {
14 empire = 1012 empire = 10
15 },13 },
1614
=== modified file 'data/tribes/wares/log/init.lua'
--- data/tribes/wares/log/init.lua 2015-12-11 16:54:00 +0000
+++ data/tribes/wares/log/init.lua 2016-04-13 15:50:42 +0000
@@ -7,11 +7,7 @@
7 descname = pgettext("ware", "Log"),7 descname = pgettext("ware", "Log"),
8 helptext_script = dirname .. "helptexts.lua",8 helptext_script = dirname .. "helptexts.lua",
9 icon = dirname .. "menu.png",9 icon = dirname .. "menu.png",
10 default_target_quantity = {10 default_target_quantity = {},
11 atlanteans = 40,
12 barbarians = 40,
13 empire = 40
14 },
15 preciousness = {11 preciousness = {
16 atlanteans = 14,12 atlanteans = 14,
17 barbarians = 14,13 barbarians = 14,
1814
=== modified file 'data/tribes/wares/thatch_reed/init.lua'
--- data/tribes/wares/thatch_reed/init.lua 2015-12-11 16:54:00 +0000
+++ data/tribes/wares/thatch_reed/init.lua 2016-04-13 15:50:42 +0000
@@ -7,9 +7,7 @@
7 descname = pgettext("ware", "Thatch Reed"),7 descname = pgettext("ware", "Thatch Reed"),
8 helptext_script = dirname .. "helptexts.lua",8 helptext_script = dirname .. "helptexts.lua",
9 icon = dirname .. "menu.png",9 icon = dirname .. "menu.png",
10 default_target_quantity = {10 default_target_quantity = {},
11 barbarians = 10
12 },
13 preciousness = {11 preciousness = {
14 barbarians = 512 barbarians = 5
15 },13 },
1614
=== modified file 'data/tribes/wares/water/init.lua'
--- data/tribes/wares/water/init.lua 2015-12-11 16:54:00 +0000
+++ data/tribes/wares/water/init.lua 2016-04-13 15:50:42 +0000
@@ -7,11 +7,7 @@
7 descname = pgettext("ware", "Water"),7 descname = pgettext("ware", "Water"),
8 helptext_script = dirname .. "helptexts.lua",8 helptext_script = dirname .. "helptexts.lua",
9 icon = dirname .. "menu.png",9 icon = dirname .. "menu.png",
10 default_target_quantity = {10 default_target_quantity = {},
11 atlanteans = 25,
12 barbarians = 25,
13 empire = 25
14 },
15 preciousness = {11 preciousness = {
16 atlanteans = 7,12 atlanteans = 7,
17 barbarians = 8,13 barbarians = 8,
1814
=== modified file 'data/tribes/wares/wheat/init.lua'
--- data/tribes/wares/wheat/init.lua 2015-12-11 16:54:00 +0000
+++ data/tribes/wares/wheat/init.lua 2016-04-13 15:50:42 +0000
@@ -7,10 +7,7 @@
7 descname = pgettext("ware", "Wheat"),7 descname = pgettext("ware", "Wheat"),
8 helptext_script = dirname .. "helptexts.lua",8 helptext_script = dirname .. "helptexts.lua",
9 icon = dirname .. "menu.png",9 icon = dirname .. "menu.png",
10 default_target_quantity = {10 default_target_quantity = {},
11 barbarians = 25,
12 empire = 25
13 },
14 preciousness = {11 preciousness = {
15 barbarians = 12,12 barbarians = 12,
16 empire = 1213 empire = 12

Subscribers

People subscribed via source and target branches

to status/vote changes: