Closed
Bug 658084
Opened 13 years ago
Closed 13 years ago
Add IDN ccTLDs + XXX to PSL
Categories
(Core Graveyard :: Networking: Domain Lists, defect)
Core Graveyard
Networking: Domain Lists
Tracking
(status2.0 .x-fixed, status1.9.2 .18-fixed, status1.9.1 .20-fixed)
RESOLVED
FIXED
People
(Reporter: jothan, Assigned: gerv)
References
Details
(Keywords: verified1.9.2)
Attachments
(1 file, 3 obsolete files)
3.77 KB,
patch
|
dveditz
:
approval2.0+
dveditz
:
approval1.9.2.18+
dveditz
:
approval1.9.1.20+
|
Details | Diff | Splinter Review |
This is a large patch which incorporates Bug 650412 to add .XXX Bug 650412 xxx ICM Registry Sponsored Community http://www.icmregistry.com It also incorporates Bug 607938, plus additional IDN ccTLD to sync with the IDN Fastrack program at ICANN (URL: https://www.icann.org/en/topics/idn/fast-track/string-evaluation-completion-en.htm) Bug 607938 Algeria الجزائر xn--lgbbat1ad8j Algeria / Al Jazair Arabic Arabic India بھارت xn--mgbbh1a71e Bharat Urdu Arabic भारत xn--h2brj9c Bharat Hindi Devanagari ভারত xn--45brj9c Bharat Bengali Bengali ਭਾਰਤ xn--s9brj9c Bharat Punjabi Gurmukhi ભારત xn--gecrj9c Bharat Gujarati Gujarati இந்தியா xn--xkc2dl3a5ee0h India Tamil Tamil భారత్ xn--fpcrj9c3d Bharat Telugu Telugu Korea, Republic of 한국 xn--3e0b707e Republic of Korea Korean Hangul Morocco المغرب xn--mgbc0a9azcg Morocco / al-Maghrib Arabic Arabic Oman عمان xn--mgb9awbf Oman Arabic Arabic Serbia срб xn--90a3ac srb Serbian Cyrillic New IDN ccTLD Strings Bangladesh বাংলা xn--54b7fta0cc Bangla Bangla Bangla Georgia გე xn--node ge Georgian Georgian (Mkhedruli) Iran, Islamic Republic of ايران xn--mgba3a4fra Iran Iran Arabic ایران xn--mgba3a4f16a Iran Iran Persian Pakistan پاكستان xn--mgbai9a5eva00b Pakistan Urdu Arabic پاکستان xn--mgbai9azgqp6j Pakistan Urdu Arabic Saudi Arabia السعوديه xn--mgbqly7cvafr AlSaudiah Arabic Arabic (Variant) السعودیة xn--mgberp4a5d4a87g AlSaudiah Arabic Arabic (Variant) السعودیۃ xn--mgbqly7c0a67fbc AlSaudiah Arabic Arabic (Variant) Syrian Arab Republic سوريا xn--mgbtf8fl Syria Arabic Arabic سورية xn--ogbpf8fl Syria Arabic Arabic Taiwan 臺灣 xn--nnx388a Taiwan Chinese Simplified Chinese Ukraine укр xn--j1amh ukr Ukrainian Cyrillic Yemen اليمن xn--mgb2ddes AlYemen Arabic Arabic
Assignee | ||
Updated•13 years ago
|
Attachment #533439 -
Attachment is patch: true
Attachment #533439 -
Attachment mime type: text/x-patch → text/plain
Assignee | ||
Comment 3•13 years ago
|
||
Comment on attachment 533439 [details] [diff] [review] PSL Add TLDs for Algeria, Bangladesh, Georgia, India, Iran, Korea, Morocco, Oman, Pakistan, Saudi Arabia, Serbia, Syria, Taiwan, Ukraine, Yemen, XXX r=gerv. Although all of these are no-ops, right? Gerv
Attachment #533439 -
Flags: review+
Assignee | ||
Comment 4•13 years ago
|
||
mozilla-central: http://hg.mozilla.org/mozilla-central/rev/b59363737df0 I will ask for branch approvals in a day or two. Gerv
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Comment 5•13 years ago
|
||
Backed out due to xpcshell failure on all platforms. See: http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1305900628.1305901763.25019.gz&fulltext=1
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 6•13 years ago
|
||
The cause of the failure was this bug; the new TLDs for .pk contain what Firefox things is a non-permitted codepoint (200E). And the Verisign IDN checker agrees: http://mct.verisign-grs.com/conversiontool/convertServlet?input=%E2%80%8E%E2%80%8E%D9%BE%D8%A7%D9%83%D8%B3%D8%AA%D8%A7%D9%86&type=UTF8 I will check this all back in without that change, and we can open another bug to fix the problem. Gerv
Assignee | ||
Comment 7•13 years ago
|
||
Attempt 2: http://hg.mozilla.org/mozilla-central/rev/e961f83ce28c Gerv
Status: REOPENED → RESOLVED
Closed: 13 years ago → 13 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•13 years ago
|
Attachment #533439 -
Flags: approval2.0?
Attachment #533439 -
Flags: approval1.9.2.18?
Attachment #533439 -
Flags: approval1.9.1.20?
Comment 8•13 years ago
|
||
Comment on attachment 533439 [details] [diff] [review] PSL Add TLDs for Algeria, Bangladesh, Georgia, India, Iran, Korea, Morocco, Oman, Pakistan, Saudi Arabia, Serbia, Syria, Taiwan, Ukraine, Yemen, XXX This is the patch that got bounced, right? Not approving. Actually what you checked in the second time looks like what you checked in the first time -- what happened to the test failures? Did \u200E magically become approved in that time? (it's invisible and in the 'punctuation' block so it seems unlikely).
Attachment #533439 -
Flags: approval2.0?
Attachment #533439 -
Flags: approval2.0-
Attachment #533439 -
Flags: approval1.9.2.18?
Attachment #533439 -
Flags: approval1.9.2.18-
Attachment #533439 -
Flags: approval1.9.1.20?
Attachment #533439 -
Flags: approval1.9.1.20-
Reporter | ||
Comment 9•13 years ago
|
||
This is the two lines, fixed to not include the offending character.
Reporter | ||
Comment 10•13 years ago
|
||
We had two illegal unicode characters that I found in the <.pk> entries, that bled through in a cut-paste operation at my desk. The unicode characters caused a fail in build. I determined the source of the error, and have fixed it in the attached file. This is a replacement for the patch that was failing, I was able to determine that there was an illegal character in the cut-paste of the Arabic and Persian strings for the Pakistan entries that was causing the fail. The newly attached has validated each way in a round trip conversion test [punicode->native->punicode], [native->punicode->native], and is confirmed valid.
Attachment #533439 -
Attachment is obsolete: true
Attachment #534476 -
Attachment is obsolete: true
Reporter | ||
Updated•13 years ago
|
Assignee: nobody → gerv
Assignee | ||
Comment 11•13 years ago
|
||
Here is a patch containing the changes which actually ended up on mozilla-central, after backing out the .pk entries (which are now bug 658582). We should handle these changes here, and the other ones in the other bug. Gerv
Attachment #534492 -
Attachment is obsolete: true
Assignee | ||
Updated•13 years ago
|
Attachment #534724 -
Flags: approval2.0?
Attachment #534724 -
Flags: approval1.9.2.18?
Attachment #534724 -
Flags: approval1.9.1.20?
Comment 12•13 years ago
|
||
Comment on attachment 534724 [details] [diff] [review] Patch v.3 Approved for 1.9.2.18 and 1.9.1.20, a=dveditz for release-drivers Approved for the mozilla2.0 repository, a=dveditz for release-drivers
Attachment #534724 -
Flags: approval2.0?
Attachment #534724 -
Flags: approval2.0+
Attachment #534724 -
Flags: approval1.9.2.18?
Attachment #534724 -
Flags: approval1.9.2.18+
Attachment #534724 -
Flags: approval1.9.1.20?
Attachment #534724 -
Flags: approval1.9.1.20+
Assignee | ||
Comment 13•13 years ago
|
||
http://hg.mozilla.org/releases/mozilla-2.0/rev/a255c89497de http://hg.mozilla.org/releases/mozilla-1.9.2/rev/7c458d3f60ae http://hg.mozilla.org/releases/mozilla-1.9.1/rev/30535aa1e0a1 Gerv
Updated•2 months ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•