Skip to content

Fix bind_artifact! when platform has compare_strategy#3624

Merged
IanButterworth merged 7 commits intoJuliaLang:masterfrom
glennmoy:gm/fix_bind_artifact
Jul 2, 2025
Merged

Fix bind_artifact! when platform has compare_strategy#3624
IanButterworth merged 7 commits intoJuliaLang:masterfrom
glennmoy:gm/fix_bind_artifact

Conversation

@glennmoy
Copy link
Copy Markdown
Contributor

@glennmoy glennmoy commented Sep 14, 2023

Fixes #3623

Note: the instructions are missing thePkg.build step needed to make the tests work locally (hence the repeated pushes). Easy to miss, I will open a follow up PR to add it #3625.

Comment thread src/Artifacts.jl Outdated
Comment thread test/artifacts.jl Outdated
Copy link
Copy Markdown
Member

@IanButterworth IanButterworth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems reasonable to me. Though I'll ping @staticfloat for visibility.

@IanButterworth IanButterworth merged commit 799f7de into JuliaLang:master Jul 2, 2025
9 checks passed
@github-project-automation github-project-automation Bot moved this from In review to Done in Pkg.jl Jul 2, 2025
@IanButterworth IanButterworth removed the merge me Merge when tests pass label Jul 2, 2025
KristofferC pushed a commit that referenced this pull request Jul 4, 2025
Co-authored-by: Ian Butterworth <i.r.butterworth@gmail.com>
(cherry picked from commit 799f7de)
@KristofferC KristofferC mentioned this pull request Jul 4, 2025
22 tasks
KristofferC pushed a commit that referenced this pull request Feb 20, 2026
Co-authored-by: Ian Butterworth <i.r.butterworth@gmail.com>
(cherry picked from commit 799f7de)
@KristofferC KristofferC mentioned this pull request Feb 20, 2026
18 tasks
KristofferC pushed a commit that referenced this pull request Feb 23, 2026
Co-authored-by: Ian Butterworth <i.r.butterworth@gmail.com>
(cherry picked from commit 799f7de)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Use platforms_match instead of isequal in bind_artifacts!

4 participants