-
Notifications
You must be signed in to change notification settings - Fork 131
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
Enable war deployment model (#197.1) #3839
Conversation
e649189
to
c7ec2a9
Compare
c7ec2a9
to
94a9351
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@WWelling it works for me. Please, double check two my comments.
installer/webapp/src/main/webResources/WEB-INF/classes/log4j.properties
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The runtime properties has some commented out code still using /usr/local
.
See:
home/src/main/resources/config/default.runtime.properties:176:#OpenSocial.tokenKeyFile = /usr/local/vivo/data/shindig/openssl/securitytokenkey.txt
home/src/main/resources/config/default.runtime.properties:361:# harvester.location = /usr/local/vivo/harvester/
I think these should be updated to /opt/vivo
paths.
superseded by #3844 |
VIVO GitHub issue: #3034
See details on previous reverted PR: #197
What's new?
Vitro.defaultNamespace
androotUser.emailAddress
from runtime properties to context.xml.How should this be tested?
Clone Vitro and VIVO and checkout feature branches
VIVO-3034-psuedo-rebase
or
Build Vitro
Build and run VIVO with Docker
If not using Docker, copy the war file into Tomcat.
Check http://localhost:8080/
Interested parties
@VIVO-project/vivo-committers @chenejac