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

FCREPO-1871. Update fcrepo-camel-toolbox client code imports. #69

Closed
wants to merge 3 commits into from

Conversation

mohideen
Copy link
Contributor

@@ -41,7 +41,8 @@
<spring.version>4.1.5.RELEASE</spring.version>
<fcrepo.version>4.4.0</fcrepo.version>
<fcrepo-audit.version>4.4.0</fcrepo-audit.version>
<fcrepo-camel.version>4.4.0</fcrepo-camel.version>
<fcrepo-camel.version>4.4.1-SNAPSHOT</fcrepo-camel.version>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Note: The camel dependency version is changed to a SNAPSHOT version here. If this is not desired, we can wait until fcrepo-camel 4.4.1 is released. And, update and merge the PR after the release.

Copy link
Contributor

Choose a reason for hiding this comment

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

I would highly recommend keeping this in line with non-SNAPSHOT releases -- it means we can version the projects independently, which is highly desirable. If you would like to have a new fcrepo-camel release (with the new fcrepo-java-client), I can do that early next week.

@acoburn
Copy link
Contributor

acoburn commented Dec 26, 2015

This should wait until there is a new release of fcrepo-camel (containing the updated fcrepo-java-client)

Updated snapshot dependency version to release version.
@mohideen
Copy link
Contributor Author

I have updated dependency scope to test for modules that uses the fcrepo-java-client for test only.

@acoburn
Copy link
Contributor

acoburn commented Mar 30, 2016

merged via e13924e

@acoburn acoburn closed this Mar 30, 2016
@mohideen mohideen deleted the master branch September 7, 2016 14:37
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.

None yet

2 participants