Bug 1251921

Summary:	Please include psl test data
Product:	[Fedora] Fedora	Reporter:	Christopher Meng <i>
Component:	publicsuffix-list	Assignee:	Yanko Kaneti <yaneti>
Status:	CLOSED NEXTRELEASE	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	low	Docs Contact:
Priority:	unspecified
Version:	rawhide	CC:	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:
Embargoed:

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.