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

[C++][Docs] Add documentation for minimal build flags #34638

Closed
benibus opened this issue Mar 19, 2023 · 0 comments · Fixed by #34693
Closed

[C++][Docs] Add documentation for minimal build flags #34638

benibus opened this issue Mar 19, 2023 · 0 comments · Fixed by #34693
Assignees
@benibus
Labels
Component: C++ Component: Documentation Type: enhancement
Milestone
12.0.0

Comments

@benibus
Copy link
Collaborator

benibus commented Mar 19, 2023

Describe the enhancement requested

It would be useful to publicly document which cmake flags are necessary for producing a minimal build of the C++ library - i.e. nothing but basic Arrow/IPC (and possibly Parquet) support. There would be some extra dependencies for test-enabled builds so we might want to make note of those as well.

Component(s)

C++, Documentation
@benibus benibus self-assigned this Mar 19, 2023
@github-actions github-actions bot mentioned this issue Mar 22, 2023
Merged
westonpace pushed a commit that referenced this issue Mar 24, 2023
@benibus
GH-34638: [C++][Docs] Add documentation for minimal build flags (#34693)
e7d6c13 
### Rationale for this change

There's an ongoing effort to support smaller builds of Arrow C++ for simple format integration so explicitly documenting the necessary flags would be useful. Also, some recent changes to the build system may require disclosure.

### What changes are included in this PR?

Adds documentation to `cpp/building.rst` regarding:
- Minimal cmake flags for basic format support
- Recent changes to `ARROW_COMPUTE` and how compute kernels are included in builds.

### Are these changes tested?

N/A

### Are there any user-facing changes?

N/A
* Closes: #34638

Authored-by: benibus <bpharks@gmx.com>
Signed-off-by: Weston Pace <weston.pace@gmail.com>
@westonpace westonpace added this to the 12.0.0 milestone Mar 24, 2023
rtpsw pushed a commit to rtpsw/arrow that referenced this issue Mar 27, 2023
@benibus @rtpsw
apacheGH-34638: [C++][Docs] Add documentation for minimal build flags (…
2a7e1be 
…apache#34693)

### Rationale for this change

There's an ongoing effort to support smaller builds of Arrow C++ for simple format integration so explicitly documenting the necessary flags would be useful. Also, some recent changes to the build system may require disclosure.

### What changes are included in this PR?

Adds documentation to `cpp/building.rst` regarding:
- Minimal cmake flags for basic format support
- Recent changes to `ARROW_COMPUTE` and how compute kernels are included in builds.

### Are these changes tested?

N/A

### Are there any user-facing changes?

N/A
* Closes: apache#34638

Authored-by: benibus <bpharks@gmx.com>
Signed-off-by: Weston Pace <weston.pace@gmail.com>
ArgusLi pushed a commit to Bit-Quill/arrow that referenced this issue May 15, 2023
@benibus @ArgusLi
apacheGH-34638: [C++][Docs] Add documentation for minimal build flags (…
665b894 
…apache#34693)

### Rationale for this change

There's an ongoing effort to support smaller builds of Arrow C++ for simple format integration so explicitly documenting the necessary flags would be useful. Also, some recent changes to the build system may require disclosure.

### What changes are included in this PR?

Adds documentation to `cpp/building.rst` regarding:
- Minimal cmake flags for basic format support
- Recent changes to `ARROW_COMPUTE` and how compute kernels are included in builds.

### Are these changes tested?

N/A

### Are there any user-facing changes?

N/A
* Closes: apache#34638

Authored-by: benibus <bpharks@gmx.com>
Signed-off-by: Weston Pace <weston.pace@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@benibus benibus

Labels
Component: C++ Component: Documentation Type: enhancement
Projects
None yet
Milestone
12.0.0
Development

Successfully merging a pull request may close this issue.

GH-34638: [C++][Docs] Add documentation for minimal build flags benibus/arrow
2 participants
@westonpace @benibus