Bug 1251921

Summary: Please include psl test data
Product: [Fedora] Fedora Reporter: Christopher Meng <i>
Component: publicsuffix-listAssignee: Yanko Kaneti <yaneti>
Status: CLOSED NEXTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: low Docs Contact:
Priority: unspecified    
Version: rawhideCC: yaneti
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: 20150831-1.fc23 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-09-23 04:13:21 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Christopher Meng 2015-08-10 10:02:28 UTC
Hi,

libpsl needs test_psl.txt from publicsuffix to test the publicsuffix( ;-) ), few days ago I encountered the problem that dated test data(*.cy->several new TLDs) failed the tests of libpsl.

Source: https://publicsuffix.org/list/test.txt

(Moved to http://mxr.mozilla.org/mozilla-central/source/netwerk/test/unit/data/test_psl.txt?raw=1

Or this one could be better(?):

https://raw.githubusercontent.com/publicsuffix/list/master/tests/test_psl.txt
)

You should put it underneath %{_datadir}/publicsuffix, but you may also create a test subfolder and put it inside.

Let me know when you finish this and I will start a new libpsl build. Thank you!

Comment 1 Yanko Kaneti 2015-08-10 11:15:13 UTC
(In reply to Christopher Meng from comment #0)
....
> Or this one could be better(?):
> 
> https://raw.githubusercontent.com/publicsuffix/list/master/tests/test_psl.txt
> )

I'd rather use
https://github.com/publicsuffix/list/raw/master/tests/test_psl.txt
which is the same as the above. 

Unfortunately currently it has a syntax error, the fix for which isn't yet merged.  https://github.com/publicsuffix/list/pull/21  . I'd rather wait for that to get merged then update tхe list and add the testс.

I hope its not a big problem to wait a little...

Comment 2 Fedora Update System 2015-09-01 08:00:53 UTC
publicsuffix-list-20150831-1.fc23 has been submitted as an update to Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2015-14732

Comment 3 Fedora Update System 2015-09-01 20:22:52 UTC
publicsuffix-list-20150831-1.fc23 has been pushed to the Fedora 23 testing repository. If problems still persist, please make note of it in this bug report.\nIf you want to test the update, you can install it with \n su -c 'yum --enablerepo=updates-testing update publicsuffix-list'. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-14732

Comment 4 Fedora Update System 2015-09-23 04:13:20 UTC
publicsuffix-list-20150831-1.fc23 has been pushed to the Fedora 23 stable repository. If problems still persist, please make note of it in this bug report.