JuliaLang / MbedTLS.jl

Wrapper around mbedtls
Other
41 stars 50 forks source link

Support Julia 1.2 #204

Closed quinnj closed 5 years ago

quinnj commented 5 years ago

Should fix https://github.com/JuliaLang/julia/issues/32220

@staticfloat, does this look right to you?

codecov-io commented 5 years ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@b71d85d). Click here to learn what that means. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #204   +/-   ##
=========================================
  Coverage          ?   71.95%           
=========================================
  Files             ?       12           
  Lines             ?      542           
  Branches          ?        0           
=========================================
  Hits              ?      390           
  Misses            ?      152           
  Partials          ?        0

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update b71d85d...62012be. Read the comment docs.

KristofferC commented 5 years ago

Should we upper bound the supported julia version for previous MbedTLS versions in the registry?

quinnj commented 5 years ago

Yes, we'll need to do that, otherwise pre-1.2 julia versions will run into the same issue.