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

Add the spring-rest overlay to allow building of the new rest7 #1646

Merged
merged 1 commit into from Feb 23, 2017

Conversation

abollini
Copy link
Member

temporary turn off mvn-enforcer dependency convergence
temporary turn off dspace-solr (to migrate to an external SOLR6)

see https://jira.duraspace.org/browse/DS-3482

temporary turn off mvn-enforcer dependency convergence
temporary turn off dspace-solr (to migrate to an external SOLR6)
@terrywbrady
Copy link
Contributor

@abollini , This code has helped. I am able to see the HAL browser.

I had to manually delete slf4j-log4j12.jar from [dspace-install]/webapps/spring-rest/WEB-INF/lib in order to get spring to launch.

Do you envision deploying spring-rest to /spring-rest or to /?

@abollini
Copy link
Member Author

I have tested it just deploying as spring-rest but it should work on any context including ROOT.
I haven't had issue with slf4j-log4j12.jar... what do you see in the logs? which version of tomcat do you use? (I have tomcat 8.5.4 on my local environment)

@tdonohue tdonohue added the code task Code cleanup task label Feb 21, 2017
Copy link
Member

@tdonohue tdonohue left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 Approved in today's DSpace 7 UI mtg

@tdonohue tdonohue merged commit 280dce2 into DSpace:rest7 Feb 23, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code task Code cleanup task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants