This was reported on IRC the other day, and the reporter never filed a bug, but I don't want to lose it. When deploying a Bugzilla with an answers file which explicitly contains everything that needs to go into localconfig, checksetup.pl stops after creating the localconfig file, complaining that new values were added to it that you might want to check before continuing, and then listing off everything in localconfig. Thus when deploying in an automated fashion (the reporter was writing a puppet module to do the deployment) he had to run checksetup.pl twice for the initial install to work. checksetup shouldn't count the addition of a variable to localconfig as a new variable if the value was explicitly provided by an answers file. I have a patch coming momentarily.

Don't push the variable name onto @new_vars unless we actually use a default value to set it.

er, I touched something funny, I didn't mean to submit yet there. New patch also doesn't push onto @new_vars if an answer file was supplied but didn't include a site_wide_secret, because it's good form to let the installer generate that anyway and not supply it, and we don't want to trip up installation if the user does that. We still notify the user if we generated it in interactive mode though.

(In reply to Dave Miller [:justdave] (justdave@bugzilla.org) from comment #0) > This was reported on IRC the other day, and the reporter never filed a bug, > but I don't want to lose it. diegodnova@gmail.com was the reporter for what it's worth, he didn't have a bugzilla account at the time so I couldn't CC him.

Comment on attachment 8410495 [details] [diff] [review] Patch v2 >+ if (!(($name eq 'site_wide_secret') && (scalar(keys %$answer) > 0))) { I think: if (!scalar(keys %$answer) || $name ne 'site_wide_secret') would be more readable. I had to ask on IRC to understand it correctly. >+ push @new_vars, $var); Problem with parens.

Don't know if you want it for 4.4 too, so I will let you set the target milestone.

Yeah, I agree, that's easier to parse. Updated.

(In reply to Frédéric Buclin from comment #6) > Don't know if you want it for 4.4 too Yeah, probably should. Set as so.

Comment on attachment 8415360 [details] [diff] [review] Patch v3 >- push(@new_vars, $name); >+ push(@new_vars, $var); Err... you want to push $name, not $var, else you get: This version of Bugzilla contains some variables that you may want to change and adapt to your local settings. The following variables are new to ./localconfig since you last ran checksetup.pl: HASH(0xc269838), HASH(0xc28b570), HASH(0xc28b5b0), HASH(0xc28b600), HASH(0xc28bf60), HASH(0xc28bfa0), HASH(0xc28bfe0), HASH(0xc295c18),

Doh. where the hell was my brain?

Comment on attachment 8415363 [details] [diff] [review] Patch v4 r=LpSolit

Trunk: To ssh://gitolite3@git.mozilla.org/bugzilla/bugzilla.git 7c6963d..082cea0 master -> master 4.4: To ssh://gitolite3@git.mozilla.org/bugzilla/bugzilla.git 3b4ddd1..6be81e8 4.4 -> 4.4