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

Bump minimum LibExpat for 0.6 #100

Closed
wants to merge 1 commit into from
Closed

Bump minimum LibExpat for 0.6 #100

wants to merge 1 commit into from

Conversation

musm
Copy link
Contributor

@musm musm commented Feb 15, 2017

No description provided.

@musm
Copy link
Contributor Author

musm commented Feb 15, 2017

Should fix #88

@yuyichao
Copy link

I don't think we do this

@tkelman
Copy link
Contributor

tkelman commented Feb 15, 2017

this version is not required on release

@musm
Copy link
Contributor Author

musm commented Feb 15, 2017

Minimums are only for the latest release version of julia?

@tkelman
Copy link
Contributor

tkelman commented Feb 15, 2017

they're applied across all julia versions the package supports

@musm
Copy link
Contributor Author

musm commented Feb 15, 2017

If I understand correctly, minimums in the REQUIRE doesn't necessarily mean minimum working version on all_ version of Julia, just the minimum version required on the lowest version supported with the package?

@tkelman
Copy link
Contributor

tkelman commented Feb 15, 2017

yes. what was in 0.2.4 that you're wanting to require?

#88 was probably already fixed by the libexpat tags

@musm
Copy link
Contributor Author

musm commented Feb 15, 2017

yes. what was in 0.2.4 that you're wanting to require?

Specifically this PR : JuliaIO/LibExpat.jl#56
which fixes the LibExpat bug that was causing issues in #88

@musm musm closed this Feb 15, 2017
@tkelman
Copy link
Contributor

tkelman commented Feb 15, 2017

and that was not required on julia 0.5, only 0.6

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.

3 participants