Skip to content

[5.1] Move copying .env.example to post-root-package-install #3398

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
Jun 2, 2015

Conversation

crynobone
Copy link
Member

post-create-project-cmd is executed only after post-install-cmd and this
could cause issue on the first installation (via composer
create-project) where environment is not prepared hence php artisan optimize would generate compiled.php.

Current Behaviour

  1. post-install-cmd execute php artisan optimize while .env doesn't exist (env=production & debug=false).
  2. post-create-project-cmd execute copying .env.example and generate key.

Expected Behaviour

  1. post-root-package-cmd copy .env.example to .env so we can have (env=local & debug=true).
  2. post-install-cmd execute php artisan optimize with correct .env.
  3. post-create-project-cmd generate key.

Refer to https://getcomposer.org/doc/articles/scripts.md#command-events

Signed-off-by: crynobone crynobone@gmail.com

post-create-project-cmd is executed only after post-install-cmd and this
could cause issue on the first installation (via composer
create-project) where environment is not prepared hence `php artisan
optimize` would generate `compiled.php`.

Signed-off-by: crynobone <crynobone@gmail.com>
@crynobone crynobone force-pushed the feature/root-install branch from 749dcef to 2ac993f Compare June 2, 2015 02:26
@crynobone
Copy link
Member Author

Rebased.

taylorotwell added a commit that referenced this pull request Jun 2, 2015
[5.1] Move copying .env.example to post-root-package-install
@taylorotwell taylorotwell merged commit 9293247 into laravel:develop Jun 2, 2015
@crynobone crynobone deleted the feature/root-install branch October 13, 2015 13:26
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