Issue 8889: test_support.transient_internet fails on Freebsd because socket has no attribute EAI_NODATA (original) (raw)

Created on 2010-06-03 18:50 by r.david.murray, last changed 2022-04-11 14:57 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
transient_internet.patch r.david.murray,2010-06-03 18:50 review
transient_internet.patch r.david.murray,2010-06-03 19:53 review
transient_internet.patch r.david.murray,2010-06-03 20:02 review
Messages (6)
msg106972 - (view) Author: R. David Murray (r.david.murray) * (Python committer) Date: 2010-06-03 18:50
test_support.transient_internet was modified to check for more errors, but one of them, socket.EAI_NODATA, is not supported by FreeBSD. The attached patch rewrites transient_internet to only test EAI error codes that exist, and also improves the skip message.
msg106975 - (view) Author: R. David Murray (r.david.murray) * (Python committer) Date: 2010-06-03 19:28
Modified patch based on feedback from Antoine Pitrou on #python-dev.
msg106978 - (view) Author: R. David Murray (r.david.murray) * (Python committer) Date: 2010-06-03 19:53
A somewhat more clever version.
msg106979 - (view) Author: R. David Murray (r.david.murray) * (Python committer) Date: 2010-06-03 20:02
Final version after more feedback.
msg106981 - (view) Author: R. David Murray (r.david.murray) * (Python committer) Date: 2010-06-03 20:21
Committed to trunk in r81678. I'll wait until the trunk buildbots show the test_ssl and test_urllib2net error clear before back/forward porting.
msg124712 - (view) Author: R. David Murray (r.david.murray) * (Python committer) Date: 2010-12-27 16:51
I never forward ported this, but it was fixed in a different way in python3 during a complete rewrite of transient_internet for other reasons.
History
Date User Action Args
2022-04-11 14:57:01 admin set github: 53135
2010-12-27 16:51:44 r.david.murray set status: open -> closedmessages: + resolution: fixedstage: commit review -> resolved
2010-06-03 20:21:11 r.david.murray set messages: + stage: commit review
2010-06-03 20:02:26 r.david.murray set files: + transient_internet.patchmessages: +
2010-06-03 19:53:23 r.david.murray set files: + transient_internet.patchmessages: +
2010-06-03 19:41:22 r.david.murray set files: - transient_internet.patch
2010-06-03 19:28:39 r.david.murray set files: + transient_internet.patchmessages: +
2010-06-03 18:50:58 r.david.murray set assignee: r.david.murray
2010-06-03 18:50:04 r.david.murray create