Re: [Widelands-dev] [Merge] lp:~gunchleoc/widelands/bug-1454371 into lp:widelands

2015-06-15 Thread TiborB
Oh, it seems I am not allowed to push here with "Transport operation not possible: readonly transport" -- https://code.launchpad.net/~gunchleoc/widelands/bug-1454371/+merge/261375 Your team Widelands Developers is requested to review the proposed merge of lp:~gunchleoc/widelands/bug-1454371

Re: [Widelands-dev] [Merge] lp:~gunchleoc/widelands/bug-1454371 into lp:widelands

2015-06-15 Thread GunChleoc
I have added a comment on the line where we need to add the mines restriction back in. Could you please take care of it for me, because I'm away from my developer machine? Diff comments: > === modified file 'src/chat/chat.h' > --- src/chat/chat.h 2014-09-20 09:37:47 + > +++ src/chat/chat.

Re: [Widelands-dev] [Merge] lp:~gunchleoc/widelands/bug-1454371 into lp:widelands

2015-06-10 Thread TiborB
What I am bit afraid is that there will be too many messages (notes) to AI now (AI has to go over all buildings to identify the proper one), because the counter is gone - that itself increases the count of messages 10-fold per a building (specifically mines - as only these used to send&receive t

[Widelands-dev] [Merge] lp:~gunchleoc/widelands/bug-1454371 into lp:widelands

2015-06-08 Thread GunChleoc
GunChleoc has proposed merging lp:~gunchleoc/widelands/bug-1454371 into lp:widelands. Requested reviews: Widelands Developers (widelands-dev) Related bugs: Bug #1454371 in widelands: ""Out of resources" messages should be triggered by productivity" https://bugs.launchpad.net/widelands/+bug