-
-
Notifications
You must be signed in to change notification settings - Fork 74
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 lookup path for installed.json
#113
Fix incorrect lookup path for installed.json
#113
Conversation
Hey @arondeparon, please also backport the tests :S My suggestion is to use |
Will do! I did not see the backport yet and wrote the exact same fix as you already did 😉 |
…son` Due to `getcwd()` returning an empty string in some environments, (or even `"."`, a dot, sometimes), this fallback versions logic was too fragile. Adding hardcoded paths that are supposed to be constant (relative to the installation path of `FallbackVersions.php`) gives us more stability.
Otherwise, composer cannot determine the current branch-alias, and fails early during installation.
@arondeparon about 03e5174, I think |
@Ocramius yeah I was not entirely sure about how you would like to see it, but noticed that I'll change the test to be a bit more flexible 👌 |
Tests passing now 😃 |
LGTM! 👍 |
installed.json
This fixes the issue that cause Laravel installations with a dependency on
1.4.x
to break that are installed usingcomposer install --no-scripts
as described in #112