[Libreoffice-commits] core.git: solenv/bin
Mike Kaganski (via logerrit)
logerrit at kemper.freedesktop.org
Fri Sep 4 17:33:35 UTC 2020
solenv/bin/modules/installer/scriptitems.pm | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
New commits:
commit 9b8a8dc74dd352820141d2b7e5320bcda39a7814
Author: Mike Kaganski <mike.kaganski at collabora.com>
AuthorDate: Fri Sep 4 17:19:38 2020 +0300
Commit: Mike Kaganski <mike.kaganski at collabora.com>
CommitDate: Fri Sep 4 19:32:52 2020 +0200
Avoid warnings about use of uninitialized value
Like these:
Use of uninitialized value in exists at C:/lo/src/core/solenv/bin/modules/installer/scriptitems.pm line 402.
Debugging the failing values, these are:
element 113
Dir => gid_Dir_Autotext_Language
HostName => LibreOfficeDev 7/share/autotext/qtz
modules => gid_Module_Langpack_Basis_qtz
specificlanguage => qtz
Styles => (CREATE)
element 512
Dir => gid_Brand_Dir_Share_Uno_Packages
HostName => LibreOfficeDev 7/share/uno_packages
modules => gid_Module_Root,gid_Module_Root_Brand
specificlanguage =>
Styles => (CREATE)
element 513
Dir => gid_Brand_Dir_Share_Uno_Packages_Cache
HostName => LibreOfficeDev 7/share/uno_packages/cache
modules => gid_Module_Root,gid_Module_Root_Brand
specificlanguage =>
Styles => (CREATE)
element 514
Dir => gid_Brand_Dir_Share_Uno_Packages_Cache_Uno_Packages
HostName => LibreOfficeDev 7/share/uno_packages/cache/uno_packages
modules => gid_Module_Root
specificlanguage =>
Styles => (CREATE)
Change-Id: I9c91e2fa9bbaab4c15e96dbbdc710141e500c482
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/102053
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski at collabora.com>
diff --git a/solenv/bin/modules/installer/scriptitems.pm b/solenv/bin/modules/installer/scriptitems.pm
index 81f0271dfe7f..54247d01d329 100644
--- a/solenv/bin/modules/installer/scriptitems.pm
+++ b/solenv/bin/modules/installer/scriptitems.pm
@@ -399,7 +399,8 @@ sub remove_not_required_spellcheckerlanguage_files
for ( my $i = 0; $i <= $#{$filesarrayref}; $i++ )
{
my $onefile = ${$filesarrayref}[$i];
- if ( exists($installer::globals::spellcheckerfilehash{$onefile->{'gid'}}) )
+ # FIXME: some items don't have 'gid'
+ if ( (defined $onefile->{'gid'}) && exists($installer::globals::spellcheckerfilehash{$onefile->{'gid'}}) )
{
$infoline = "Spellchecker selection: Removing file $onefile->{'gid'}\n";
push( @installer::globals::logfileinfo, $infoline);
More information about the Libreoffice-commits
mailing list