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.

Sign up for GitHub

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

Jump to bottom

[CI][Release][GLib] Verification test can't detect arrow-dataset #34977

Closed
kou opened this issue Apr 7, 2023 · 0 comments · Fixed by #34978
Closed

[CI][Release][GLib] Verification test can't detect arrow-dataset #34977

kou opened this issue Apr 7, 2023 · 0 comments · Fixed by #34978
Assignees
@kou
Labels
Component: Continuous Integration Component: GLib Type: bug
Milestone
12.0.0

Comments

@kou
Copy link
Member

kou commented Apr 7, 2023

Describe the bug, including details regarding any error messages, version, and platform.

https://github.com/ursacomputing/crossbow/actions/runs/4638281145/jobs/8207935726#step:6:9824

Run-time dependency arrow-dataset found: NO (tried pkgconfig and cmake)

Component(s)

Continuous Integration, GLib
@kou kou added the Type: bug label Apr 7, 2023
kou added a commit to kou/arrow that referenced this issue Apr 7, 2023
@kou
apacheGH-34977: [C++] Fix "Requires" format in arrow-dataset.pc
c89e487
@github-actions github-actions bot mentioned this issue Apr 7, 2023
Merged
@kou kou closed this as completed in #34978 Apr 7, 2023
@kou kou added this to the 12.0.0 milestone Apr 7, 2023
kou added a commit that referenced this issue Apr 7, 2023
@kou
GH-34977: [C++] Fix "Requires" format in arrow-dataset.pc (#34978)
82d48a9 
### Rationale for this change

`Requires: arrow_acero;parquet` is invalid.

### What changes are included in this PR?

Use `Requires: arrow-acero parquet`.

### Are these changes tested?

Yes.

### Are there any user-facing changes?

Yes.
* Closes: #34977

Authored-by: Sutou Kouhei <kou@clear-code.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
liujiacheng777 pushed a commit to LoongArch-Python/arrow that referenced this issue May 11, 2023
@kou @liujiacheng777
apacheGH-34977: [C++] Fix "Requires" format in arrow-dataset.pc (apac…
4de334f 
…he#34978)

### Rationale for this change

`Requires: arrow_acero;parquet` is invalid.

### What changes are included in this PR?

Use `Requires: arrow-acero parquet`.

### Are these changes tested?

Yes.

### Are there any user-facing changes?

Yes.
* Closes: apache#34977

Authored-by: Sutou Kouhei <kou@clear-code.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
ArgusLi pushed a commit to Bit-Quill/arrow that referenced this issue May 15, 2023
@kou @ArgusLi
apacheGH-34977: [C++] Fix "Requires" format in arrow-dataset.pc (apac…
2e20646 
…he#34978)

### Rationale for this change

`Requires: arrow_acero;parquet` is invalid.

### What changes are included in this PR?

Use `Requires: arrow-acero parquet`.

### Are these changes tested?

Yes.

### Are there any user-facing changes?

Yes.
* Closes: apache#34977

Authored-by: Sutou Kouhei <kou@clear-code.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
rtpsw pushed a commit to rtpsw/arrow that referenced this issue May 16, 2023
@kou @rtpsw
apacheGH-34977: [C++] Fix "Requires" format in arrow-dataset.pc (apac…
02a6aed 
…he#34978)

### Rationale for this change

`Requires: arrow_acero;parquet` is invalid.

### What changes are included in this PR?

Use `Requires: arrow-acero parquet`.

### Are these changes tested?

Yes.

### Are there any user-facing changes?

Yes.
* Closes: apache#34977

Authored-by: Sutou Kouhei <kou@clear-code.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kou kou

Labels
Component: Continuous Integration Component: GLib Type: bug
Projects
None yet
Milestone
12.0.0
Development

Successfully merging a pull request may close this issue.

GH-34977: [C++] Fix "Requires" format in arrow-dataset.pc kou/arrow
1 participant
@kou