bugzilla.mozilla.org will be intermittently unavailable on Saturday, March 24th, from 16:00 until 20:00 UTC.

localconfig says that $webservergroup = "" will prevent permission changes

RESOLVED FIXED in Bugzilla 2.16



16 years ago
5 years ago


(Reporter: Hixie (not reading bugmail), Assigned: justdave)


Bugzilla 2.16


(Whiteboard: want for 2.16rc2)


(1 attachment)



16 years ago
Apparently the following is no longer true:

# This is the group your web server runs on.
# If you have a windows box, ignore this setting.
# If you do not wish for checksetup to adjust the permissions of anything,
# set this to "".
# If you set this to anything besides "", you will need to run checksetup.pl
# as root.
$webservergroup = "";

Yet that is what it still said in my localconfig after I updated.
This is not really a docs issue. Moving to Bugzilla-General. 2.16 issue - a
comment tidyup?

Assignee: mbarnson → justdave
Component: Documentation → Bugzilla-General
Target Milestone: --- → Bugzilla 2.16
The text in checksetup has changed, but that won't affect the stuff already in
localconfig, since its not a new variable...
So people upgrading will have localconfig files whose comments are incorrect?

Is there any way we can fix this?

We make this a docs issue, I guess. There aren't any delimeters - we just check
if the var is defined. The only way to force a comment change would be to rename
the variable, which would add a new comment. It wouldn't delete the old one, though.

Comment 5

16 years ago
Or, we could add code to change the comment...
Long term it might be a good idea to get checksetup.pl to read in localconfig
and rewrite it with the current comments?  We would need to add a comment to the
top that any comment changes will get lost, and we should release note it, however.

Whether this is too much for 2.16, I'm not sure.
I suppose if we found an exact match on the comment, we could change it.
However, people may have there own comments in localconfig.

In any event, I dont think this is a 2.16 issue at this stage.
I'd say the ability to keep comments up to date far outweighs the ability to add
comments to localconfig.
Lets fix the comment for new installs to be accurate.  If people want the latest
and greatest comments they can blow away localconfig and recreate it.

I agree it would be best if checksetup.pl were to read in and re-write the
entire file with the current comments every time it ran, but That's Another Bug
(bug 147776, in fact).  Let's just fix the issue at hand here.
Keywords: patch, review
Whiteboard: want for 2.16rc2

Comment 11

16 years ago
Comment on attachment 85386 [details] [diff] [review]

Although this is not totally accurate either, it does convey the right message
("set your webservergroup || die").

Attachment #85386 - Flags: review+
Attachment #85386 - Flags: review+
Checking in checksetup.pl;
/cvsroot/mozilla/webtools/bugzilla/checksetup.pl,v  <--  checksetup.pl
new revision: 1.153; previous revision: 1.152

Checking in checksetup.pl;
/cvsroot/mozilla/webtools/bugzilla/checksetup.pl,v  <--  checksetup.pl
new revision:; previous revision:
Last Resolved: 16 years ago
Resolution: --- → FIXED
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.