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

Backports for Julia 1.5-RC1 (or beta2) #36098

Merged
merged 29 commits into from
Jun 22, 2020
Merged

Conversation

KristofferC
Copy link
Sponsor Member

@KristofferC KristofferC commented Jun 1, 2020

Backported PRs:

Non-merged PRs with backport label:

simeonschaub and others added 3 commits June 1, 2020 12:03
* fix #36031

* Apply suggestions from code review

Co-authored-by: Simon Byrne <[email protected]>
(cherry picked from commit 150311f)
…35232)

* LibGit2: amend GitError enum

Since upstream commit
libgit2/libgit2@e9cef7c
("http: introduce GIT_ERROR_HTTP") an invalid content type yields a
GIT_ERROR_HTTP instead of a GIT_ERROR_NET error. Update the enum to
include this new error so that the unit test for LibGit2 doesn't fail
with "invalid value for Enum Class: 34".

* LibGit2: add resolve_url to RemoteCallbacksStruct for LibGit2 0.99.0

Upstream commit
libgit2/libgit2@59647e1
("remote: add callback to resolve URLs before connecting") introduced a
new callback "resolve_url" in LibGit2 0.99.0. Even though it is not
currently used in Julia, it needs to be accounted for to get the correct
size for FetchOptionsStruct. An incorrectly aligned FetchOptionsStruct
leads to error messages like "invalid version 0 on git_proxy_options"
when trying to use the latest LibGit2 version.

* LibGit2: update error message checking for LibGit2 0.99.0

Upstream commit
libgit2/libgit2@b9c5b15
("http: use the new httpclient") changed the error message to include
additional quotes. Relax the unit test to allow these if present.

Co-authored-by: Milan Bouchet-Valat <[email protected]>
(cherry picked from commit 59a315c)
@KristofferC KristofferC added status:DO NOT MERGE Do not merge this PR! needs nanosoldier run This PR should have benchmarks run on it needs pkgeval Tests for all registered packages should be run with this change kind:release Release management and versioning. labels Jun 1, 2020
@KristofferC
Copy link
Sponsor Member Author

@nanosoldier runtests(["AMRVW", "AdvancedHMC", "ArchGDAL", "ArrayLayouts", "AugmentedGaussianProcesses", "AutoPreallocation", "AxisIndices", "BangBang", "BlockArrays", "BloomFilters", "Bridge", "CBindingGen", "CUDAatomics", "CancerSeqSim", "ChainRulesCore", "Chemfiles", "ClinicalTrialUtilities", "ColorVectorSpace", "CompScienceMeshes", "ContinuumArrays", "CoverageCore", "CoverageTools", "CurveProximityQueries", "DRIPs", "DataAssim", "DataFlow", "DimensionalData", "DirectGaussianSimulation", "DistributedArrays", "Diversity", "Documenter", "EfficientGlobalOptimization", "ExprRules", "ExtensibleUnions", "FEMBase", "FEMBasis", "FIGlet", "FastIOBuffers", "FillArrays", "FunctionalCollections", "FunctionalTables", "GeometryTypes", "Gtk", "GtkReactive", "GtkUtilities", "HomotopyContinuation", "ImageInpainting", "ImageView", "Immerse", "InfiniteArrays", "InspectDR", "IntervalArithmetic", "JLD2", "JSONSchema", "JuliaVariables", "KrigingModel", "LazyStack", "Lerche", "LibGEOS", "LinearMapsAA", "LocalRegistry", "MCHammer", "MIToS", "MLStyle", "Manifolds", "Maxvol", "MbedTLS", "Miletus", "MultiplesOfPi", "NTFk", "Nabla", "NarrativeTest", "NiLangCore", "OceanGrids", "OpenEphysLoader", "OptionalArgChecks", "PassiveTracerFlows", "Persa", "PlotUtils", "Plots", "Polymake", "PowerDynSolve", "ProfileView", "QuantumOpticsBase", "QuartetNetworkGoodnessFit", "QuasiArrays", "RecursiveArrayTools", "RevealedPreferences", "RigidBodyDynamics", "RoMEPlotting", "SimpleGF2", "StaticArrays", "StaticNumbers", "StratiGraphics", "StructuralInheritance", "TaylorIntegration", "TensorKitManifolds", "ThreePhasePowerModels", "TikzQTrees", "Turf", "TuringModels", "TwoFAST", "VarStructs", "VerTeX", "Winston", "Wrangling", "XLSXasJSON"], vs = ":release-1.4")

@nanosoldier
Copy link
Collaborator

Your package evaluation job has completed - possible new issues were detected. A full report can be found here. cc @maleadt

@rfourquet
Copy link
Member

@KristofferC I pushed a commit to update the NEWS entry for popat! to this branch. I hope this is OK, otherwise please tell me where I should have pushed it.

tkf and others added 15 commits June 4, 2020 22:51
fixes #36072

(cherry picked from commit 147d5b1)
* rename pop!(vector, idx, [default]) to popat!

* popat! : explain what `default` is

(cherry picked from commit 0164101)
* fix #36116, diff(::AbstractRange) returns an Array

(cherry picked from commit b49a0d5)
The error thrown by `mkdir` when the directory already exists was changed in #33422.

(cherry picked from commit 912a8ed)
(cherry picked from commit 7385876)
It could only handle a couple of pairs, e.g.
ImmutableDict(1=>1, 2=>2, 3=>3) would throw.

The fix is implemented by adding the
`ImmutableDict(t::ImmutableDict, pairs...)` constructor,
which generalizes `ImmutableDict(t::ImmutableDict, pair)`
(with some similarity to how `push!` accepts multiple items
to be pushed).

(cherry picked from commit 162cde1)
(cherry picked from commit a6354d9)
@KristofferC
Copy link
Sponsor Member Author

@nanosoldier runtests(["AMRVW", "AdvancedHMC", "ArchGDAL", "ArrayLayouts", "AugmentedGaussianProcesses", "AutoPreallocation", "AxisIndices", "BlockArrays", "BloomFilters", "Bridge", "CBindingGen", "CUDAatomics", "CancerSeqSim", "ChainRulesCore", "Chemfiles", "ClinicalTrialUtilities", "ColorVectorSpace", "CompScienceMeshes", "ContinuumArrays", "CoverageCore", "CoverageTools", "CurveProximityQueries", "DataAssim", "DataFlow", "DimensionalData", "DirectGaussianSimulation", "DistributedArrays", "Diversity", "Documenter", "EfficientGlobalOptimization", "ExprRules", "ExtensibleUnions", "FEMBase", "FEMBasis", "FIGlet", "FastIOBuffers", "FillArrays", "FunctionalCollections", "FunctionalTables", "GeometryTypes", "Gtk", "GtkReactive", "GtkUtilities", "HomotopyContinuation", "ImageInpainting", "ImageView", "Immerse", "InspectDR", "IntervalArithmetic", "JLD2", "JSONSchema", "JuliaVariables", "KrigingModel", "LazyStack", "LinearMapsAA", "LocalRegistry", "MCHammer", "MIToS", "MLStyle", "Manifolds", "Maxvol", "MbedTLS", "Miletus", "MultiplesOfPi", "NTFk", "Nabla", "NiLangCore", "OceanGrids", "OpenEphysLoader", "OptionalArgChecks", "PassiveTracerFlows", "Persa", "PlotUtils", "Plots", "Polymake", "PowerDynSolve", "ProfileView", "QuantumOpticsBase", "QuartetNetworkGoodnessFit", "QuasiArrays", "RecursiveArrayTools", "RevealedPreferences", "RigidBodyDynamics", "RoMEPlotting", "SimpleGF2", "StaticArrays", "StaticNumbers", "StratiGraphics", "StructuralInheritance", "TaylorIntegration", "TensorKitManifolds", "ThreePhasePowerModels", "TikzQTrees", "Turf", "TuringModels", "TwoFAST", "VarStructs", "VerTeX", "Winston", "XLSXasJSON"], vs = ":release-1.4")

@nanosoldier
Copy link
Collaborator

Your package evaluation job has completed - possible new issues were detected. A full report can be found here. cc @maleadt

@dkarrasch
Copy link
Member

@KristofferC Could you please consider including #35281 in here?

simeonschaub and others added 5 commits June 16, 2020 12:21
(cherry picked from commit 33659c7)
* deleteat! : check bounds for the first passed index

All other indices are bound-checked.
Currently, the behavior looks like:
```julia
julia> deleteat!([1:1000;], [-100])

signal (11): Segmentation fault
[...]

julia>  deleteat!(BigInt[0], [0])
free(): invalid next size (normal)

signal (6): Aborted
[...]

julia> deleteat!(BigInt[0], [-100])
ERROR: UndefRefError: access to undefined reference
[...]

julia> deleteat!([0], [0])
Int64[]

julia> deleteat!([0], [2])
1-element Array{Int64,1}:
 0

julia> deleteat!([0], [3])
ERROR: InexactError: check_top_bit(UInt64, -1)
[...]
```
With this commit, all these expressions throw a `BoundsError`.

* Update base/array.jl

Co-authored-by: Simeon Schaub <[email protected]>

Co-authored-by: Simeon Schaub <[email protected]>
(cherry picked from commit 6cdfcf9)
@KristofferC KristofferC removed status:DO NOT MERGE Do not merge this PR! needs nanosoldier run This PR should have benchmarks run on it needs pkgeval Tests for all registered packages should be run with this change labels Jun 16, 2020
staticfloat and others added 2 commits June 18, 2020 17:47
This allows constant-folding NamedTuple constructors, in turn
allowing constant prop through keyword arguments.

(cherry picked from commit 7124472)
@KristofferC KristofferC changed the title WIP: Backports for Julia 1.5-RC1 Backports for Julia 1.5-RC1 Jun 22, 2020
@KristofferC KristofferC changed the title Backports for Julia 1.5-RC1 Backports for Julia 1.5-RC1 (or beta2) Jun 22, 2020
@KristofferC KristofferC merged commit 3f4e3f6 into release-1.5 Jun 22, 2020
@KristofferC KristofferC deleted the backports-release-1.5 branch June 22, 2020 05:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind:release Release management and versioning.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet