Skip to content
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

Upgrade deegree 3.6.0-pre2 with using WAR file in system tomcat #407

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

sanak
Copy link
Contributor

@sanak sanak commented Nov 5, 2024

Trac ticket: https://trac.osgeo.org/osgeolive/ticket/2494

Changes proposed in this pull request:

When using Java 11 + Tomcat 9 environment, the following changes should be enough.

--- a/bin/install_deegree.sh
+++ b/bin/install_deegree.sh
@@ -59,9 +59,9 @@ if [ -z "$USER_NAME" ] ; then
 fi
 USER_HOME="/home/$USER_NAME"
 TOMCAT_USER_NAME="tomcat"
-TOMCAT_SCRIPT_NAME="tomcat10"
+TOMCAT_SCRIPT_NAME="tomcat9"
 DEEGREE_WEB_APP_NAME="deegree"
-DEEGREE_VERSION="3.6.0-pre2"
+DEEGREE_VERSION="3.5.8"

I haven't tested this from ISO build, but it should work.
I will try to build ISO and test it, later.

@sanak
Copy link
Contributor Author

sanak commented Nov 5, 2024

I will try to build ISO and test it, later.

Well, I confirmed that it works without error.
But, I am not sure what is correct in deegree...
I will try to check quick start document with translating Japanese, later.

@tfr42
Copy link
Contributor

tfr42 commented Nov 5, 2024

@sanak I can confirm, that deegree 3.5.x works with Tomcat 9.x/JDK11 and the early access release 3.6.0-pre2 works with Tomcat 10.1/JDK 17. So the PR looks good and the changes to downgrade to 3.5 too. Please get in contact with us in case you need further information.

@tfr42
Copy link
Contributor

tfr42 commented Nov 5, 2024

@sanak And we will try to get the documentation online at https://download.deegree.org/documentation/ for 3.6.0 asap...

@sanak
Copy link
Contributor Author

sanak commented Nov 6, 2024

@tfr42 Thanks for your reply!

Regarding the document, I think at least changing http://localhost:8033 to http://localhost:8080/degree will be necessary, but further review and creation of PR on OSGeoLive-doc side is really helpful. 🙏🙇

I uploaded built ISO image (from 1st commit) to the following Google Drive folder, so if necessary, please try to use it. 🙇

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants