Skip to content

GH-49092: [C++][FlightRPC][CI] Nightly Packaging: Add dev-yyyy-mm-dd to ODBC MSI name#49151

Open
alinaliBQ wants to merge 3 commits intoapache:mainfrom
Bit-Quill:gh-49092-rename-nighly-odbc
Open

GH-49092: [C++][FlightRPC][CI] Nightly Packaging: Add dev-yyyy-mm-dd to ODBC MSI name#49151
alinaliBQ wants to merge 3 commits intoapache:mainfrom
Bit-Quill:gh-49092-rename-nighly-odbc

Conversation

@alinaliBQ
Copy link
Collaborator

@alinaliBQ alinaliBQ commented Feb 4, 2026

Rationale for this change

#49092

What changes are included in this PR?

  • Add dev-yyyy-mm-dd to ODBC MSI name. This is a similar approach to R nightly.

Before: Apache Arrow Flight SQL ODBC-1.0.0-win64.msi. After: Apache Arrow Flight SQL ODBC-1.0.0-dev-2026-02-04-win64.msi.

Are these changes tested?

Tested in CI. Successfully renamed file: https://github.com/apache/arrow/actions/runs/21686252848/job/62534629714?pr=49151#step:3:26

Are there any user-facing changes?

Yes, the nightly ODBC file names will be changed as described above.

@github-actions github-actions bot added CI: Extra: C++ Run extra C++ CI awaiting review Awaiting review labels Feb 4, 2026
@alinaliBQ alinaliBQ marked this pull request as ready for review February 4, 2026 20:21
@alinaliBQ
Copy link
Collaborator Author

Hi @raulcd @kou This PR for renaming ODBC for Nightly package is ready for review. 🙂
cc @amoeba @lidavidm

@alinaliBQ alinaliBQ force-pushed the gh-49092-rename-nighly-odbc branch from 283fd67 to 406bba2 Compare February 4, 2026 20:26
Comment on lines +478 to +485
msi_name=$(ls *.msi)
echo $msi_name
SUFFIX=dev-$(date +%Y-%m-%d)-
echo $SUFFIX
INDEX=${#msi_name}-9
dev_msi_name="${msi_name:0:$INDEX}${SUFFIX}${msi_name:$INDEX}"
echo $msi_name
echo $dev_msi_name
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How about using sed?

Suggested change
msi_name=$(ls *.msi)
echo $msi_name
SUFFIX=dev-$(date +%Y-%m-%d)-
echo $SUFFIX
INDEX=${#msi_name}-9
dev_msi_name="${msi_name:0:$INDEX}${SUFFIX}${msi_name:$INDEX}"
echo $msi_name
echo $dev_msi_name
msi_name=$(ls *.msi)
dev_msi_name=$(echo ${msi_name} | sed -e "s/win64\.msi$/dev-$(date +%Y-%m-%d)-win64.msi/")

@github-actions github-actions bot added awaiting changes Awaiting changes and removed awaiting review Awaiting review labels Feb 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting changes Awaiting changes CI: Extra: C++ Run extra C++ CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants