Skip to content

ARROW-16792: [C++][CMake] Add support for using Arrow options when Arrow is used as subproject#13348

Merged
kou merged 1 commit intoapache:masterfrom
kou:cpp-force-options
Jun 10, 2022
Merged

ARROW-16792: [C++][CMake] Add support for using Arrow options when Arrow is used as subproject#13348
kou merged 1 commit intoapache:masterfrom
kou:cpp-force-options

Conversation

@kou
Copy link
Copy Markdown
Member

@kou kou commented Jun 9, 2022

No description provided.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Jun 9, 2022

@github-actions
Copy link
Copy Markdown

github-actions bot commented Jun 9, 2022

⚠️ Ticket has not been started in JIRA, please click 'Start Progress'.

@kou
Copy link
Copy Markdown
Member Author

kou commented Jun 9, 2022

@github-actions crossbow submit -g conan

@github-actions
Copy link
Copy Markdown

github-actions bot commented Jun 9, 2022

Revision: 1dbcea8

Submitted crossbow builds: ursacomputing/crossbow @ actions-e2510e4efd

Task Status
conan-maximum Github Actions
conan-minimum Github Actions

@pitrou
Copy link
Copy Markdown
Member

pitrou commented Jun 9, 2022

Hmm, why are there steps such as "Dump R install logs" in the conan builds?

Also, I notice the maximum build has a lot of components disabled still (even CSV).

@kou
Copy link
Copy Markdown
Member Author

kou commented Jun 10, 2022

Hmm, why are there steps such as "Dump R install logs" in the conan builds?

Because they reuse general Docker based test configuration: https://github.com/apache/arrow/blob/master/dev/tasks/docker-tests/github.linux.yml

Also, I notice the maximum build has a lot of components disabled still (even CSV).

I'm enabling components step by step. I'll enable all or most components eventually. :-)

@kou kou merged commit 452f11c into apache:master Jun 10, 2022
@kou kou deleted the cpp-force-options branch June 10, 2022 01:15
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