Skip to content

GH-55275: [C++][FlightRPC][ODBC] Unset CPACK_WIX_PRODUCT_GUID to allow Windows installer upgrade - #55276

Open
alinaliBQ wants to merge 1 commit into
apache:mainfrom
dremio:gh-55275-msi-allow-upgrade
Open

alinaliBQ wants to merge 1 commit into
apache:mainfrom
dremio:gh-55275-msi-allow-upgrade

Conversation

@alinaliBQ

@alinaliBQ alinaliBQ commented Oct 26, 2026 •

Copy link
Copy Markdown
Collaborator

Rationale for this change

GH-55275

https://cmake.org/cmake/help/v4.5/cpack_gen/wix.html#variable:CPACK_WIX_PRODUCT_GUID

CPACK_WIX_PRODUCT_GUID
Product GUID (Product/@Id)

Will be automatically generated unless explicitly provided.

If explicitly provided this will set the Product Id of your installer.

The installer will abort if it detects a pre-existing installation that uses the same GUID.

Highlights my own.

What changes are included in this PR?

Removed hard-coded CPACK_WIX_PRODUCT_GUID. Then CPack will generate the UUID value for Product ID during each build, and it should unblock users from upgrading using ODBC installer.

Are these changes tested?

Regression tested in CI.
Tested installer on local Windows machine and was able to upgrade.

Are there any user-facing changes?

no

Was AI used for this PR?

In accordance to the AI generation guidelines, please disclose below whether and how AI was used in this PR.

PR code and description written by:

  • Human
  • AI

Reviewed before submission by:

  • Human
  • AI
  • Not reviewed

@alinaliBQ alinaliBQ added the CI: Extra: C++ Run extra C++ CI label Oct 26, 2026
@alinaliBQ
alinaliBQ marked this pull request as ready for review October 27, 2026 18:47
@alinaliBQ
alinaliBQ requested a review from lidavidm as a code owner October 27, 2026 18:47
@alinaliBQ

Copy link
Copy Markdown
Collaborator Author

@kou @lidavidm PR is ready for review

@amoeba amoeba left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

+1, thanks for catching this

@github-actions github-actions Bot added awaiting committer review Awaiting committer review and removed awaiting review Awaiting review labels Oct 27, 2026
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