Last Comment Bug 757005 - Add .sx to PSL
: Add .sx to PSL
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: Networking: Domain Lists (show other bugs)
: unspecified
: All All
: -- normal (vote)
: mozilla16
Assigned To: Nobody; OK to take it and work on it
:
: Patrick McManus [:mcmanus]
Mentors:
http://ccnso.icann.org/announcements/...
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-05-21 04:26 PDT by Gervase Markham [:gerv]
Modified: 2012-10-26 13:13 PDT (History)
3 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
Patch from Samuel Giles (267 bytes, patch)
2012-05-21 04:26 PDT, Gervase Markham [:gerv]
no flags Details | Diff | Splinter Review
Updated patch including confirmation from registry (1.24 KB, patch)
2012-06-07 02:13 PDT, Simone Carletti
gerv: review-
Details | Diff | Splinter Review
Updated patch including confirmation from registry (717 bytes, patch)
2012-06-20 05:00 PDT, Simone Carletti
gerv: review+
Details | Diff | Splinter Review

Description Gervase Markham [:gerv] 2012-05-21 04:26:36 PDT
Created attachment 625605 [details] [diff] [review]
Patch from Samuel Giles

See http://ccnso.icann.org/announcements/announcement-11may11-en.htm . It's for Sint Maarten (the Dutch half of the island of Saint Martin).

Gerv
Comment 1 Simone Carletti 2012-06-07 02:13:08 PDT
Created attachment 630897 [details] [diff] [review]
Updated patch including confirmation from registry

Here's an updated patch.

I contacted the registry to get a confirmation. I also added the .gov.sx zone as reported by the registry.
Comment 2 Gervase Markham [:gerv] 2012-06-20 03:08:32 PDT
Comment on attachment 630897 [details] [diff] [review]
Updated patch including confirmation from registry

This patch also contains changes for .au.

Gerv
Comment 3 Simone Carletti 2012-06-20 03:33:40 PDT
(In reply to Gervase Markham [:gerv] from comment #2)
> Comment on attachment 630897 [details] [diff] [review]
> Updated patch including confirmation from registry
> 
> This patch also contains changes for .au.
> 
> Gerv

It seems that either the clone on my Mac OSX machine or the instructions here [1] causes some line-ending changes.
I'll update the patch.

[1] https://developer.mozilla.org/en/Mercurial_FAQ#How_can_I_generate_a_patch_for_somebody_else_to_check-in_for_me.3F
Comment 4 Simone Carletti 2012-06-20 05:00:01 PDT
Created attachment 634848 [details] [diff] [review]
Updated patch including confirmation from registry

I updated the patch to include only the changes to the .SX entry.

As as side note, Mercurial continues to show me the other changes, I'll need to investigate further what's going on.
Comment 5 Gervase Markham [:gerv] 2012-06-20 08:07:31 PDT
https://hg.mozilla.org/integration/mozilla-inbound/rev/2c330a313099

Gerv
Comment 6 Ed Morley [:emorley] 2012-06-21 04:07:45 PDT
https://hg.mozilla.org/mozilla-central/rev/2c330a313099
Comment 7 Peter Kasting 2012-10-26 13:13:20 PDT
Comment on attachment 625605 [details] [diff] [review]
Patch from Samuel Giles

Clearing obsolete flag

Note You need to log in before you can comment on or make changes to this bug.