Skip to content

Conversation

@huntc
Copy link
Contributor

@huntc huntc commented Jun 15, 2017

Less scary.

@huntc
Copy link
Contributor Author

huntc commented Jun 15, 2017

Sample output:

huntc$ sandbox run 2.1.0 --no-default-features
Error: 
	Unfortunately, because of a known Java 8 issue on macOS,
	hostname lookups on your machine will take more than 5 seconds.

	This will prevent ConductR from starting.

	Fortunately, there is an easy, quick fix. Just add 'ChristohersMBP2.lan' to your /etc/hosts file. e.g.:

		127.0.0.1   localhost ChristohersMBP2.lan
		::1         localhost ChristohersMBP2.lan

	Note that you will need to use sudo to edit the /etc/hosts file.
	Use your favorite editor to do this e.g. 'sudo vi /etc/hosts'

	To learn more about the Java 8 issue: http://stackoverflow.com/questions/39636792/jvm-takes-a-long-time-to-resolve-ip-address-for-localhost

huntc$ 

Copy link
Contributor

@fsat fsat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code change itself lgtm - but I think we should continue asserting the output of the message.

|Error: ::1 localhost MagicBox
|"""))
self.assertEqual(self.output(stderr), expected_output)
self.assertTrue(len(self.output(stderr)) > 0)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should continue asserting the message, especially since there is part of the message where the socket.gethostname is being inserted in the message.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I disagree - it is quite painful to maintain such tests...

@huntc huntc force-pushed the jvm-error-report branch from d55b349 to 2eda614 Compare June 15, 2017 00:46
@huntc huntc force-pushed the jvm-error-report branch from 2eda614 to b1c7301 Compare June 15, 2017 00:57
@huntc huntc merged commit aa9733d into typesafehub:master Jun 15, 2017
@huntc huntc deleted the jvm-error-report branch June 15, 2017 01:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants