Merge pull request #37 from pinepain/drop-php70-support

Drop PHP 7.0 support

Author: pinepain

.travis.yml

@@ -4,7 +4,6 @@ dist: trusty
 language: php
 
 php:
-  - 7.0
   - 7.1
   - nightly
 

README.md

@@ -4,7 +4,7 @@ PHP extension for V8 JavaScript engine
 [![Build Status](https://travis-ci.org/pinepain/php-v8.svg)](https://travis-ci.org/pinepain/php-v8)
 [![GitHub license](https://img.shields.io/badge/license-MIT-blue.svg)](https://raw.githubusercontent.com/pinepain/php-v8/master/LICENSE)
 
-This extension is for PHP 7 only.
+**This extension requires PHP >= 7.1**. Last version that supports PHP 7.0 is v0.1.9.
 
 **This extension is still under heavy development and its public API may change without any warning. Use at your own risk.**
 

config.m4

@@ -2,6 +2,17 @@ PHP_ARG_WITH(v8, for V8 Javascript Engine,
 [  --with-v8               Include V8 JavaScript Engine])
 
 if test "$PHP_V8" != "no"; then
+
+  AC_MSG_CHECKING([Check for supported PHP versions])
+  PHP_REF_FOUND_VERSION=`${PHP_CONFIG} --version`
+  PHP_REF_FOUND_VERNUM=`${PHP_CONFIG} --vernum`
+
+  if test "$PHP_REF_FOUND_VERNUM" -lt "70100"; then
+    AC_MSG_ERROR([not supported. PHP version >= 7.1 required (found $PHP_REF_FOUND_VERSION)])
+  else
+    AC_MSG_RESULT([supported ($PHP_REF_FOUND_VERSION)])
+  fi
+
   V8_LIB_DIR=""
   V8_INCLUDE_DIR=""
 

package.xml

@@ -379,7 +379,7 @@
     <dependencies>
         <required>
             <php>
-                <min>7.0</min>
+                <min>7.1</min>
             </php>
             <pearinstaller>
                 <min>1.4.0</min>

php_v8.h

@@ -50,6 +50,11 @@ extern zend_module_entry php_v8_module_entry;
 #define PHP_V8_LIBV8_VERSION "undefined"
 #endif
 
+#if PHP_VERSION_ID < 70100
+// should never get her, but just in case
+#error PHP >= 7.1 required
+#endif
+
 
 ZEND_BEGIN_MODULE_GLOBALS(v8)
     bool v8_initialized;
@@ -71,10 +76,6 @@ ZEND_END_MODULE_GLOBALS(v8)
         ZEND_BEGIN_ARG_WITH_RETURN_TYPE_INFO_EX(name, return_reference, required_num_args, type, NULL, allow_null)
 #endif
 
-#if PHP_VERSION_ID < 70100
-    #define zend_get_executed_scope() EG(scope)
-#endif
-
 #define PHP_V8_IS_UP_AND_RUNNING() (zend_is_executing() && !CG(unclean_shutdown))
 
 #define PHP_V8_ME(classname, name, flags)   PHP_ME(classname, name, arginfo_##name, flags)

scripts/provision/provision.sh

@@ -5,7 +5,7 @@ echo Provisioning...
 # Add Ondřej Surý's PPA with co-installable PHP versions:
 sudo add-apt-repository -y ppa:ondrej/php
 # Add libv8 PPA:
-sudo add-apt-repository ppa:pinepain/libv8-6.1
+sudo add-apt-repository ppa:pinepain/libv8-6.2
 
 # Let's update packages list:
 sudo apt-get update
@@ -19,10 +19,9 @@ sudo apt-get install -y git htop curl pkgconf
 
 
 # Build and development requirements
-sudo apt-get install -y libv8-6.1 libv8-6.1-dev libv8-6.1-dbg
+sudo apt-get install -y libv8-6.2 libv8-6.2-dev libv8-6.2-dbg
 sudo apt-get install -y dh-make valgrind
 sudo apt-get install -y libssl-dev openssl
-sudo apt-get install -y php7.0 php7.0-cli php7.0-dev php7.0-fpm
 sudo apt-get install -y php7.1 php7.1-cli php7.1-dev php7.1-fpm
 sudo apt-get install -y php-pear autoconf automake curl libcurl3-openssl-dev build-essential libxslt1-dev re2c libxml2 libxml2-dev bison libbz2-dev libreadline-dev
 sudo apt-get install -y libfreetype6 libfreetype6-dev libpng12-0 libpng12-dev libjpeg-dev libjpeg8-dev libjpeg8  libgd-dev libgd3 libxpm4 libltdl7 libltdl-dev