New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Java 11 build not available due to gradle version being so old #65
Comments
That ticket was the first communication towards the community. The details in that ticket are all we know, but we can set up a meeting to discuss it better if you want.
graph TD;
GeoServer-->MapfishPrintV2;
GeoServer-->GeoTools
MapfishPrintV2-->GeoTools;
The mapfish-print version used directly by MapStore is likely not affected, as MapStore does not have a GeoTools dependency of its own (as far as I know, at least). |
Please see mapfish/mapfish-print-v2#12 branch which merges all the work here into something that can be updated for geotools refactor. I worked very hard @aaime with and @sebr72(camptocamp) at the code sprint to try and come up with a solution that would satisfy GeoNode community and be support out-of-the box by gs-printing module. |
More info also here for internal coordination ad testing purposes. |
@tdipisa let me know when tests on your side are done, then I will setup the tasks to upgrade GeoNode to the new mapfish print libs. |
Thanks, we want the GeoNode community to be happy out-of-the-box before making this GeoServer release. |
I cannot see that issue, perhaps it is a private repository. |
That's only for internal coordination for testing purposes @jodygarnett |
@offtherailz the new artifact is now ready on GS MVN repo to be used for MS dev. @ElenaGallo will be involved for functional tests there. |
That was my subtle reminder that you are on a public communication channel :) |
Note that use of https://maven.geo-solutions.it/org/mapfish/print/print-lib/2.3-SNAPSHOT/ is not going to work well for anyone as there is a conflict between this and repo.osgeo.org for the same artifact coordinate. If you wish to have a fork use a different classifier or group id etc…. The correct org.mapfish.print:print-lib:2.2-SNAPSHOT is already available from build.geoserver.org nightly builds packaged in geosever printing extension and ready to use. |
@jodygarnett @aaime @giohappy The test has been performed using this map with the MapStore printing tool. All layout combinations provided by that tool have been tested as well as other available options such as:
Test resultComparing prints between DEV and QA using the same options, the outputs are the same. There are therefore no regressions from tested use cases. Printing outputs derived from the tests are available here. The only issue found during tests is captured by geosolutions-it/MapStore2#9459 and need to be further investigated. It doesn't seems anyway connected with printing capabilities. The new print-lib version will be anyway available for the next major version of MapStore v2024.01.00. In the meantime it will remain in MS master branch for further tests during next months. |
yes, @giohappy |
FYI A task for testing the new plugin in GeoNode has been created GeoNode/geoserver-geonode-ext#175 |
Cross-referencing mapfish/mapfish-print-v2#11
GeoTools now requires Java 11, and as part OpenGIS Harmonization we need to change some package names in GeoTools requiring a new release of mapfish-print-v2 for downstream projects GeoServer and GeoNode.
@tdipisa are you aware of this? Is it something that has been already discussed?
@aaime might add more details.
The text was updated successfully, but these errors were encountered: