146134 - checksetup.pl gives weird error message.

Status: RESOLVED FIXED

(Bugzilla :: Installation & Upgrading, defect, P2)

Description

[Matthew Tuck [:CodeMachine]]

What The Heck?

---

Checking user setup ...

Warning: you have entered a value for the "webservergroup" parameter
in localconfig, but you are not running this script as root.
This can cause permissions problems and decreased security.  If you
experience problems running Bugzilla scripts, log in as root and re-run
this script, or remove the value of the "webservergroup" parameter.
Note that any warnings about "uninitialized values" that you may
see below are caused by this.

Content-type: text/html

<H1>Software error:</H1>
<PRE>Couldn't rmdir attachment: Permission denied
</PRE>
<P>
For help, please send mail to this site's webmaster, giving this error message 
and the time and date of the error.

[Wed May 22 18:46:06 2002] checksetup.pl: Couldn't rmdir attachment: Permission
denied

Comment 1

[Bradley Baetz (:bbaetz)]

Why are we getting html-style errors? Is this cause we're not saving the die
handler arround calls to templates which load CGI.pm (And the associated eval)?

The rmdir call is coming from the template stuff (and I'm curious why its
triggereing at all, since noone change the .lastRebuild flag file recently).
What are the permissions/ownerships on data/template/* ?

Comment 2

[Dave Miller [:justdave]]

Attachment: Fix for HTML in error messages

Comment 3

[Bradley Baetz (:bbaetz)]

Attachment: I prefer this

Comment 4

[Dave Miller [:justdave]]

Comment on attachment 97856 [details] [diff] [review]
I prefer this

2xr= justdave

it works.
And it looks much cleaner :-)

You might want to change the comment to reflect what you actually did before
checking it in though.

Comment 5

[Bradley Baetz (:bbaetz)]

Attachment: stupid cvs conflicts

Comment 6

[Dave Miller [:justdave]]

Comment on attachment 97859 [details] [diff] [review]
stupid cvs conflicts

2xr= justdave

works for me.

Comment 7

[Bradley Baetz (:bbaetz)]

-> me

Comment 8

[Bradley Baetz (:bbaetz)]

Checked in.