Chassis / v8js

A Chassis extension to install V8JS
0 stars 2 forks source link

Dependency issues #25

Closed stuartshields closed 5 years ago

stuartshields commented 5 years ago

PHP: 7.2

Currently when provisioning the box we get the following errors:

[vagrant] ==> default: Notice: /Stage[main]/Apt::Update/Exec[apt_update]: Dependency Exec[add-apt-repository-ppa:pinepain/libv8] has failures: true
[vagrant] ==> default: Error: /usr/bin/add-apt-repository -y ppa:pinepain/libv8 returned 1 instead of one of [0]
[vagrant] ==> default: Error: /Stage[main]/V8js::Extension/Apt::Ppa[ppa:pinepain/libv8]/Exec[add-apt-repository-ppa:pinepain/libv8]/returns: change from notrun to 0 failed: /usr/bin/add-apt-repository -y ppa:pinepain/libv8 returned 1 instead of one of [0]
[vagrant] ==> default: Warning: /Stage[main]/Apt::Update/Exec[apt_update]: Skipping because of failed dependencies
[vagrant] ==> default: Notice: /Stage[main]/Apt::Update/Exec[apt_update]: Triggered 'refresh' from 1 events
[vagrant] ==> default: Notice: /Stage[main]/V8js::Extension/Apt::Ppa[ppa:pinepain/libv8]/File[/etc/apt/sources.list.d/pinepain-ubuntu-libv8-xenial.list]: Dependency Exec[add-apt-repository-ppa:pinepain/libv8] has failures: true
[vagrant] ==> default: Warning: /Stage[main]/V8js::Extension/Apt::Ppa[ppa:pinepain/libv8]/File[/etc/apt/sources.list.d/pinepain-ubuntu-libv8-xenial.list]: Skipping because of failed dependencies
[vagrant] ==> default: Notice: /Package[libv8-6.6-dev]: Dependency Exec[add-apt-repository-ppa:pinepain/libv8] has failures: true
[vagrant] ==> default: Warning: /Package[libv8-6.6-dev]: Skipping because of failed dependencies
[vagrant] ==> default: Notice: /Package[libv8-6.6]: Dependency Exec[add-apt-repository-ppa:pinepain/libv8] has failures: true
[vagrant] ==> default: Warning: /Package[libv8-6.6]: Skipping because of failed dependencies

short term solution is located here: https://gist.github.com/acidka/6e2692c70aa69daac548518a8bcf753c

long term solution is to update extension to not have failed dependencies.

Update: I actually ran into a few issues so I vagrant ssh into the box and ran:

sudo add-apt-repository ppa:pinepain/libv8-archived
sudo apt-get update

exited and ran vagrant provision and it worked.

BronsonQuick commented 5 years ago

I'm going to double check this with the following versions of PHP:

BronsonQuick commented 5 years ago

@stuartshields I couldn't replicate this on 7.2 with the following config:

extensions:
  - chassis/v8js
php: 7.2

Can you shoot through your other extensions please in case it's a combo of those plus this extension? Taa muchly!

BronsonQuick commented 5 years ago

Alrighty, so it's only broken for PHP 7.3. I forgot that I throw a warning at it only works for PHP 7+.

BronsonQuick commented 5 years ago

https://github.com/phpv8/v8js/issues/392 we've gotta wait for another release for PHP 7.3 to work. It's been a while since the last release: https://github.com/phpv8/v8js/releases/tag/2.1.0

BronsonQuick commented 5 years ago

I just got the extension list from @stuartshields:

version: 2
dependencies:
  - Chassis/phpdbg
  - shadyvb/chassis-redis
  - Chassis/Chassis-Elasticsearch
  - Chassis/v8js
  - Chassis/Tachyon
  - Chassis/intl
  - Chassis/composer
  - Chassis/nodejs
  - Chassis/mcrypt
  - Chassis/Cavalcade
  - Chassis/chassis-openssl
BronsonQuick commented 5 years ago

I think I fixed this in: https://github.com/Chassis/v8js/pull/27 as well.