Skip to content
This repository has been archived by the owner on Jan 20, 2025. It is now read-only.

Update dependencies. #59

Closed
wants to merge 1 commit into from
Closed

Update dependencies. #59

wants to merge 1 commit into from

Conversation

belucid
Copy link
Contributor

@belucid belucid commented Sep 5, 2015

Updates for a couple of out of date dependencies.

@belucid
Copy link
Contributor Author

belucid commented Sep 5, 2015

Worked fine for me locally but apparently not on Travis. Should (obviously) hold off on merging this until we find out what's up with the updated AWS S3 lib and Jackson

clojure.lang.Compiler$CompilerException: java.lang.NoSuchMethodError: com.fasterxml.jackson.core.JsonFactory.requiresPropertyOrdering()Z, compiling:(aws/sdk/s3.clj:1:1)

@xsc
Copy link
Owner

xsc commented Sep 9, 2015

@belucid Thanks for providing those - there are actually reasons for not updating, and I might as well document them here.

AWS SDK

I can find some related tickets but nothing that's currently open, but my suspicion is that some pre-compiled parts of Leiningen itself depend on the method mentioned in the Exception which disappears in later versions of Jackson. We could try excluding the Jackson dependencies - but this might cause other problems.

lein-midje

I keep it at 3.1.1 because of marick/lein-midje#47. It might not even affect lein-ancient's tests, I just don't want any future surprises there. :)

We can keep this MR as a reminder, though.

@belucid
Copy link
Contributor Author

belucid commented Sep 9, 2015

Cool. At least we have it documented now. That's half the battle.

@belucid belucid closed this Oct 17, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants