Skip to content

GH-51682: [CI][C++] Add liburiparser-dev to system dependency image - #51683

Merged
raulcd merged 3 commits into
apache:mainfrom
hiroyuki-sato:topic/example-cpp-minimal-ci-test
Oct 2, 2026
Merged

raulcd merged 3 commits into
apache:mainfrom
hiroyuki-sato:topic/example-cpp-minimal-ci-test

Conversation

@hiroyuki-sato

@hiroyuki-sato hiroyuki-sato commented Oct 2, 2026 •

Copy link
Copy Markdown
Collaborator

Rationale for this change

Changes around UriParser, including #51007, caused example-cpp-minimal-build-static-system-dependency to fail to build.

What changes are included in this PR?

This change adds liburiparser-dev to the system dependency image.

Are these changes tested?

Yes.

Are there any user-facing changes?

No.

Was AI used for this PR?

In accordance to the AI generation guidelines, please disclose below whether and how AI was used in this PR.

PR code and description written by:

  • Human
  • AI

Reviewed before submission by:

  • Human
  • AI
  • Not reviewed

@github-actions

github-actions Bot commented Oct 2, 2026

Copy link
Copy Markdown

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

@hiroyuki-sato

Copy link
Copy Markdown
Collaborator Author

@github-actions crossbow submit example-cpp-minimal-build-static-system-dependency

@github-actions github-actions Bot added the awaiting review Awaiting review label Oct 2, 2026
@github-actions

github-actions Bot commented Oct 2, 2026

Copy link
Copy Markdown

Revision: e87bf41

Submitted crossbow builds: ursacomputing/crossbow @ actions-0ddc819ea7

Task Status
example-cpp-minimal-build-static-system-dependency GitHub Actions

hiroyuki-sato and others added 2 commits October 2, 2026 17:05
…tem-dependency build failure"

This reverts commit e87bf41.
Co-Authored-By: Raúl Cumplido <raulcumplido@gmail.com>
@hiroyuki-sato

Copy link
Copy Markdown
Collaborator Author

@github-actions crossbow submit example-cpp-minimal-build-static-system-dependency

@github-actions

github-actions Bot commented Oct 2, 2026

Copy link
Copy Markdown

Revision: 12e2758

Submitted crossbow builds: ursacomputing/crossbow @ actions-7e4b29c6c6

Task Status
example-cpp-minimal-build-static-system-dependency GitHub Actions

@hiroyuki-sato hiroyuki-sato changed the title GH-51682: [CI] Fix example-cpp-minimal-build-static-system-dependency build failure GH-51682: [CI] Add liburiparser-dev to system dependency image Oct 2, 2026
@hiroyuki-sato hiroyuki-sato changed the title GH-51682: [CI] Add liburiparser-dev to system dependency image GH-51682: [CI][C++] Add liburiparser-dev to system dependency image Oct 2, 2026
@hiroyuki-sato
hiroyuki-sato marked this pull request as ready for review October 2, 2026 08:31
@github-actions github-actions Bot added awaiting merge Awaiting merge and removed awaiting review Awaiting review labels Oct 2, 2026
@raulcd
raulcd merged commit 142829f into apache:main Oct 2, 2026
59 of 60 checks passed
@raulcd raulcd removed the awaiting merge Awaiting merge label Oct 2, 2026
raulcd pushed a commit that referenced this pull request Oct 2, 2026
…51683)

### Rationale for this change

Changes around UriParser, including #51007, caused example-cpp-minimal-build-static-system-dependency to fail to build.

### What changes are included in this PR?

This change adds liburiparser-dev to the system dependency image.

### Are these changes tested?

Yes.

### Are there any user-facing changes?

No.

### Was AI used for this PR?

In accordance to the [AI generation guidelines](https://arrow.apache.org/docs/dev/developers/overview.html#ai-generated-code), please disclose below whether and how AI was used in this PR.

**PR code and description written by:**

- [x] Human
- [ ] AI

**Reviewed before submission by:**

- [x] Human
- [ ] AI
- [ ] Not reviewed

* GitHub Issue: #51682

Authored-by: Hiroyuki Sato <hiroysato@gmail.com>
Signed-off-by: Raúl Cumplido <raulcumplido@gmail.com>
@hiroyuki-sato
hiroyuki-sato deleted the topic/example-cpp-minimal-ci-test branch October 2, 2026 08:57
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.

2 participants