Skip to content
This repository has been archived by the owner on Feb 13, 2023. It is now read-only.

Zip & unzip not installed #550

Closed
heddn opened this issue Apr 7, 2016 · 3 comments
Closed

Zip & unzip not installed #550

heddn opened this issue Apr 7, 2016 · 3 comments
Labels

Comments

@heddn
Copy link
Contributor

heddn commented Apr 7, 2016

Zip & unzip are not installed as part of the base box. If I add them to extra_packages, that doesn't seem to run until after a bunch of composer stuffs. On my slow internet connection, that means vagrant provision always fails. Why should the absence of zip/unzip matter? Composer cannot use prefer-dist if those aren't available, forcing it to do a git clone. That pushes me over the 300 second limit and the composer install fails, killing the provision.

Solution: add zip/unzip as part of the base box so they are just there.

@oxyc
Copy link
Collaborator

oxyc commented Apr 7, 2016

I don't think @geerlingguy has added anything extra to the boxes. Everything is done through ansible.

For a quick hacky solution to get this fixed right away you could do (edit: assuming you're using ubuntu box):

php_packages:
  - php5
  - php5-mcrypt
  - php5-cli
  - php5-common
  - php5-curl
  - php5-dev
  - php5-fpm
  - php5-gd
  - php-pear
  - php-apc
  - libpcre3-dev
  - unzip

This would install it before the composer task runs.

As for a more stable solution. We can't really bump the extra package install task further up as it's common to use this for "addon" packages that depend on the roles that run before. Maybe we just add it as a dependency in the init pre_tasks https://github.com/geerlingguy/drupal-vm/blob/master/provisioning/tasks/init-debian.yml.

oxyc added a commit to oxyc/drupal-vm that referenced this issue Apr 7, 2016
@geerlingguy
Copy link
Owner

Yeah, I'm thinking we could just do it in pre_tasks for debian (and RHEL too, if it's not available there).

geerlingguy added a commit that referenced this issue Apr 7, 2016
Issue #550: Make unzip a dependency
@geerlingguy
Copy link
Owner

Fixed by #551.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

3 participants