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

Some setup:di:compile errors not reported #41

Closed
erikhansen opened this issue Nov 10, 2016 · 1 comment
Closed

Some setup:di:compile errors not reported #41

erikhansen opened this issue Nov 10, 2016 · 1 comment

Comments

@erikhansen
Copy link
Contributor

This issue is a similar problem to the one reported in #25. It is also somewhat related to #35.

Here is an example screenshot demonstrating the problem:
17-27-55 new message-cjlba

I'm using Magento EE 2.1.2. Errors generated during setup:di:compile are not being reported when deploying. The errors are essentially being ignored and the deployment continues as if nothing happened.

I think the problem is that we assumed that Magento 2.1.x fixed the issue where setup:di:compile and setup:static-content:deploy were not returning non-zero exit codes upon failure, but this is not always the case. While the magento/magento2#3189 PR did fix the return code issue in many situations, it looks there are certain "soft" errors that don't result in error codes: https://github.com/magento/magento2/blob/1ade3b769a937f19682bbbe4e51b6c956147952f/setup/src/Magento/Setup/Module/Di/Compiler/Log/Writer/Console.php#L72

davidalger pushed a commit that referenced this issue Nov 11, 2016
…etection

Catch all di:compile errors on 2.1.x - fixes #41
@davidalger
Copy link
Owner

Released in 0.5.6.

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

No branches or pull requests

2 participants