Skip to content

Added a note about virtual machine for local build #11115

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

Merged
merged 1 commit into from
Mar 11, 2019

Conversation

HeahDude
Copy link
Contributor

@HeahDude HeahDude commented Mar 8, 2019

No description provided.

@HeahDude HeahDude added this to the 3.4 milestone Mar 8, 2019
@@ -33,3 +33,6 @@ $ docker build . -t symfony-docs
# (if it's already in use, change the '8080' port by any other port)
$ docker run --rm -p 8080:80 symfony-docs
```

If you use a virtual machine use its ip instead of the localhost to access the
server (i.e `http://192.168.99.100:8080`).
Copy link
Member

Choose a reason for hiding this comment

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

What would be the real full command to execute in this case? This one?

docker run --rm -p http://192.168.99.100:8080:80 symfony-docs

Copy link
Contributor Author

@HeahDude HeahDude Mar 11, 2019

Choose a reason for hiding this comment

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

Yes, do you prefer that I inline it all with both the comment below the other one? I hesitated.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No sorry the command is the same, just the access is different.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The real diff when using a VM is:

- http://127.0.0.1:8080
+ http://IP_VM:8080

@javiereguiluz
Copy link
Member

Thanks Jules.

@javiereguiluz javiereguiluz merged commit 091bb22 into symfony:3.4 Mar 11, 2019
javiereguiluz added a commit that referenced this pull request Mar 11, 2019
…s Pietri)

This PR was merged into the 3.4 branch.

Discussion
----------

Added a note about virtual machine for local build

<!--

If your pull request fixes a BUG, use the oldest maintained branch that contains
the bug (see https://symfony.com/roadmap for the list of maintained branches).

If your pull request documents a NEW FEATURE, use the same Symfony branch where
the feature was introduced (and `master` for features of unreleased versions).

-->

Commits
-------

091bb22 Added a note about virtual machine for local build
@HeahDude HeahDude deleted the fix-docker branch March 11, 2019 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants