Skip to content
This repository has been archived by the owner on Aug 12, 2022. It is now read-only.

Set a maximum version of StatsFuns for Distributions #5613

Merged
merged 6 commits into from
Jul 18, 2016

Conversation

ararslan
Copy link
Member

StatsFuns 0.3.0 introduces breaking changes to all current Distributions versions.

@tkelman
Copy link
Contributor

tkelman commented Jul 14, 2016

I think we may need to add an artificial StatsFuns 0.0.0 0.3.0 dependency to even earlier Distributions.jl versions that support julia 0.3 or newer, otherwise the conflict may result in downgrading Distributions.jl to something from before it depended on StatsFuns.

@ararslan
Copy link
Member Author

Ah right, should have remembered that from last time. Will do.

@tkelman
Copy link
Contributor

tkelman commented Jul 14, 2016

I'm a little hesitant on what to do about any versions of Distributions that don't have a minimum Julia version, or have one that's pre-0.3.0. We should probably test this and see whether Distributions could ever get downgraded to anything REALLY old. Hopefully not, but we probably want to wait until after a Distributions tag that does support the new StatsFuns version before merging this.

@ararslan
Copy link
Member Author

A Distributions tag is in the works, ref JuliaStats/Distributions.jl#512.

@tkelman
Copy link
Contributor

tkelman commented Jul 15, 2016

If that's going to take a while, we should maybe raise the julia minimum for statsfuns 0.3.0 temporarily to 0.5-.

For the very old Distributions versions with no julia dependency or before 0.3.0 so adding an extra statsfuns dependency would not work, we should add a julia 0.3 upper bound.

@ararslan
Copy link
Member Author

I'm hoping it won't be long. If someone says it's okay to merge my current Distributions PR then I'll go ahead and do that, tag a version, and update this. Hoping to get this done ASAP today.

@ararslan
Copy link
Member Author

Ref #5624.

@andreasnoack
Copy link
Member

What is required to resolve the StatsFuns situation?

@tkelman
Copy link
Contributor

tkelman commented Jul 17, 2016

we could either do fixes that say they're 0.5-only at first until distributions and this change are ready, or we could do the new tags in the same pr that loosens the statsfuns julia requirement back to allowing 0.4

@ararslan
Copy link
Member Author

ararslan commented Jul 17, 2016

We might have to set upper bounds on StatsFuns for StatsBase too.

@ararslan
Copy link
Member Author

Distributions versions 0.2.2-0.3.0 had no Julia version requirement but 0.2.1 requires Julia 0.2-, so I set 0.3 as an upper bound on Julia for those versions.

Distributions versions with no dependency on StatsFuns got a line that sets bounds at 0.0.0, 0.3.0. Existing StatsFuns dependencies got 0.3.0 as an upper bound if none existed, or were left as-is otherwise.

Is this what you had in mind, @tkelman?

@ararslan
Copy link
Member Author

I won't have much time to help 🔥💧🚒 for the next several hours but I'll try to check in occasionally if I can.

@ararslan
Copy link
Member Author

In the meantime I've added you two (@tkelman and @andreasnoack) as collaborators on my METADATA fork, you can make any changes to this PR if need be.

@tkelman
Copy link
Contributor

tkelman commented Jul 17, 2016

ok I'll adjust this when I get in front of a desk then

Anything that supported pre-0.3.0 Julia can't use StatsFuns,
instead add a Julia 0.3 upper bound to those tags
@ararslan
Copy link
Member Author

Thanks @tkelman!

@tkelman tkelman merged commit e8cffaa into JuliaLang:metadata-v2 Jul 18, 2016
@ararslan ararslan deleted the aa/distributions_on_fire branch July 18, 2016 04:15
rofinn pushed a commit to rofinn/METADATA.jl that referenced this pull request Jul 23, 2016
* Set a maximum version of StatsFuns for Distributions

* Set Julia and StatsFuns requirements where none existed

* Don't add StatsFuns dependency to early Distributions tags

Anything that supported pre-0.3.0 Julia can't use StatsFuns,
instead add a Julia 0.3 upper bound to those tags

* missed a few

* Allow 0.3 prereleases for Mamba 0.3.7

it's being picky with an upper bound on Distributions

* and only 0.3 prereleases
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

3 participants