Closed
Bug 1053068
Opened 10 years ago
Closed 7 years ago
Correct misspelling "compatab{le,ility}" in source code
Categories
(Core :: General, defect)
Core
General
Tracking
()
RESOLVED
INCOMPLETE
People
(Reporter: emorley, Unassigned)
References
Details
(Whiteboard: [good first bug])
Broken out from bug 577266 just so we can close that bug out. In quite a few places "compatible/compatibility" is spelt as "compatable/compatability": http://mxr.mozilla.org/mozilla-central/search?string=compatab Some of these places are in upstream projects (eg files under /security/nss/, /media/webrtc/trunk/tools/gyp/ and /nsprpub/), so will need fixing upstream separately. Marking this a good first bug.
Updated•10 years ago
|
Assignee: nobody → bkerensa
Comment 1•10 years ago
|
||
(In reply to Ed Morley [:edmorley] from comment #0) > Broken out from bug 577266 just so we can close that bug out. > > In quite a few places "compatible/compatibility" is spelt as > "compatable/compatability": > http://mxr.mozilla.org/mozilla-central/search?string=compatab > > Some of these places are in upstream projects (eg files under > /security/nss/, /media/webrtc/trunk/tools/gyp/ and /nsprpub/), so will need > fixing upstream separately. > > Marking this a good first bug. I will take this... Do you want these as individual patches for each instance?
Updated•10 years ago
|
Flags: needinfo?(emorley)
Reporter | ||
Comment 2•10 years ago
|
||
(In reply to Benjamin Kerensa [:bkerensa] from comment #1) > I will take this... Do you want these as individual patches for each > instance? For each instance that is in something upstream, we'll need a separate patch. Ways to tell this are by looking at the file header, as well as glancing at the hg log to see if there are "import latest foo from upstream" type commits. Everything else can go into one patch, which I'll rubberstamp - apart from any non comment changes (eg those in mobile/android/app/mobile.js), which would be good in another patch.
Flags: needinfo?(emorley)
Reporter | ||
Updated•10 years ago
|
Status: NEW → ASSIGNED
Reporter | ||
Comment 3•9 years ago
|
||
No activity; resetting assignee.
Assignee: bkerensa → nobody
Status: ASSIGNED → NEW
Comment 4•9 years ago
|
||
(In reply to Ed Morley [:emorley] from comment #2) > (In reply to Benjamin Kerensa [:bkerensa] from comment #1) > > I will take this... Do you want these as individual patches for each > > instance? > > For each instance that is in something upstream, we'll need a separate > patch. Ways to tell this are by looking at the file header, as well as > glancing at the hg log to see if there are "import latest foo from upstream" > type commits. > > Everything else can go into one patch, which I'll rubberstamp - apart from > any non comment changes (eg those in mobile/android/app/mobile.js), which > would be good in another patch. /security/nss/lib/dbm/include/hash.h was mentioned as being upstream, so I checked its header and hg log, but I don't see any comments related to upstream. Are there any more obvious/specific ways to tell a file is upstream?
Hey, I want to work on this for my first bug. How can I tell which files are upstream so that I can create a separate patch for them? You mentioned files under /security/nss/, /media/webrtc/trunk/tools/gyp/ and /nsprpub/ are upstream and their hg logs will contain "import latest foo from upstream" type commits. Unfortunately I can't find any comments related to upstream.
Flags: needinfo?(emorley)
Reporter | ||
Comment 6•9 years ago
|
||
Unfortunately there isn't a clear cut answer for all cases. I'd use your best guess (looking at hg log, file headers, checking for readmes/separate build files in the parent directory etc) to exclude the obvious upstream cases, attach a patch and then iterate from there :-)
Flags: needinfo?(emorley)
Hi Ed, This is what I found that contains the string "compatab", https://dxr.mozilla.org/mozilla-central/search?q=compatab I see one /nsprpub/ file, one /security/nss/lib and mobile/android/app/mobile.js. Should I make each of these into three separate patches, and the rest into a single patch? Also, what is the reason behind "netwerk" directory in mozilla-central, is that also a typo? https://dxr.mozilla.org/mozilla-central/source/netwerk Thanks.
Flags: needinfo?(emorley)
Reporter | ||
Comment 9•8 years ago
|
||
I no longer work with mozilla-central so I'm not the best person to ask. However at first glance at the DXR results, the following are all upstream so shouldn't be touched: gfx/cairo/cairo/src/* gfx/skia/skia/* ipc/chromium/src/* media/libvorbis/* netwerk/sctp/* nsprpub/* obj-x86_64-pc-linux-gnu/_virtualenv/* python/mock-1.0.0/* security/nss/* toolkit/crashreporter/google-breakpad/ (albeit I think we've forked this one now; ask :ted) ...and anything else is likely fair game.
Flags: needinfo?(emorley)
Reporter | ||
Comment 10•7 years ago
|
||
Mass-closing old bugs I filed that have not had recent activity/no longer affect me.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → INCOMPLETE
You need to log in
before you can comment on or make changes to this bug.
Description
•