Skip to content

GH-50581: [R] Use bundled simdjson by default#50587

Open
Reranko05 wants to merge 1 commit into
apache:mainfrom
Reranko05:R-simdjson-bundle
Open

GH-50581: [R] Use bundled simdjson by default#50587
Reranko05 wants to merge 1 commit into
apache:mainfrom
Reranko05:R-simdjson-bundle

Conversation

@Reranko05

@Reranko05 Reranko05 commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Rationale

The R build may pick up a system-installed simdjson when simdjson_SOURCE is not explicitly set. This causes build failures in environments where the system version is incompatible or unsuitable for the build.

What changes are included?

This change defaults simdjson_SOURCE to BUNDLED when it is not already set, ensuring the R build uses the bundled simdjson by default while still allowing users to override it explicitly.

This addresses the macOS CRAN release and ubuntu-r-only-r (AUTO deps) failures reported in GH-50581.

Related: #50581

@Reranko05
Reranko05 requested a review from assignUser as a code owner July 22, 2026 01:26
Copilot AI review requested due to automatic review settings July 22, 2026 01:26

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@github-actions

Copy link
Copy Markdown

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

@kou kou added the CI: Extra: R Run extra R CI label Jul 22, 2026
@kou

kou commented Jul 22, 2026

Copy link
Copy Markdown
Member

@github-actions crossbow submit -g r

@github-actions

Copy link
Copy Markdown

Revision: 9864597

Submitted crossbow builds: ursacomputing/crossbow @ actions-357652d8fe

Task Status
r-binary-packages GitHub Actions
r-recheck-most GitHub Actions
test-r-alpine-linux-cran GitHub Actions
test-r-arrow-backwards-compatibility GitHub Actions
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-fedora-clang 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-sanitizers GitHub Actions
test-r-linux-valgrind GitHub Actions
test-r-m1-san GitHub Actions
test-r-macos-as-cran GitHub Actions
test-r-offline-maximal GitHub Actions
test-r-ubuntu-22.04 GitHub Actions
test-r-versions GitHub Actions
test-r-wasm GitHub Actions

@Reranko05

Copy link
Copy Markdown
Contributor Author

@kou It looks like the remaining failures are related to the ongoing simdjson version discussion. If we go with requiring simdjson 4.0+ and falling back to the bundled version when the system package is too old, I think that should address this issue.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants