Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/amazons-coding-changes into lp:widelands

2019-08-23 Thread TiborB
So some sites (barbarian well and forester) will also "upgrade" land types as a 
side effect of cutting wood or mining water? Anyway this does not seem 
dangerous to me.
What I dont understand is capability to build a sentry over a tree. Did you 
create another buildable field type, because this can not be handled by SMALL 
buildcaps... (I know that I can look into code, but the diff is too long.
My basic question is - does this affect AI on current 4 tribes?

-- 
https://code.launchpad.net/~widelands-dev/widelands/amazons-coding-changes/+merge/371725
Your team Widelands Developers is requested to review the proposed merge of 
lp:~widelands-dev/widelands/amazons-coding-changes into lp:widelands.

___
Mailing list: https://launchpad.net/~widelands-dev
Post to : widelands-dev@lists.launchpad.net
Unsubscribe : https://launchpad.net/~widelands-dev
More help   : https://help.launchpad.net/ListHelp


Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/amazons-coding-changes into lp:widelands

2019-08-23 Thread TiborB
"treetop" means - in the branches of the mature tree, or does the building 
replace a tree?
-- 
https://code.launchpad.net/~widelands-dev/widelands/amazons-coding-changes/+merge/371725
Your team Widelands Developers is requested to review the proposed merge of 
lp:~widelands-dev/widelands/amazons-coding-changes into lp:widelands.

___
Mailing list: https://launchpad.net/~widelands-dev
Post to : widelands-dev@lists.launchpad.net
Unsubscribe : https://launchpad.net/~widelands-dev
More help   : https://help.launchpad.net/ListHelp


Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/amazons-coding-changes into lp:widelands

2019-08-23 Thread Benedikt Straub
The barbarian sites modified in this branch are only testcases, not intended 
for merging. 
The new amazons tribe (under development in another branch) will have a 
"Gardening Center" whose only purpose is that the worker walks around and turns 
the terrain nearby into a different, more fertile terrain type.

The sentry idea is that this "Treetop Sentry" is a platform in the branches of 
a tree. Internally, I have created a value "built_over_immovable" in addition 
to the Size value – a building with this value set can be built only on a 
player-owned node where the max nodecaps is >= the building´s size and an 
immovable with the required attribute (in this case: big tree) stands on the 
node. The immovable is then removed when placing the building, but I hacked the 
animations so it looks as if it were still there.

The AI is currently unable to build this sentry, and it won´t build a gardening 
center either, but otherwise it should be unaffected.
-- 
https://code.launchpad.net/~widelands-dev/widelands/amazons-coding-changes/+merge/371725
Your team Widelands Developers is requested to review the proposed merge of 
lp:~widelands-dev/widelands/amazons-coding-changes into lp:widelands.

___
Mailing list: https://launchpad.net/~widelands-dev
Post to : widelands-dev@lists.launchpad.net
Unsubscribe : https://launchpad.net/~widelands-dev
More help   : https://help.launchpad.net/ListHelp


Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/amazons-coding-changes into lp:widelands

2019-08-23 Thread Benedikt Straub
For more information about the concepts, see the discussion at 
https://www.widelands.org/forum/post/29315/ onwards…
-- 
https://code.launchpad.net/~widelands-dev/widelands/amazons-coding-changes/+merge/371725
Your team Widelands Developers is requested to review the proposed merge of 
lp:~widelands-dev/widelands/amazons-coding-changes into lp:widelands.

___
Mailing list: https://launchpad.net/~widelands-dev
Post to : widelands-dev@lists.launchpad.net
Unsubscribe : https://launchpad.net/~widelands-dev
More help   : https://help.launchpad.net/ListHelp


[Widelands-dev] [Merge] lp:~widelands-dev/widelands/amazons-coding-changes into lp:widelands

2019-08-23 Thread bunnybot
Continuous integration builds have changed state:

Travis build 5337. State: errored. Details: 
https://travis-ci.org/widelands/widelands/builds/575756820.
Appveyor build 5108. State: success. Details: 
https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_amazons_coding_changes-5108.
-- 
https://code.launchpad.net/~widelands-dev/widelands/amazons-coding-changes/+merge/371725
Your team Widelands Developers is requested to review the proposed merge of 
lp:~widelands-dev/widelands/amazons-coding-changes into lp:widelands.

___
Mailing list: https://launchpad.net/~widelands-dev
Post to : widelands-dev@lists.launchpad.net
Unsubscribe : https://launchpad.net/~widelands-dev
More help   : https://help.launchpad.net/ListHelp