scp2/source/ooo/common_brand.scp | 37 ---------------------- solenv/bin/modules/installer.pm | 1 solenv/bin/modules/installer/scriptitems.pm | 3 - solenv/bin/modules/installer/windows/msiglobal.pm | 28 ---------------- 4 files changed, 1 insertion(+), 68 deletions(-)
New commits: commit 9919d02c1e9be7ab2f2690ca16da65b75ebde666 Author: Stephan Bergmann <stephan.bergm...@allotropia.de> AuthorDate: Fri Jan 12 16:46:39 2024 +0100 Commit: Stephan Bergmann <stephan.bergm...@allotropia.de> CommitDate: Mon Jan 22 09:33:27 2024 +0100 Remove (Windows-only) ProductCode from setup.ini and version.ini (See a31f334d36b5735ba6fc8d0f89e834a73bdcc561 "Windows MAR update issues with program/{setup,version}.ini" for why this cleanup is needed.) There were traces of actually reading that entry from ini-files in commits like 3f01a5e9e4ec757eb26cc056a161fb7832ccd5d6 "INTEGRATION: CWS nativefixer12 (1.2.2); FILE MERGED", d9d86a2d662f38aa0473211fbf3ef2241b1c9c93 "INTEGRATION: CWS nativefixer18 (1.3.2); FILE MERGED", and 2521869690771c7af094520edf92ee8d2c5573a1 "INTEGRATION: CWS native36 (1.7.4); FILE MERGED", but all of them have since been removed with commits like 50c26300e5b5ae9671f18a9e449516604d16105f "Remove lots of dead code", 26c142ca5f2b405b02ab5701dfaeab7bf281a727 "Kill the ancient StarOffice "patch" concept", and e0ea85f61a5914508921b5e73119516219afa158 "we don't build language packs on Windows". (And all of the original code appears to having been related to ancient Windows patch mechanisms.) Change-Id: I552830faf9006bae951fc60e97804abd9de718bb Reviewed-on: https://gerrit.libreoffice.org/c/core/+/161979 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <stephan.bergm...@allotropia.de> diff --git a/scp2/source/ooo/common_brand.scp b/scp2/source/ooo/common_brand.scp index 68d19628c0f3..303bf24470dd 100644 --- a/scp2/source/ooo/common_brand.scp +++ b/scp2/source/ooo/common_brand.scp @@ -500,19 +500,6 @@ End #ifdef WNT -ProfileItem gid_Brand_Profileitem_Version_Productcode - ProfileID = gid_Brand_Profile_Version_Ini; - ModuleID = gid_Module_Root_Brand; - Section = "Version"; - Order = 8; - Key = "ProductCode"; - Value = "<productcode>"; -End - -#endif - -#ifdef WNT - ProfileItem gid_Brand_Profileitem_Version_Upgradecode ProfileID = gid_Brand_Profile_Version_Ini; ModuleID = gid_Module_Root_Brand; @@ -831,17 +818,6 @@ ProfileItem gid_Brand_Profileitem_Setup_Findproduct End #endif -#ifdef WNT -ProfileItem gid_Brand_Profileitem_Setup_Productcode - ProfileID = gid_Brand_Profile_Setup_Ini; - ModuleID = gid_Module_Root_Brand; - Section = "Bootstrap"; - Order = 6; - Key = "ProductCode"; - Value = "<productcode>"; -End -#endif - #ifdef WNT ProfileItem gid_Brand_Profileitem_Setup_Upgradecode ProfileID = gid_Brand_Profile_Setup_Ini; diff --git a/solenv/bin/modules/installer/scriptitems.pm b/solenv/bin/modules/installer/scriptitems.pm index 18ccd81d2c87..040bc068889f 100644 --- a/solenv/bin/modules/installer/scriptitems.pm +++ b/solenv/bin/modules/installer/scriptitems.pm @@ -625,7 +625,7 @@ sub changing_name_of_language_dependent_keys ################################################################################ # Replacement of setup variables in ConfigurationItems and ProfileItems -# <productkey>, <buildid>, <sequence_languages>, <productcode>, <upgradecode>, <productupdate> +# <productkey>, <buildid>, <sequence_languages>, <upgradecode>, <productupdate> ################################################################################ sub replace_setup_variables @@ -665,7 +665,6 @@ sub replace_setup_variables $value =~ s/\<buildid\>/$buildidstring/; $value =~ s/\<sequence_languages\>/$languagesstring/; $value =~ s/\<productkey\>/$productkey/; - $value =~ s/\<productcode\>/$installer::globals::productcode/; $value =~ s/\<upgradecode\>/$installer::globals::upgradecode/; $value =~ s/\<alllanguages\>/$languagesstring/; $value =~ s/\<sourceid\>/$installer::globals::build/; commit f9ff0df6f3a3339c891f764e5eecda9aa03612c0 Author: Stephan Bergmann <stephan.bergm...@allotropia.de> AuthorDate: Fri Jan 12 15:00:02 2024 +0100 Commit: Stephan Bergmann <stephan.bergm...@allotropia.de> CommitDate: Mon Jan 22 09:33:19 2024 +0100 Remove (Windows-only) MsiProductVersion from version.ini (See a31f334d36b5735ba6fc8d0f89e834a73bdcc561 "Windows MAR update issues with program/{setup,version}.ini" for why this cleanup is needed.) The ini-file entry had originally been added with d295757e7ec2cf0f7d83cf903ecb7c115a354bb9 "INTEGRATION: CWS nativefixer11 (1.23.2); FILE MERGED", and setting its value with b993422e59287924d7d85afc75eb9cffc66311db "INTEGRATION: CWS nativefixer11 (1.21.38); FILE MERGED" and 127dd8d32a87cecdc678d4b8288142f65a7a1fc9 "INTEGRATION: CWS nativefixer11 (1.45.2); FILE MERGED", but it doesn't appear to ever have been actually used for anything, other than being recorded in the ini-file. Lets remove it completely for good. Change-Id: I5f0372360416c2c6949ee2d5f0cc7e6babd8ba79 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/161978 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <stephan.bergm...@allotropia.de> diff --git a/scp2/source/ooo/common_brand.scp b/scp2/source/ooo/common_brand.scp index 5b8b907f2a9c..68d19628c0f3 100644 --- a/scp2/source/ooo/common_brand.scp +++ b/scp2/source/ooo/common_brand.scp @@ -533,19 +533,6 @@ ProfileItem gid_Brand_Profileitem_Version_Alllanguages Value = "<alllanguages>"; End -#ifdef WNT - -ProfileItem gid_Brand_Profileitem_Version_Msiproductversion - ProfileID = gid_Brand_Profile_Version_Ini; - ModuleID = gid_Module_Root_Brand; - Section = "Version"; - Order = 14; - Key = "MsiProductVersion"; - Value = "<msiproductversion>"; -End - -#endif - ProfileItem gid_Brand_Profileitem_Version_Updateurl ProfileID = gid_Brand_Profile_Version_Ini; ModuleID = gid_Module_Root_Brand; diff --git a/solenv/bin/modules/installer.pm b/solenv/bin/modules/installer.pm index a22385924b92..33c0c5b6e459 100644 --- a/solenv/bin/modules/installer.pm +++ b/solenv/bin/modules/installer.pm @@ -1325,7 +1325,6 @@ sub run { # Creating the important dynamic idt files installer::windows::msiglobal::set_msiproductversion($allvariableshashref); - installer::windows::msiglobal::put_msiproductversion_into_bootstrapfile($filesinproductlanguageresolvedarrayref); # Add cabinet assignments to files installer::windows::file::assign_cab_to_files($filesinproductlanguageresolvedarrayref); diff --git a/solenv/bin/modules/installer/windows/msiglobal.pm b/solenv/bin/modules/installer/windows/msiglobal.pm index 6d4282709ab9..38146abb51d2 100644 --- a/solenv/bin/modules/installer/windows/msiglobal.pm +++ b/solenv/bin/modules/installer/windows/msiglobal.pm @@ -1417,34 +1417,6 @@ sub set_msiproductversion } } -################################################################################# -# Including the msi product version into the bootstrap.ini, Windows only -################################################################################# - -sub put_msiproductversion_into_bootstrapfile -{ - my ($filesref) = @_; - - for ( my $i = 0; $i <= $#{$filesref}; $i++ ) - { - my $onefile = ${$filesref}[$i]; - - if ( $onefile->{'gid'} eq "gid_Brand_Profile_Version_Ini" ) - { - my $file = installer::files::read_file($onefile->{'sourcepath'}); - - for ( my $j = 0; $j <= $#{$file}; $j++ ) - { - ${$file}[$j] =~ s/\<msiproductversion\>/$installer::globals::msiproductversion/; - } - - installer::files::save_file($onefile->{'sourcepath'}, $file); - - last; - } - } -} - #################################################################################### # Updating the file Property.idt dynamically # Content: