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)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: timeless, Assigned: cls)
Details
Attachments
(1 file, 1 obsolete file)
1.19 KB,
patch
|
bear
:
review+
|
Details | Diff | Splinter Review |
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
Comment on attachment 133928 [details] [diff] [review] draft can you test this?
Attachment #133928 -
Flags: review?(justdave)
Comment 3•21 years ago
|
||
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-
Attachment #133928 -
Attachment is obsolete: true
Attachment #224273 -
Flags: review?(bear)
Updated•18 years ago
|
Assignee: mcafee → cls
Comment 5•18 years ago
|
||
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
Updated•10 years ago
|
Product: Webtools → Webtools Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•