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

Remove minter bean from integration test configuration #97

Closed
wants to merge 1 commit into from

Conversation

awoods
Copy link

@awoods awoods commented Nov 1, 2015

@acoburn
Copy link
Contributor

acoburn commented Nov 1, 2015

#98 should take the place of this PR

@awoods
Copy link
Author

awoods commented Nov 5, 2015

This PR should be merged once the fcrepo-camel component upgrades to a fcrepo4 version beyond 4.4.0.
Reference from http://irclogs.fcrepo.org/2015-11-05.html :

<awoods> acoburn: You have a comment that the fcrepo-camel PR should not be included... at least not before: https://github.com/fcrepo4-exts/fcrepo-camel/pull/98
<awoods> acoburn: do you want to have the following included now? https://github.com/fcrepo4-exts/fcrepo-camel/pull/97
<acoburn> awoods: no, I think not
<acoburn> awoods: definitely not
<awoods> acoburn: ok, you want the minter?
<awoods> acoburn: or you do not like the scanning?
<acoburn> awoods: the thing is we've decoupled the versioning
<acoburn> awoods: I like the scanning!
<acoburn> awoods: but we're using fcrepo version 4.4.0
<acoburn> awoods: and that has the minter built in, so we need the spring config to contain the minter stuff
<awoods> acoburn: ok, that makes sense
<acoburn> awoods: once the minter project is pulled out of fcrepo and there's a new release, then we can change this on fcrepo-camel
<acoburn> awoods: that is, then we'll *have* to change this
<awoods> acoburn: exactly
<awoods> acoburn: I will keep the PR around for later

@acoburn
Copy link
Contributor

acoburn commented Apr 20, 2016

merged via bc35a94

@acoburn acoburn closed this Apr 20, 2016
@awoods awoods deleted the fcrepo-1696 branch April 20, 2016 19:19
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