Skip to content

GH-39656: [Release] Update platform tags for macOS wheels to macosx_10_15 - #39657

Merged
kou merged 1 commit into
apache:mainfrom
raulcd:GH-39656
Jan 18, 2024
Merged

GH-39656: [Release] Update platform tags for macOS wheels to macosx_10_15#39657
kou merged 1 commit into
apache:mainfrom
raulcd:GH-39656

Conversation

@raulcd

@raulcdraulcd commented Jan 17, 2024

Copy link
Copy Markdown
Member

Rationale for this change

Currently the binary verification for releases fails due to wrong macOS platform version.

What changes are included in this PR?

Update to the current generated platform tag.

Are these changes tested?

No, but I've validated this is the corrected generated platform tag for the wheels on the Release Candidate: https://apache.jfrog.io/ui/native/arrow/python-rc/15.0.0-rc1/

Are there any user-facing changes?

Not because of this change but there was a minimum version of macOS as part of the PR that caused this issue.

@github-actions

Copy link
Copy Markdown

⚠️ GitHub issue #39656has been automatically assigned in GitHub to PR creator.

@github-actionsgithub-actionsBot added the awaiting committer review Awaiting committer review label Jan 17, 2024
kou
kou approved these changes Jan 18, 2024

@koukou left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

+1

We may want to use dev/release/veriry-release-candidate.sh for our wheel related nightly CI jobs.

FYI: We use dev/release/verify-{apt,yum}.sh for our .deb/.rpm related nightly CI jobs.

@conbench-apache-arrow

Copy link
Copy Markdown

After merging your PR, Conbench analyzed the 6 benchmarking runs that have been run so far on merge-commit 1c49b9f.

There were 2 benchmark results with an error:

There were no benchmark performance regressions. 🎉

The full Conbench report has more details.

dgreiss pushed a commit to dgreiss/arrow that referenced this pull request Feb 19, 2024
…cosx_10_15 (apache#39657)
### Rationale for this change
Currently the binary verification for releases fails due to wrong macOS platform version.
### What changes are included in this PR?
Update to the current generated platform tag.
### Are these changes tested?
No, but I've validated this is the corrected generated platform tag for the wheels on the Release Candidate: https://apache.jfrog.io/ui/native/arrow/python-rc/15.0.0-rc1/
### Are there any user-facing changes?
Not because of this change but there was a minimum version of macOS as part of the PR that caused this issue.
* Closes: apache#39656
Authored-by: Raúl Cumplido <raulcumplido@gmail.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
raulcd added a commit that referenced this pull request Feb 20, 2024
…0_15 (#39657)
### Rationale for this change
Currently the binary verification for releases fails due to wrong macOS platform version.
### What changes are included in this PR?
Update to the current generated platform tag.
### Are these changes tested?
No, but I've validated this is the corrected generated platform tag for the wheels on the Release Candidate: https://apache.jfrog.io/ui/native/arrow/python-rc/15.0.0-rc1/
### Are there any user-facing changes?
Not because of this change but there was a minimum version of macOS as part of the PR that caused this issue.
* Closes: #39656
Authored-by: Raúl Cumplido <raulcumplido@gmail.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
Sign up for freeto 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.

[Release] platform_tags for macOS were not updated on verification script after increasing minimum version

2 participants

@raulcd@kou