Overview
+- Update to version 0.11.6: + * Various new features and bugfixes + * see http://www.freecol.org/
Imho indenting this far in just looks bad :) I'll fix this up though
-sed -i "s/1.5/1.6/g" build.xml +sed -i "s/1.8/1.10/g" build.xml
I doubt this is correct; this was to move the JAVA version level from 1.5 to 1.6; if upstream now sets 1.8 by default, that is good enough and there should be no need to change that.
especially in line with
-Requires: jre >= 1.6 +Requires: jre >= 1.8
you still claim 1.8 as being sufficient.
I thought the one is build-require the other runtime. Hasnt TW 1.10 now as standard, if I pull in this?: BuildRequires: java-devel
Yes, we use J10 as default, but the version set in build.xml is to define the minimal binary compatibility version required (source code version and target version); so leaving at 1.8 is good for that.
thanks for the supersede
BuildRequires: java-devel >= 1.8 would match the requirement of upstream