Closed Bug 985507 Opened 10 years ago Closed 10 years ago

please remove trailing whitespace in effective_tld_names.dat

Categories

(Core Graveyard :: Networking: Domain Lists, defect)

x86_64
Linux
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla31

People

(Reporter: dkg, Unassigned)

Details

Attachments

(1 file)

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:27.0) Gecko/20100101 Firefox/27.0 Iceweasel/27.0 (Beta/Release)
Build ID: 20140205132948

Steps to reproduce:

i noticed effective tld_names.dat has trailing whitespace on some lines and none on others.  To be nicer to machine parsers (who may not realize that ".it " is supposed to be ".it", the trailing whitespace should be removed.  The attached patch should resolve things, or you can use GNU sed like this:

sed -i -r 's/\s+$//' effective_tld_names.dat
This Bug has been repaired:

$ date
Mon May  5 15:41:05 MDT 2014
$ wget -q https://publicsuffix.org/list/effective_tld_names.dat
$ egrep '^\w.*\s$' effective_tld_names.dat || echo Bug 985507 is fixed
Bug 985507 is fixed
$

It is safe to close now. Thank you very much!
Flags: needinfo?(dkg)
Status: UNCONFIRMED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla31
Flags: needinfo?(dkg)
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: