Closed Bug 142792 Opened 21 years ago Closed 21 years ago

Output from checksetup.pl contains misspelled words

Categories

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

2.15
defect

Tracking

()

RESOLVED FIXED
Bugzilla 2.16

People

(Reporter: ddkilzer, Assigned: ddkilzer)

Details

Attachments

(1 file)

The patch for Bug 140355 added some output that the user will
see under certain circumstances.  It contains a couple
misspellings.

Also changed "Perl" to "Perl code" and "as the user" to "as
the user that".

Patch to follow shortly.
Status: NEW → ASSIGNED
Keywords: patch, review
Priority: -- → P1
Target Milestone: --- → Bugzilla 2.16
Attached patch Patch v.1Splinter Review
Fixes misspellings of "databasa" and "arbitary".
Adds "code" and "that" to make sentence clearer.
Comment on attachment 82649 [details] [diff] [review]
Patch v.1

2xr=gerv.

Gerv
Attachment #82649 - Flags: review+
Fixed.

Checking in checksetup.pl;
/cvsroot/mozilla/webtools/bugzilla/checksetup.pl,v  <--  checksetup.pl
new revision: 1.144; previous revision: 1.143
done

Gerv
Status: ASSIGNED → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
It strikes me misspellings like these would be a good thing to add to the
testing suite.
AU or US spelling? :) Besides spell checkers don't like technical stuff, even if we did just check comments, we'd still be mentioning variable names and the like.
AU spelling of course.

I didn't mention hooking up a spell checker, just looking for specific
misspellings.

I don't see a problem with checking variable names.
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.