diff options
author | Dave Miller <justdave@mozilla.com> | 2014-05-01 22:12:25 -0400 |
---|---|---|
committer | Dave Miller <justdave@mozilla.com> | 2014-05-01 22:12:25 -0400 |
commit | 6be81e8923e9d9254b9358601b27fc7a844c3d3a (patch) | |
tree | e56753aa96a70fd3da2bcaea5362adf2d048273a | |
parent | Bug 995209 - Create a Build.PL script using Module::Build for testing/install... (diff) | |
download | bugzilla-6be81e8923e9d9254b9358601b27fc7a844c3d3a.tar.gz bugzilla-6be81e8923e9d9254b9358601b27fc7a844c3d3a.tar.bz2 bugzilla-6be81e8923e9d9254b9358601b27fc7a844c3d3a.zip |
Bug 999296: Make checksetup.pl not complain about having added new
values to localconfig when those values were supplied by an answer file.
r=LpSolit, a=justdave
-rw-r--r-- | Bugzilla/Install/Localconfig.pm | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/Bugzilla/Install/Localconfig.pm b/Bugzilla/Install/Localconfig.pm index 4f1579c86..881f6c956 100644 --- a/Bugzilla/Install/Localconfig.pm +++ b/Bugzilla/Install/Localconfig.pm @@ -205,14 +205,20 @@ sub update_localconfig { # a 256-character string for site_wide_secret. $value = undef if ($name eq 'site_wide_secret' and defined $value and length($value) == 256); - + if (!defined $value) { - push(@new_vars, $name); $var->{default} = &{$var->{default}} if ref($var->{default}) eq 'CODE'; if (exists $answer->{$name}) { $localconfig->{$name} = $answer->{$name}; } else { + # If the user did not supply an answers file, then they get + # notified about every variable that gets added. If there was + # an answer file, then we don't notify about site_wide_secret + # because we assume the intent was to auto-generate it anyway. + if (!scalar(keys %$answer) || $name ne 'site_wide_secret') { + push(@new_vars, $name); + } $localconfig->{$name} = $var->{default}; } } |