Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix incorrect setup instructions in README #19

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rrrnld
Copy link

@rrrnld rrrnld commented Nov 9, 2017

There's a small typo that leads to an incorrect line in /etc/hosts. Instead of 127.0.1.1 it should be 127.0.0.1 that the cartodb.localhost domain points to.

@sverhoeven
Copy link
Owner

127.0.1.1 is valid for Debian/Ubuntu based operating systems.
Which OS are you using which made the resolving of cartodb.localhost not work?

@rrrnld
Copy link
Author

rrrnld commented Nov 11, 2017

I am using macOS 10.12.6 with Docker version 17.09.0-ce, build afdb6d4. Should I just add a note about the differences then?

@sverhoeven
Copy link
Owner

A note would be nice, we should be careful not to break the hosts file.

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.

2 participants