Skip to content
This repository has been archived by the owner on Apr 8, 2021. It is now read-only.

more robust way to find the ivy resolution file. #30

Closed
wants to merge 1 commit into from

Conversation

ebowman
Copy link
Contributor

@ebowman ebowman commented Apr 5, 2013

We use a range of versions of sbt (including our own forks), and have
struggled with this piece of code. This tries to be a bit more robust
... instead of assuming behavior about versions, it just looks
everywhere the file might be, and of the files that actually exist,
takes the most recent version. This is sort flawed in theory, useful in
practice.

We use a range of versions of sbt (including our own forks), and have
struggled with this piece of code. This tries to be a bit more robust
... instead of assuming behavior about versions, it just looks
everywhere the file might be, and of the files that actually exist,
takes the most recent version. This is sort flawed in theory, useful in
practice.
@buildhive
Copy link

Johannes Rudolph » sbt-dependency-graph #5 SUCCESS
This pull request looks good
(what's this?)

@jrudolph
Copy link
Member

jrudolph commented Apr 8, 2013

Hi Eric, this certainly looks useful. Do you know in which sbt version which one of the options is used? If we'd know we could at least add a comment for each of the alternatives about our knowledge about when which option is the right one.

@ebowman
Copy link
Contributor Author

ebowman commented Apr 8, 2013

I agree, and I got a bit confused about which version has which behavior, TBH (we have our own hacked 0.12.1 as well as 0.12.2 and 3) I'll try to figure out and add that.

In the meantime, the extra case I added seems to be needed for 0.12.3.

Thanks,
Eric

On Apr 8, 2013, at 1:11 PM, Johannes Rudolph notifications@github.com wrote:

Hi Eric, this certainly looks useful. Do you know in which sbt version which one of the options is used? If we'd know we could at least add a comment for each of the alternatives about our knowledge about when which option is the right one.


Reply to this email directly or view it on GitHub.

@seanparsons
Copy link

Is there anything I can do to help with getting this released? It's currently biting me quite hard.

@jrudolph jrudolph closed this in dfe76a2 Apr 28, 2013
@jrudolph
Copy link
Member

Fixed in version 0.7.3.

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.

None yet

4 participants