GH-47877: [C++][FlightRPC] ODBC Linux rpm installer support#49787
Draft
alinaliBQ wants to merge 2 commits intoapache:mainfrom
Draft
GH-47877: [C++][FlightRPC] ODBC Linux rpm installer support#49787alinaliBQ wants to merge 2 commits intoapache:mainfrom
alinaliBQ wants to merge 2 commits intoapache:mainfrom
Conversation
* Implement RPM installer Revert "TEMP - Disable non-ODBC items" This reverts commit efa8d84. fix formatting Remove todos ODBC rpm Installer wrap up * installing `rpm` package fixed the `cpack` command issue * Fix cpack command on CI (merge into last commit when ready) Change to use `awk` so script works on both platforms Comment out `find` and `tree` steps Change build path to be under /arrow Since docker is mounted, changing build path to be in `/arrow` will allow host machine to access docker build contents TEMP - Disable non-ODBC items Add commands to find build folder Attempt to build RPM installer * not sure if `/build/cpp` can be accessed outside of docker. Hopefully it should. * add `rpm` dependency install Indicate RPM in workflow run Add rpm `postinstall` (not tested) Remove merge conflict code * Enable ODBC installer build in CI instead of `compose.yaml`
|
|
github-actions
bot
added
Component: C++
Component: FlightRPC
Component: Documentation
CI: Extra: C++
alinaliBQ
commented
Apr 20, 2026
Comment on lines
+181
to
+200
| else() | ||
| # Linux | ||
| set(CPACK_PACKAGE_INSTALL_DIRECTORY "${CPACK_PACKAGE_NAME}") | ||
| if(${ODBC_PACKAGE_FORMAT} STREQUAL "DEB") | ||
| # GH-49595 TODO: implement DEB installer | ||
| message(STATUS "ODBC_PACKAGE_FORMAT DEB not implemented, see GH-49595") | ||
| elseif(${ODBC_PACKAGE_FORMAT} STREQUAL "RPM") | ||
| set(CPACK_RPM_PACKAGE_ARCHITECTURE x86_64) | ||
| set(CPACK_GENERATOR RPM) | ||
| set(CPACK_RPM_POST_INSTALL_SCRIPT_FILE | ||
| "${CMAKE_CURRENT_SOURCE_DIR}/install/linux/rpm/postinstall") | ||
| set(CPACK_RPM_FILE_NAME "${ODBC_UNIX_FILE_NAME}.rpm") | ||
| # Disable dependency check as ODBC embeds all third party dependencies | ||
| set(CPACK_RPM_PACKAGE_AUTOREQPROV "no") | ||
| set(CPACK_COMPONENTS_ALL_IN_ONE_PACKAGE 1) | ||
| set(CPACK_RPM_COMPONENT_INSTALL ON) | ||
| else() | ||
| message(FATAL_ERROR "ODBC_PACKAGE_FORMAT '${ODBC_PACKAGE_FORMAT}' must be DEB or RPM for Linux installer." | ||
| ) | ||
| endif() |
Collaborator
Author
There was a problem hiding this comment.
The code changes for RPM installer are ready for review. This PR contains code changes that are dependent on #49766
github-actions
bot
added
awaiting committer review
Update cpp_extra.yml
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Rationale for this change
GH-47877
What changes are included in this PR?
This PR is dependent on #49766 to undraft and merge.
Add support for Linux ODBC
.rpminstaller.In
install_odbc.sh, changed to useawkon unix platforms asseddoesn't work well on Linux.Add installer readme section.
Add
ARROW_FLIGHT_SQL_ODBC_INSTALLERoption.The
.rpminstaller installs:Are these changes tested?
Yes
Are there any user-facing changes?
Users can download RPM installer from the CI.