Merge lp:~widelands-dev/widelands/bug-1809838-blackroot-target into lp:widelands
Proposed by
Notabilis
Status: | Rejected | ||||
---|---|---|---|---|---|
Rejected by: | Notabilis | ||||
Proposed branch: | lp:~widelands-dev/widelands/bug-1809838-blackroot-target | ||||
Merge into: | lp:widelands | ||||
To merge this branch: | bzr merge lp:~widelands-dev/widelands/bug-1809838-blackroot-target | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Toni Förster | Disapprove | ||
Review via email: mp+361290@code.launchpad.net |
Commit message
Adding missing check whether blackroot flour is required.
Description of the change
Checking in the production program whether blackroot flour is required before grinding blackroot, so the economic target is respected.
It might be that this isn't checked on purpose, since different from corn there is no other use for blackroot. If the check shouldn't be in the production program, the economy target for blackroot flour should be removed instead.
To post a comment you must log in.
Continuous integration builds have changed state:
Travis build 4369. State: passed. Details: https:/ /travis- ci.org/ widelands/ widelands/ builds/ 472649794. /ci.appveyor. com/project/ widelands- dev/widelands/ build/_ widelands_ dev_widelands_ bug_1809838_ blackroot_ target- 4162.
Appveyor build 4162. State: success. Details: https:/