Closed Bug 714903 Opened 10 years ago Closed 10 years ago

getLanIp in devicemanager.py throws an exception on Mac OS X

Categories

(Testing :: Mochitest, defect)

x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla12

People

(Reporter: kats, Assigned: kats)

References

Details

Attachments

(1 file)

I'm trying to get the mochitests running on my machine locally, and the first roadblock I hit was that getLanIp was throwing a socket.gaierror exception. According to http://stackoverflow.com/a/7842004 this can be fixed by appending a ".local" to the hostname, and I have verified this works for me locally. Patch attached.

Not sure who should review this patch but sending request to jmaher to start; feel free to tag somebody else with the request.
Attachment #585492 - Flags: review?(jmaher)
Comment on attachment 585492 [details] [diff] [review]
Try with .local if hostname throws gaierror

Review of attachment 585492 [details] [diff] [review]:
-----------------------------------------------------------------

this seems like a pretty good fix.  Thanks for finding and fixing this.
Attachment #585492 - Flags: review?(jmaher) → review+
https://hg.mozilla.org/mozilla-central/rev/cc4c6c59c516
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Target Milestone: --- → mozilla12
You need to log in before you can comment on or make changes to this bug.