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

[Bundle][FrameworkBundle] be smarter when guessing the document root #12525

Merged
merged 1 commit into from Nov 20, 2014

Conversation

xabbuh
Copy link
Member

@xabbuh xabbuh commented Nov 20, 2014

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets #12524
License MIT
Doc PR

@xabbuh
Copy link
Member Author

xabbuh commented Nov 20, 2014

This change makes it easy to start the built-in web server even when the user is not the in project directory. However, this may affect people how changed the kernel's root directory, but who kept web/ being the document root.

@javiereguiluz
Copy link
Member

Thank you @xabbuh! It worked like a charm for me.

Regarding the potential issue that you mentioned, in my opinion is not that important because people changing the kernel's root directory are advanced enough to:

  • Not use the built-in web server
  • Know what to do if this error happens

@stof
Copy link
Member

stof commented Nov 20, 2014

👍

@fabpot
Copy link
Member

fabpot commented Nov 20, 2014

Thank you @xabbuh.

@fabpot fabpot merged commit e28f5b8 into symfony:2.3 Nov 20, 2014
fabpot added a commit that referenced this pull request Nov 20, 2014
…ument root (xabbuh)

This PR was merged into the 2.3 branch.

Discussion
----------

[Bundle][FrameworkBundle] be smarter when guessing the document root

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #12524
| License       | MIT
| Doc PR        |

Commits
-------

e28f5b8 be smarter when guessing the document root
@xabbuh xabbuh deleted the smarter-doc-root-detection branch November 20, 2014 12:15
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.

None yet

4 participants