This is an automated email from the ASF dual-hosted git repository.

lidavidm pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow.git


The following commit(s) were added to refs/heads/main by this push:
     new 9eb748dde4 GH-55275: [C++][FlightRPC][ODBC] Unset 
`CPACK_WIX_PRODUCT_GUID` to allow Windows installer upgrade (#55276)
9eb748dde4 is described below

commit 9eb748dde427a102306d1aae3fbc8939f3cdc9fb
Author: Alina (Xi) Li <[email protected]>
AuthorDate: Tue Oct 27 16:02:58 2026 -0700

    GH-55275: [C++][FlightRPC][ODBC] Unset `CPACK_WIX_PRODUCT_GUID` to allow 
Windows installer upgrade (#55276)
    
    ### 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](https://arrow.apache.org/docs/dev/developers/overview.html#ai-generated-code),
 please disclose below whether and how AI was used in this PR.
    
    **PR code and description written by:**
    
    - [x] Human
    - [x] AI
    
    **Reviewed before submission by:**
    
    - [x] Human
    - [x] AI
    - [ ] Not reviewed
    
    * GitHub Issue: #55275
    
    Authored-by: Alina (Xi) Li <[email protected]>
    Signed-off-by: David Li <[email protected]>
---
 cpp/src/arrow/flight/sql/odbc/CMakeLists.txt | 1 -
 1 file changed, 1 deletion(-)

diff --git a/cpp/src/arrow/flight/sql/odbc/CMakeLists.txt 
b/cpp/src/arrow/flight/sql/odbc/CMakeLists.txt
index a280eb6260..033a2775b6 100644
--- a/cpp/src/arrow/flight/sql/odbc/CMakeLists.txt
+++ b/cpp/src/arrow/flight/sql/odbc/CMakeLists.txt
@@ -247,7 +247,6 @@ if(ARROW_FLIGHT_SQL_ODBC_INSTALLER)
 
     # Upgrade GUID is required to be unchanged for ODBC installer to upgrade
     set(CPACK_WIX_UPGRADE_GUID "DBF27A18-F8BF-423F-9E3A-957414D52C4B")
-    set(CPACK_WIX_PRODUCT_GUID "279D087B-93B5-4DC3-BA69-BCF485022A26")
   elseif(APPLE)
     # macOS and Linux
     set(CPACK_GENERATOR "productbuild")

Reply via email to