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

WIP: [R] Verify CRAN release 18.1.0.1 #45173

Open
wants to merge 5 commits into
base: maint-18.1.0-r
Choose a base branch
from

Conversation

jonkeane
Copy link
Member

@jonkeane jonkeane commented Jan 4, 2025

This is minor and a non-official release to get past a GCC 15 error.

Caution

Do not merge this PR.

This PR is being used to prepare for the CRAN submission for 18.1.0.1.

See packaging checklist: #45172.

…45097)

### Rationale for this change

Apache Thrift misses `#include <cstdint>` for `int64_t`: apache/thrift#3078
GCC 15 requires it.

### What changes are included in this PR?

Apply apache/thrift#3078 only with GCC 15.

### Are these changes tested?

Yes.

### Are there any user-facing changes?

No.
* GitHub Issue: #45096

Authored-by: Sutou Kouhei <[email protected]>
Signed-off-by: Jonathan Keane <[email protected]>
@jonkeane
Copy link
Member Author

jonkeane commented Jan 4, 2025

@github-actions crossbow submit --group r

Copy link

github-actions bot commented Jan 4, 2025

Revision: cc4e0f3

Submitted crossbow builds: ursacomputing/crossbow @ actions-0533803ddb

Task Status
r-binary-packages GitHub Actions
r-recheck-most GitHub Actions
test-r-arrow-backwards-compatibility GitHub Actions
test-r-clang-sanitizer GitHub Actions
test-r-depsource-bundled Azure
test-r-depsource-system GitHub Actions
test-r-dev-duckdb GitHub Actions
test-r-devdocs GitHub Actions
test-r-extra-packages GitHub Actions
test-r-gcc-11 GitHub Actions
test-r-gcc-12 GitHub Actions
test-r-install-local GitHub Actions
test-r-install-local-minsizerel GitHub Actions
test-r-linux-as-cran GitHub Actions
test-r-linux-rchk GitHub Actions
test-r-linux-valgrind GitHub Actions
test-r-macos-as-cran GitHub Actions
test-r-minimal-build Azure
test-r-offline-maximal GitHub Actions
test-r-offline-minimal Azure
test-r-rhub-debian-gcc-devel-lto-latest Azure
test-r-rhub-debian-gcc-release-custom-ccache Azure
test-r-rhub-ubuntu-release-latest Azure
test-r-rocker-r-ver-latest Azure
test-r-rstudio-r-base-4.1-opensuse155 Azure
test-r-rstudio-r-base-4.2-focal Azure
test-r-ubuntu-22.04 GitHub Actions
test-r-versions GitHub Actions
test-ubuntu-r-sanitizer GitHub Actions

### Rationale for this change

GitHub phased out R from their Ubuntu images in their 24.04 image and this job started failing because of it. We now need to install R ourselves if a job needs it and uses ubuntu-latest (>=24.04).

### What changes are included in this PR?

Change in GHA workflow config to always install R.

### Are these changes tested?

They're testing here with crossbow.

### Are there any user-facing changes?

No.

* GitHub Issue: #45069

Authored-by: Bryce Mecum <[email protected]>
Signed-off-by: Sutou Kouhei <[email protected]>
@jonkeane
Copy link
Member Author

jonkeane commented Jan 4, 2025

@github-actions crossbow submit r-binary-packages

Copy link

github-actions bot commented Jan 4, 2025

Revision: 133303e

Submitted crossbow builds: ursacomputing/crossbow @ actions-e6592b6365

Task Status
r-binary-packages GitHub Actions

@jonkeane
Copy link
Member Author

jonkeane commented Jan 4, 2025

@github-actions crossbow submit r-binary-packages

Copy link

github-actions bot commented Jan 4, 2025

Revision: 6255228

Submitted crossbow builds: ursacomputing/crossbow @ actions-5e529189a9

Task Status
r-binary-packages GitHub Actions

@jonkeane
Copy link
Member Author

jonkeane commented Jan 4, 2025

@github-actions crossbow submit r-binary-packages

Copy link

github-actions bot commented Jan 4, 2025

Revision: cdd0b79

Submitted crossbow builds: ursacomputing/crossbow @ actions-68bea26e94

Task Status
r-binary-packages GitHub Actions

@jonkeane jonkeane changed the title WIP: [R] Verify CRAN release 18.1.0.1 WIP: [R] Verify CRAN release 18.1.1 Jan 4, 2025
@jonkeane
Copy link
Member Author

jonkeane commented Jan 4, 2025

I'm trying to get a release ready to send to CRAN before our 14 January deadline. The only operative change that this branch has is cc4e0f3 (I also had to backport @amoeba 's CI fixes thanks for those!).

The binary job runs, but I don't actually know the process to get the binaries that are created in the CI job to the jfrog repository. @amoeba @assignUser any thoughts about what to do here? Is this a manual step (with key signing and all of that)?

I wanted to make the version 18.1.0.1 (that's why the branch is named such) since it's just this one patch to a dependency, but our version number + binary finder only looks at the first three version components

VERSION <- VERSION[1, 1:3]
Or maybe we could exploit this fact slightly and release 18.1.0.1, use the released 18.1.0 binaries everywhere, but include the patch in the source code (which is what CRAN will build + check with GCC15). That's a little bit funny because the binary folks are using is slightly different from the source version, but I'm pretty sure that would work. cc @nealrichardson and @thisisnic here too if there are 🚨 that that would be a bad plan.

@assignUser
Copy link
Member

release 18.1.0.1, use the released 18.1.0 binaries everywhere, but include the patch in the source code (which is what CRAN will build + check with GCC15).

Yes this. As there is no actual change to the binaries (we don't build them with GCC 15) we can just update the source tarball used for cran so we pass on an unreleased compiler version (... 🤦).

Thanks for getting this ready and a happy new year!

@amoeba
Copy link
Member

amoeba commented Jan 4, 2025

Thanks for working on this @jonkeane. Shipping the patch in the source tarball seems like the best approach to me too.

@nealrichardson
Copy link
Member

Agreed

@thisisnic
Copy link
Member

+1 to the simpler approach given the reason we're doing it. Thanks @jonkeane!

@jonkeane jonkeane changed the title WIP: [R] Verify CRAN release 18.1.1 WIP: [R] Verify CRAN release 18.1.0.1 Jan 5, 2025
@jonkeane jonkeane changed the base branch from main to maint-18.1.0-r January 5, 2025 14:55
@jonkeane
Copy link
Member Author

jonkeane commented Jan 5, 2025

Thanks, all! I've got everything prepared and will submit on the seventh when CRAN is back open.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants