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

Exclude autoload_static.php #23935

Merged
merged 1 commit into from
Apr 12, 2016
Merged

Exclude autoload_static.php #23935

merged 1 commit into from
Apr 12, 2016

Conversation

LukasReschke
Copy link
Member

Composer 1.1 has since yesterday a new performance improvement that will be automatically used for PHP >= 5.6, however this file is incompatible with older PHP versions and thus we need to exclude it from the checks.

Note that this performance improvement is only used on >= 5.6 so ownCloud will still run fine on older PHP versions as well.

@DeepDiver1975

Composer 1.1 has since yesterday a new performance improvement that will be automatically used for PHP >= 5.6, however this file is incompatible with older PHP versions and thus we need to exclude it from the checks.

Note that this performance improvement is only used on >= 5.6 so ownCloud will still run fine on older PHP versions as well.
@mention-bot
Copy link

By analyzing the blame information on this pull request, we identified @DeepDiver1975 and @nickvergessen to be potential reviewers

@DeepDiver1975
Copy link
Member

Nice - THX

@nickvergessen
Copy link
Contributor

Exclude from https://github.com/owncloud/core/tree/master/apps/files_external/3rdparty/composer as well, in case it ever gets an update?

@DeepDiver1975
Copy link
Member

Travis is green - let's merge - I want my PRs succeed 👍

@DeepDiver1975 DeepDiver1975 merged commit 276b8a5 into master Apr 12, 2016
@DeepDiver1975 DeepDiver1975 deleted the exclude-autoload-static branch April 12, 2016 10:07
@DeepDiver1975
Copy link
Member

Just seen on stable9 pr - we need to backport this

@DeepDiver1975 DeepDiver1975 self-assigned this Apr 12, 2016
DeepDiver1975 pushed a commit that referenced this pull request Apr 12, 2016
Composer 1.1 has since yesterday a new performance improvement that will be automatically used for PHP >= 5.6, however this file is incompatible with older PHP versions and thus we need to exclude it from the checks.

Note that this performance improvement is only used on >= 5.6 so ownCloud will still run fine on older PHP versions as well.
DeepDiver1975 added a commit that referenced this pull request Apr 12, 2016
@MorrisJobke
Copy link
Contributor

@DeepDiver1975 Could we backport this to stable8.2? #24008 fails for the same reason.

@lock
Copy link

lock bot commented Aug 6, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Aug 6, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants