Skip to content

Conversation

Skunnyk
Copy link
Contributor

@Skunnyk Skunnyk commented Nov 17, 2017

This PR update spotlight dependencies, I hope without breaking anything.
This is a big cleanup in pom.xml files, some dependencies were unused since a long time, other were outdated for ages etc.
Sometimes I removed bundle dep and replace them with needed packages
I also removed all the log4j things now that spotlight use slf4j.

The cleaning is based on mvn versions:display-dependency-update and dependency:analyze

If you can, please test ! (I haven't noticed any problems yet)

- Update to latest minor versions
- Still a WIP
- Based on versions:display-dependency-updates, then try to update
deps without breaking anything
- Also using mvn dependency:analyze to remove unused deps (there are a
lot !)
- For now the rest server still works, haven't tested all cases.
- some log4j were still used and worked due to dependencie chain ...
- Replace it with SpotlightLog
- Still a WIP of course
- We don't need net.liftweb.lift-json_2.10 and net.sf.json-lib anymore
- Their usage has been removed during the split of Lucene and Model (for spotlight 1.0) /  Apache 2.0 Licensing
- UIMA and Mahout are not used anymore
- remove jersey-bundle and use correct dependencies
- also rio/stax/trove
- Revert the jersey-bundle removal, spotlight is broken by declaring
individual deps

This reverts commit ce05d80.
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.

1 participant