[Libreoffice-commits] core.git: 3 commits - solenv/bin

Michael Stahl mstahl at redhat.com
Mon Mar 12 20:54:00 UTC 2018


 solenv/bin/modules/installer.pm                  |    4 ----
 solenv/bin/modules/installer/environment.pm      |    2 --
 solenv/bin/modules/installer/globals.pm          |    2 --
 solenv/bin/modules/installer/parameter.pm        |    6 ------
 solenv/bin/modules/installer/scriptitems.pm      |    8 +-------
 solenv/bin/modules/installer/windows/property.pm |    6 ------
 solenv/bin/modules/installer/ziplist.pm          |   19 ++-----------------
 solenv/bin/ooinstall                             |    1 -
 8 files changed, 3 insertions(+), 45 deletions(-)

New commits:
commit b718c29265826af8e15eafc649443602624875a1
Author: Michael Stahl <mstahl at redhat.com>
Date:   Mon Mar 12 21:40:40 2018 +0100

    solenv: the installer's -b parameter is unused
    
    "build" is hard-coded to "majorminor0".
    
    Change-Id: I15271544917f70b8ae029bd7542aecfe0a46befe

diff --git a/solenv/bin/modules/installer/parameter.pm b/solenv/bin/modules/installer/parameter.pm
index 7fcec04de70f..352b6f94c19b 100644
--- a/solenv/bin/modules/installer/parameter.pm
+++ b/solenv/bin/modules/installer/parameter.pm
@@ -41,7 +41,6 @@ The following parameter are needed:
 -i: Install path of the product (/opt/openofficeorg20) (optional)
 -p: Product from product list to be created (required)
 -l: Language of the product (comma and hash) (optional, defined in productlist)
--b: Build, e.g. srx645 (optional)
 -simple: Path to do a simple install to
 -u: Path, in which zipfiles are unpacked (optional)
 -msitemplate: Source of the msi file templates (Windows compiler only)
@@ -112,7 +111,6 @@ sub getparameter
         elsif ($param eq "-s") { $installer::globals::setupscriptname = shift(@ARGV); }
         elsif ($param eq "-p") { $installer::globals::product = shift(@ARGV); }
         elsif ($param eq "-l") { $installer::globals::languagelist = shift(@ARGV); }
-        elsif ($param eq "-b") { $installer::globals::build = shift(@ARGV); }
         elsif ($param eq "-dontunzip") { $installer::globals::dounzip = 0; }
         elsif ($param eq "-pro") { $installer::globals::pro = 1; }
         elsif ($param eq "-format") { $installer::globals::packageformat = shift(@ARGV); }
commit 396f085610860104390e47cd78a8b48c057f5f71
Author: Michael Stahl <mstahl at redhat.com>
Date:   Mon Mar 12 21:13:15 2018 +0100

    solenv: installer's -m parameter & globals::minor & PRODUCTMINOR unused
    
    Change-Id: I2f550ca5b02551b84d3631e0101918af49924fe2

diff --git a/solenv/bin/modules/installer.pm b/solenv/bin/modules/installer.pm
index 3d88b9986e14..cd72284c5bc8 100644
--- a/solenv/bin/modules/installer.pm
+++ b/solenv/bin/modules/installer.pm
@@ -458,9 +458,6 @@ sub run {
         @installer::globals::logfileinfo = ();  # new logfile array and new logfile name
         installer::logger::copy_globalinfo_into_logfile();
 
-        my $logminor = "";
-        $logminor = $installer::globals::minor;
-
         my $loglanguagestring = $$languagestringref;
         my $loglanguagestring_orig = $loglanguagestring;
         if (length($loglanguagestring) > $installer::globals::max_lang_length)
@@ -474,7 +471,6 @@ sub run {
         }
 
         $installer::globals::logfilename = "log_" . $installer::globals::build;
-        if ( $logminor ne "" ) { $installer::globals::logfilename .= "_" . $logminor; }
         $installer::globals::logfilename .= "_" . $loglanguagestring;
         $installer::globals::logfilename .= ".log";
         $loggingdir = $loggingdir . $loglanguagestring . $installer::globals::separator;
diff --git a/solenv/bin/modules/installer/globals.pm b/solenv/bin/modules/installer/globals.pm
index c681b05a6d0b..a7ec5e50b209 100644
--- a/solenv/bin/modules/installer/globals.pm
+++ b/solenv/bin/modules/installer/globals.pm
@@ -37,7 +37,6 @@ BEGIN
 
     @languageproducts = ();
     $build = "";
-    $minor = "";
     $os = "";
     $cpuname = "";
     $com = "";
diff --git a/solenv/bin/modules/installer/parameter.pm b/solenv/bin/modules/installer/parameter.pm
index 505cdd7c3152..7fcec04de70f 100644
--- a/solenv/bin/modules/installer/parameter.pm
+++ b/solenv/bin/modules/installer/parameter.pm
@@ -42,7 +42,6 @@ The following parameter are needed:
 -p: Product from product list to be created (required)
 -l: Language of the product (comma and hash) (optional, defined in productlist)
 -b: Build, e.g. srx645 (optional)
--m: Minor, e.g. m10 (optional)
 -simple: Path to do a simple install to
 -u: Path, in which zipfiles are unpacked (optional)
 -msitemplate: Source of the msi file templates (Windows compiler only)
@@ -114,7 +113,6 @@ sub getparameter
         elsif ($param eq "-p") { $installer::globals::product = shift(@ARGV); }
         elsif ($param eq "-l") { $installer::globals::languagelist = shift(@ARGV); }
         elsif ($param eq "-b") { $installer::globals::build = shift(@ARGV); }
-        elsif ($param eq "-m") { $installer::globals::minor = shift(@ARGV); }
         elsif ($param eq "-dontunzip") { $installer::globals::dounzip = 0; }
         elsif ($param eq "-pro") { $installer::globals::pro = 1; }
         elsif ($param eq "-format") { $installer::globals::packageformat = shift(@ARGV); }
@@ -521,8 +519,6 @@ sub outputparameter
     push(@output, "Product: $installer::globals::product\n");
     push(@output, "BuildID: $installer::globals::buildid\n");
     push(@output, "Build: $installer::globals::build\n");
-    if ( $installer::globals::minor ) { push(@output, "Minor: $installer::globals::minor\n"); }
-    else  { push(@output, "No minor set\n"); }
     if ( $installer::globals::pro ) { push(@output, "Product version\n"); }
     else  { push(@output, "Non-Product version\n"); }
     if ( $installer::globals::rootpath eq "" ) { push(@output, "Using default installpath\n"); }
diff --git a/solenv/bin/modules/installer/scriptitems.pm b/solenv/bin/modules/installer/scriptitems.pm
index 267ae20cdcfa..b9572dcb27dc 100644
--- a/solenv/bin/modules/installer/scriptitems.pm
+++ b/solenv/bin/modules/installer/scriptitems.pm
@@ -638,20 +638,15 @@ sub replace_setup_variables
 
     # string $buildid, which is used to replace the setup variable <buildid>
 
-    my $localminor = "flat";
-    if ( $installer::globals::minor ne "" ) { $localminor = $installer::globals::minor; }
-
     my $localbuild = $installer::globals::build;
 
     if ( $localbuild =~ /^\s*(\w+?)(\d+)\s*$/ ) { $localbuild = $2; }   # using "680" instead of "src680"
 
     my $buildidstring = `cd $ENV{'SRCDIR'} 2>&1 >/dev/null && git log -n 1 --pretty=format:"%H"`;
     if ($? || !$buildidstring) {
-        $buildidstring = $localbuild . $localminor . "(Build:" . $installer::globals::buildid . ")";
+        $buildidstring = $localbuild . "(Build:" . $installer::globals::buildid . ")";
     }
 
-    if ( $localminor =~ /^\s*\w(\d+)\w*\s*$/ ) { $localminor = $1; }
-
     my $updateid = $productname . "_" . $libo_version_major . "_" . $$languagestringref;
     $updateid =~ s/ /_/g;
 
diff --git a/solenv/bin/modules/installer/windows/property.pm b/solenv/bin/modules/installer/windows/property.pm
index 61b18ac29a56..baf8421ced97 100644
--- a/solenv/bin/modules/installer/windows/property.pm
+++ b/solenv/bin/modules/installer/windows/property.pm
@@ -221,12 +221,6 @@ sub set_important_properties
         push(@{$propertyfile}, $onepropertyline);
     }
 
-    if ( $allvariables->{'PRODUCTMINOR'} )
-    {
-        my $onepropertyline = "PRODUCTMINOR" . "\t" . $allvariables->{'PRODUCTMINOR'} . "\n";
-        push(@{$propertyfile}, $onepropertyline);
-    }
-
     if ( $allvariables->{'PRODUCTBUILDID'} )
     {
         my $onepropertyline = "PRODUCTBUILDID" . "\t" . $allvariables->{'PRODUCTBUILDID'} . "\n";
diff --git a/solenv/bin/modules/installer/ziplist.pm b/solenv/bin/modules/installer/ziplist.pm
index 2d80061dd419..b76c62fc77d9 100644
--- a/solenv/bin/modules/installer/ziplist.pm
+++ b/solenv/bin/modules/installer/ziplist.pm
@@ -768,14 +768,10 @@ sub replace_variables_in_ziplist_variables
 {
     my ($blockref) = @_;
 
-    my $localminor = "";
-    if ( $installer::globals::minor ) { $localminor = $installer::globals::minor; }
-
     for ( my $i = 0; $i <= $#{$blockref}; $i++ )
     {
         ${$blockref}[$i] =~ s/\{milestone\}//;
-        if ( $localminor ) { ${$blockref}[$i] =~ s/\{minor\}/$localminor/; }
-        else { ${$blockref}[$i] =~ s/\{minor\}//; }
+        ${$blockref}[$i] =~ s/\{minor\}//;
         if ( $installer::globals::buildid ) { ${$blockref}[$i] =~ s/\{buildid\}/$installer::globals::buildid/; }
         else { ${$blockref}[$i] =~ s/\{buildid\}//; }
         if ( $installer::globals::build ) { ${$blockref}[$i] =~ s/\{buildsource\}/$installer::globals::build/; }
@@ -826,11 +822,6 @@ sub add_variables_to_allvariableshashref
     if ( $localbuild =~ /^\s*(\w+?)(\d+)\s*$/ ) { $localbuild = $2; }   # using "680" instead of "src680"
     $variableshashref->{'PRODUCTMAJOR'} = $localbuild;
 
-    my $localminor = "";
-    if ( $installer::globals::minor ne "" ) { $localminor = $installer::globals::minor; }
-    if ( $localminor =~ /^\s*\w(\d+)\w*\s*$/ ) { $localminor = $1; }
-    $variableshashref->{'PRODUCTMINOR'} = $localminor;
-
     $variableshashref->{'PRODUCTBUILDID'} = $installer::globals::buildid;
 }
 
commit 36b23fdd9419619bacd07311b10c34640836c044
Author: Michael Stahl <mstahl at redhat.com>
Date:   Mon Mar 12 20:41:03 2018 +0100

    solenv: LAST_MINOR is effectively dead
    
    Change-Id: I07d90c68934712295952e6e6725e0206c16a5b7e

diff --git a/solenv/bin/modules/installer/environment.pm b/solenv/bin/modules/installer/environment.pm
index 38448a8ba9fb..05dfdbfb6787 100644
--- a/solenv/bin/modules/installer/environment.pm
+++ b/solenv/bin/modules/installer/environment.pm
@@ -108,8 +108,6 @@ sub set_global_environment_variables
     $installer::globals::cpuname = $environment->{'CPUNAME'};
     $installer::globals::platformid = $environment->{'PLATFORMID'};
 
-    if ( $ENV{'LAST_MINOR'} ) { $installer::globals::lastminor = $ENV{'LAST_MINOR'}; }
-
     if ( $ENV{'ENABLE_DBGUTIL'} ) {} else { $installer::globals::pro = 1; }
 
     if ( $ENV{'VERBOSE'} && ( (lc $ENV{'VERBOSE'}) eq "false" ) ) { $installer::globals::quiet = 1; }
diff --git a/solenv/bin/modules/installer/globals.pm b/solenv/bin/modules/installer/globals.pm
index 2cd0cc21062c..c681b05a6d0b 100644
--- a/solenv/bin/modules/installer/globals.pm
+++ b/solenv/bin/modules/installer/globals.pm
@@ -38,7 +38,6 @@ BEGIN
     @languageproducts = ();
     $build = "";
     $minor = "";
-    $lastminor = "";
     $os = "";
     $cpuname = "";
     $com = "";
diff --git a/solenv/bin/modules/installer/scriptitems.pm b/solenv/bin/modules/installer/scriptitems.pm
index bd4437d315de..267ae20cdcfa 100644
--- a/solenv/bin/modules/installer/scriptitems.pm
+++ b/solenv/bin/modules/installer/scriptitems.pm
@@ -640,7 +640,6 @@ sub replace_setup_variables
 
     my $localminor = "flat";
     if ( $installer::globals::minor ne "" ) { $localminor = $installer::globals::minor; }
-    else { $localminor = $installer::globals::lastminor; }
 
     my $localbuild = $installer::globals::build;
 
diff --git a/solenv/bin/modules/installer/ziplist.pm b/solenv/bin/modules/installer/ziplist.pm
index 638d6716dd95..2d80061dd419 100644
--- a/solenv/bin/modules/installer/ziplist.pm
+++ b/solenv/bin/modules/installer/ziplist.pm
@@ -768,17 +768,12 @@ sub replace_variables_in_ziplist_variables
 {
     my ($blockref) = @_;
 
-    my $milestonevariable = $installer::globals::lastminor;
-    $milestonevariable =~ s/m//;
-    $milestonevariable =~ s/s/\./;
-
-    my $localminor = $installer::globals::lastminor;
+    my $localminor = "";
     if ( $installer::globals::minor ) { $localminor = $installer::globals::minor; }
 
     for ( my $i = 0; $i <= $#{$blockref}; $i++ )
     {
-        if ($installer::globals::lastminor) { ${$blockref}[$i] =~ s/\{milestone\}/$milestonevariable/; }
-        else { ${$blockref}[$i] =~ s/\{milestone\}//; }
+        ${$blockref}[$i] =~ s/\{milestone\}//;
         if ( $localminor ) { ${$blockref}[$i] =~ s/\{minor\}/$localminor/; }
         else { ${$blockref}[$i] =~ s/\{minor\}//; }
         if ( $installer::globals::buildid ) { ${$blockref}[$i] =~ s/\{buildid\}/$installer::globals::buildid/; }
@@ -833,7 +828,6 @@ sub add_variables_to_allvariableshashref
 
     my $localminor = "";
     if ( $installer::globals::minor ne "" ) { $localminor = $installer::globals::minor; }
-    else { $localminor = $installer::globals::lastminor; }
     if ( $localminor =~ /^\s*\w(\d+)\w*\s*$/ ) { $localminor = $1; }
     $variableshashref->{'PRODUCTMINOR'} = $localminor;
 
diff --git a/solenv/bin/ooinstall b/solenv/bin/ooinstall
index 83507a7bf647..aca6b01ae11d 100755
--- a/solenv/bin/ooinstall
+++ b/solenv/bin/ooinstall
@@ -56,7 +56,6 @@ if ($help) {
 }
 
 my $BUILD=$ENV{LIBO_VERSION_PATCH};
-$ENV{LAST_MINOR} = 'm0';
 $ENV{OUT} = "../FIXME";
 $ENV{LOCAL_OUT} = $ENV{OUT};
 $ENV{LOCAL_COMMON_OUT} = $ENV{OUT};


More information about the Libreoffice-commits mailing list