Closed Bug 1053067 Opened 5 years ago Closed 5 years ago

Correct misspelling "verion" in source code


(Core :: General, defect, trivial)

Not set





(Reporter: emorley, Assigned:



(Whiteboard: [good first bug])


(2 files, 1 obsolete file)

Broken out from bug 577266 just so we can close that bug out.

In quite a few places "version" is spelt as "verion":

Some of these places are in upstream projects (files under /security/nss/ and /media/webrtc/signaling/src/sipcc/), so will need fixing upstream separately.

Marking this a good first bug.
I want to work on this bug.
Go ahead :-)
Assignee: nobody →
code changes done will post for review soon
Corrected spelling, please review my first patch.
just to make sure, also did grep to find 'verion', and not result found except patch file
Thank you for the patch - some of the changes touch files that need to be fixed upstream (see comment 0), so are likely best separated out.

I'd suggest:
* Patch for the mach change
* Patch for the webrtc change (with review requested from :jesup, since I'm not sure which files are from upstream and so not supposed to be modified)
* Perhaps filing a separate bug for the NSS changes, since they will likely take a lot longer (and need to be landed in a separate repository), here:
Patch for the mach change
Attachment #8473666 - Attachment is obsolete: true
Patch for the webrtc change
Attachment #8474077 - Flags: review?(rjesup)
For NSS change new Bug created.
Also posted patch there.
Depends on: 1054625
Comment on attachment 8474076 [details] [diff] [review]

These look great - thank you for breaking the NSS one out :-)

:jesup will need to say whether the sipcc changes need to go upstream or not.
Attachment #8474076 - Flags: review+
@Ed Morley, Thanks for reviewing patch. May I know what is next step ? should I checkin the mach_version_spelling_correction.patch or wait till review of webrtc_version_spelling_correction.patch and checkin both together?
Comment on attachment 8474077 [details] [diff] [review]

Review of attachment 8474077 [details] [diff] [review]:

r+; and there's no upstreaming of suipcc fixes so this can just land
Attachment #8474077 - Flags: review?(rjesup) → review+
Whiteboard: [good first bug] → [good first bug] [checkin-needed]
The two patches here have been landed on mozilla-inbound, and will be merged to mozilla-central later today (at which point this bug will be marked resolved fixed).

Thank you for the patches! :-)
Whiteboard: [good first bug] [checkin-needed] → [good first bug]
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla34
Flags: qe-verify-
You need to log in before you can comment on or make changes to this bug.