Review: Approve I looked through this and I wondered why you did not also delete the patch file. Unused files in a repository are usually a bad idea, since they are forgotten over time. If somebody needs it back in the future, they can reverse-cherrypick your commit.
Otherwise lgtm. -- https://code.launchpad.net/~hjd/widelands/disabled-s390-patch/+merge/217293 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/debian. _______________________________________________ Mailing list: https://launchpad.net/~widelands-dev Post to : [email protected] Unsubscribe : https://launchpad.net/~widelands-dev More help : https://help.launchpad.net/ListHelp
