-
Notifications
You must be signed in to change notification settings - Fork 14
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
Errors when running composer install: Unable to add newly converted phar task_linux_amd64.tar #135
Comments
That seems odd that it's trying to interpret this as a phar. Oh, no it's not, we're just borrowing the phar extension's built in support for tar.gz files: https://github.com/Lullabot/drainpipe/blob/main/src/BinaryInstaller.php#L212 @beto-aveiga can you replicate this locally? I wonder if there's a bug causing this if statement to not trigger correctly: https://github.com/Lullabot/drainpipe/blob/main/src/BinaryInstaller.php#L200-L202 Somewhat related: #114 |
@deviantintegral I can't replicate it locally. I ran I tried this and didn't work either 🤷🏼 |
@deviantintegral: it fails when installing the binary for |
@beto-aveiga I've updated the task binary in the latest release, do you still have this issue? |
@deviantintegral @justafish I am seeing the aforementioned error while running composer install on the same project Luis has worked on. It seems this is a Lando only error, cause I am not able to reproduce this on my DDev setup. Here is a gist of the output |
@elvism-lullabot If you run composer install again (twice), does it complete the second time ok? I ask, because I ran into the error
and @mrdavidburns 's suggestion of running it twice worked for my local. |
Noting here that we think this issue will be resolved, when #172 is completed. |
Until #172 is completed, is there a workaround? |
@tess-ten7 reviews and testing of #207 would be appreciated. We've been pulling that branch into a single project for a while now, but knowing it's working well on a second (and independent one at that) would be great feedback. And, anything in the code too! |
@deviantintegral Trying that PR as a patch results in a similar failure, and doesn't solve the underlying issue:
|
@tess-ten7 you will have to instead point to the branch in composer.json, and then update the extra section to opt-in to global binaries. If you patch it in it comes in too late for the hooks to work. |
I'm trying to run functional tests.
When I run the "Build Project" step, on the first line that runs
composer install
, it fails.Do you know what is happening?
Thanks!
composer [install] failed, composer command failed: ComposeCmd failed to run 'COMPOSE_PROJECT_NAME=ddev-abc-exampleorg-website docker-compose -f /home/runner/work/abc-example.org-website/abc-example.org-website/.ddev/.ddev-docker-compose-full.yaml exec -w /var/www/html/drupal -T web composer install', action='[exec -w /var/www/html/drupal -T web composer install]', err='exit status 1', stdout='Gathering patches for root package.
In BinaryInstaller.php line 214:
Unable to add newly converted phar "/var/www/html/.ddev/.drainpipe-composer-cache/files/lullabot/drainpipe/bin/task/3.9.0/task_linux_amd64.tar" to the list of phars, a phar with that name already exists
Workflow code
The text was updated successfully, but these errors were encountered: