Skip to content

Fix the CI on Appveyor #654

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
Sep 20, 2017
Merged

Fix the CI on Appveyor #654

merged 1 commit into from
Sep 20, 2017

Conversation

stof
Copy link
Member

@stof stof commented Sep 20, 2017

On windows, composer does not use a symlink when installing binaries but creates a proxy script instead. This proxy is not a PHP script, so running it through PHP does not work.
The goal of composer is to allow running the script in a way similar to the Linux usage relying on shebangs.

So tests were not running on AppVeyor: https://ci.appveyor.com/project/fabpot/symfony-demo/build/1.0.1088#L140

On windows, composer does not use a symlink when installing binaries but
creates a proxy script instead. This proxy is not a PHP script, so running it
through PHP does not work.
The goal of composer is to allow running the script in a way similar to the
Linux usage relying on shebangs.
@javiereguiluz
Copy link
Member

Thanks for fixing this issue!

@javiereguiluz javiereguiluz merged commit 8434ba5 into symfony:master Sep 20, 2017
javiereguiluz added a commit that referenced this pull request Sep 20, 2017
This PR was merged into the master branch.

Discussion
----------

Fix the CI on Appveyor

On windows, composer does not use a symlink when installing binaries but creates a proxy script instead. This proxy is not a PHP script, so running it through PHP does not work.
The goal of composer is to allow running the script in a way similar to the Linux usage relying on shebangs.

So tests were not running on AppVeyor: https://ci.appveyor.com/project/fabpot/symfony-demo/build/1.0.1088#L140

Commits
-------

8434ba5 Fix the CI on Appveyor
@stof stof deleted the fix_appveyor branch September 20, 2017 17:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants