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

Make compilation faster in debug mode and added new compilation options #142

Merged
merged 1 commit into from
Apr 22, 2018

Conversation

theofidry
Copy link
Member

@theofidry theofidry commented Apr 22, 2018

  • Instead of dumping the files as they are added to the PHAR in debug mode, the code of the PHAR is
    simply extracted at the end of the compilation process to the debug directory
  • A new --no-parallel option has been added to the compile command to not process the files in
    parallel instead of doing so only in debug mode
  • A new --no-restart option has been added to the compile command to not restart the PHP process
    allow more easily to keep xdebug enabled
  • The verbosity is now forced to debug earlier when the debug mode is enabled. This ensures the new
    verbosity set applies to the process restart check

- Instead of dumping the files as they are added to the PHAR in debug mode, the code of the PHAR is
  simply extracted at the end of the compilation process to the debug directory
- A new `--no-parallel` option has been added to the `compile` command to not process the files in
  parallel instead of doing so only in debug mode
- A new `--no-restart` option has been added to the `compile` command to not restart the PHP process
  allow more easily to keep xdebug enabled
- The verbosity is now forced to debug ealier when the debug mode is enabled. This ensures the new
  verbosity set applies to the process restart check
@theofidry theofidry merged commit 8d22c82 into box-project:master Apr 22, 2018
@theofidry theofidry deleted the feature/fast-debug-mode branch April 22, 2018 13:36
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.

1 participant