Skip to content

Eliminate haskey piracy#47

Merged
andyferris merged 1 commit intoJuliaData:mainfrom
timholy:teh/end_piracy
Jan 18, 2022
Merged

Eliminate haskey piracy#47
andyferris merged 1 commit intoJuliaData:mainfrom
timholy:teh/end_piracy

Conversation

@timholy
Copy link
Copy Markdown
Contributor

@timholy timholy commented Jan 12, 2022

On nightly, we get this warning:

WARNING: Method definition haskey(Any, Any) in module Base at abstractdict.jl:17 overwritten in module SplitApplyCombine at /home/tim/.julia/packages/SplitApplyCombine/ulZAx/src/SplitApplyCombine.jl:46.
** incremental compilation may be fatally broken for this module **

Make the definition conditional on whether it's already present in Base.

This method was added in JuliaLang/julia#42679. Investigated due to this package triggering PkgEval errors in JuliaLang/julia#43759.

On nightly, we get this warning:

WARNING: Method definition haskey(Any, Any) in module Base at abstractdict.jl:17 overwritten in module SplitApplyCombine at /home/tim/.julia/packages/SplitApplyCombine/ulZAx/src/SplitApplyCombine.jl:46.
  ** incremental compilation may be fatally broken for this module **

Make the definition conditional on whether it's already present in Base.
@andyferris
Copy link
Copy Markdown
Member

Thanks @timholy - as always!

@andyferris andyferris merged commit 6ff8b89 into JuliaData:main Jan 18, 2022
@timholy timholy deleted the teh/end_piracy branch January 18, 2022 15:30
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.

2 participants