Closed Bug 223304 Opened 21 years ago Closed 18 years ago

AddNote doesn't report errors if it can't write to notes.txt

Categories

(Webtools Graveyard :: Tinderbox, defect)

x86
Windows 2000
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: timeless, Assigned: cls)

Details

Attachments

(1 file, 1 obsolete file)

MozillaTest had:
-rw-r--r--    1 nobody   nobody        117 Sep 21 10:00 notes.txt

egrep '^(User|Group) ' /etc/httpd/conf/httpd.conf
User apache
Group apache

Using addnote resulted in the normal your comment was added message. it should
have complained
Attached patch draft (obsolete) — Splinter Review
Comment on attachment 133928 [details] [diff] [review]
draft

can you test this?
Attachment #133928 - Flags: review?(justdave)
Comment on attachment 133928 [details] [diff] [review]
draft

doesn't compile.  Complains about || being overloaded.	Changing the test to:

$cant_print = 1 unless close NOTES;

Makes it work.

I still don't like the patch, but it works. :)	 I think it would be cleaner to
use a -w test before attempting to open it to begin with.
Attachment #133928 - Flags: review?(justdave) → review-
Attached patch v2.0Splinter Review
Attachment #133928 - Attachment is obsolete: true
Attachment #224273 - Flags: review?(bear)
Assignee: mcafee → cls
Comment on attachment 224273 [details] [diff] [review]
v2.0

I agree that this is not the best way to solve the issue.  Tinderbox2 writes each note to it's own file (named for the tree, build and time) and then html generation code pulls them in when it runs.
Attachment #224273 - Flags: review?(bear) → review+
Checking the error result of the open() is functionality equivalent to the -w test.  Using separate files for each comment is beyond the (simple) scope of the bug and should be an enhancement request.

Checking in webtools/tinderbox/addnote.cgi;
/cvsroot/mozilla/webtools/tinderbox/addnote.cgi,v  <--  addnote.cgi
new revision: 1.17; previous revision: 1.16
done
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Product: Webtools → Webtools Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: