libv8 5.1.281.59.1 → 5.2.361.43.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: bb0457ce92b62911a3520bab6259b048378adfed
-  data.tar.gz: 5f629dd5295fba00475ae87459da69adf81da94f
+  metadata.gz: d2d214217489251ff3cc384d2f717358bc010132
+  data.tar.gz: 7ef46315f67750f3615f81790ae4b2c4a48a9c58
 SHA512:
-  metadata.gz: f0478e44a7a15fb06bdeb263ca35a841f5818eb1b3799f1ebc1e8e725a4e75f10197355d3ec8ce0d69ea11f27d3327b7915e581a1d0e9a315e4dbd2e618e1f3f
-  data.tar.gz: 3ead59dfe9599e14344f515a72387380bef274aee6d5116e463be9bd1489f96090ad50360d1ac56914ae7f111b7fc4962ab55c7f06c8665157ecb1abc86c8adf
+  metadata.gz: df2615b61c27ed8684e30c9dc800a731fb2d94d407b68950bdfda9ddca4ee1412c8844834647af7aa504e732b5a75a333e08299a631ee44b67a7266ab569c9b8
+  data.tar.gz: fdb034e795fbd8643b68b323cba30530ce15380775068521fdce1be841d9bd7f4df55b2ec6ce854389609cd31b9dd91b2fe869b105d26413d143b7e53385d332

data/.travis.yml

@@ -31,8 +31,6 @@ before_install:
   - if [ "$TRAVIS_OS_NAME" == "linux" ]; then gem update bundler; fi
   - if [ "$TRAVIS_OS_NAME" == "osx" ]; then sudo gem install bundler; fi
 script:
-  - if ! [ "$(ls -A vendor/v8/.git)" ] ; then rm -rf vendor/v8; fi
-  - rm -rf vendor/depot_tools
   - git submodule update --init
   - MAKEFLAGS+=-j8 bundle exec rake spec binary --trace
 deploy:
@@ -44,13 +42,8 @@ deploy:
   on:
     tags: true
     condition: $TRAVIS_OS_NAME = osx
-before_cache:
-  - cd vendor/v8 && git clean -xdf && git checkout . && cd ../..
-  - rm -rf vendor/depot_tools
 cache:
   bundler: true
-  directories:
-    - vendor
 notifications:
   recipients:
     - cowboyd@thefrontside.net

data/CHANGELOG.md

@@ -1,4 +1,12 @@
-### Unreleased
+### 5.2.361.43.0, 5.2.361.43.1 - 2016-06-26:
+
+* Compare compiler versions part by part as integers instead of using string
+  comparison on the whole version string #154 (thanks @ltk)
+* Update upstream v8 to version 5.2.361.43 and refresh the patch set
+
+### 5.1.281.67.0, 5.1.281.67.1 - 2016-06-26:
+
+* Update upstream v8 version to 5.1.281.67 to address #219
 
 ### 5.1.281.59.0, 5.1.281.59.1 - 2016-06-15:
 

data/README.md

@@ -1,7 +1,6 @@
 # libv8
 [![Gem Version](https://badge.fury.io/rb/libv8.svg)](http://badge.fury.io/rb/libv8)
 [![Build Status](https://travis-ci.org/cowboyd/libv8.svg?branch=master)](https://travis-ci.org/cowboyd/libv8)
-<!-- [![Build status](https://ci.appveyor.com/api/projects/status/ugeqff972kxlba5j?svg=true)](https://ci.appveyor.com/project/cowboyd/libv8) -->
 [![Join the chat at https://gitter.im/cowboyd/therubyracer](https://badges.gitter.im/Join%20Chat.svg)](https://gitter.im/cowboyd/therubyracer?utm_source=badge&utm_medium=badge&utm_campaign=pr-badge&utm_content=badge)
 
 A gem for distributing the v8 runtime libraries and headers in both
@@ -32,13 +31,25 @@ If you don't see your platform on this list, first, make sure that it
 installs from source, and second talk to us about setting up a binary
 distro for you.
 
+#### Note on ~~OS X~~ macOS binaries
+
+If you're installing libv8 on a macOS system that is present in the list above,
+and despite that, RubyGems insists on downloading a source version and compiling
+it, check the output of `ruby -e 'puts Gem::Platform.local'`. If it does not
+reflect the current version of your OS, recompile Ruby.
+
+The platform gets hardcoded in Ruby during compilation and if you've updated
+your OS since you've compiled Ruby, it does not represent correctly your current
+platform which leads to RubyGems trying to download a platform-specific gem for
+the older version of your OS.
+
 ### Versioning
 
-Versions of the libv8 gem track the version of v8 itself, adding its
-own point release after the main v8 version. So libv8 `5.0.71.35.5`
-and `5.0.71.35.14` both correspond to v8 version `5.0.71.35`. Another
+Versions of the libv8 gem track the version of V8 itself, adding its
+own point release after the main V8 version. So libv8 `5.0.71.35.5`
+and `5.0.71.35.14` both correspond to V8 version `5.0.71.35`. Another
 way to think about it would be that `5.0.71.35.14` is the 14th release
-of the libv8 rubygem based on v8 version `5.0.71.35`
+of the libv8 rubygem based on V8 version `5.0.71.35`
 
 #### Source and Binary Releases
 
@@ -61,17 +72,17 @@ source-based distribution
 
 ##### Use with different standard C libraries
 
-The binary versions of this gem are linked against the most used standard
-library - glibc. Currently rubygems does not differentiate binary gems by
-standard library so we have no way of distributing different binaries for
-differnt standard libraries.
+The Linux binary versions of this gem are linked against the most used standard
+library - glibc. Currently rubygems has no mechanism to differentiate
+platform-specific gems by standard library so we have no way of distributing
+different binaries for different standard libraries.
 
 What this means is that if you're running a distro that does not use glibc
 (like Alpine Linux), you'll have to use a source version of the gem.
 
 ### Requirements
 
-Building the v8 library from source imposes the following requirements:
+Building the V8 library from source imposes the following requirements:
 
 *  A compiler that supports C++11 (such as GCC 4.8 and above or clang,
 preferably 3.5 and above)
@@ -93,7 +104,7 @@ You can find more info on using a git repo as a gem source in
 
 ### What if I can't install from source?
 
-If you can fix v8's build system so that it correctly compiles for your
+If you can fix V8's build system so that it correctly compiles for your
 platform, we'll pull it right in!
 
 To get the source, these commands will get you started:
@@ -105,11 +116,16 @@ To get the source, these commands will get you started:
 
 ### Bring your own V8
 
-Because libv8 is the interface for the V8 engine used by
-[therubyracer](http://github.com/cowboyd/therubyracer), you may need
-to use libv8, even if you have V8 installed already. If you wish to
-use your own V8 installation, rather than have it built for you, use
-the `--with-system-v8` option.
+*This is a great way to ensure that the builds of all gems that depend on libv8
+fail. Please see the Gotchas section below and use the follwing instructions
+only if you know what you're doing. If you're resorting to this because the
+build of the gem is failing on your system or because there's no
+platform-specific gem for your platform, please open up an issue.*
+
+Because libv8 is the interface for the V8 engine used by several gems, you may
+need to use libv8, even if you have V8 installed already. If you wish to use
+your own V8 installation, rather than have it built for you, use the
+`--with-system-v8` option.
 
 Using RubyGems:
 
@@ -122,12 +138,14 @@ Using Bundler (in your Gemfile):
 #### Gotchas
 
 Please note that if you intend to run your own V8, you must install
-both V8 *and its headers* (found in libv8-dev for Debian distros).
+both V8 *and its headers* (found in libv8-dev for Debian-based distros).
 
-Also keep in mind that v8's API does not tend to be stable and you
-need to make sure that the version of therubyracer you intend to use
-is compatible with the version of v8 present on your system. Otherwise
-therubyracer's build will fail.
+Also, keep in mind that V8's API does not tend to be stable and in case you're
+using your local version of V8 you *need* to **make sure that the the gems that
+depend on libv8 are compatible with the API of the version of V8 present on your
+system**. Otherwise those gems' builds *will* fail. Ideally you want the same
+version of V8 as the one packaged in the installed version of the gem. See the
+Versioning section for more information.
 
 ### Bring your own compiler
 
@@ -141,11 +159,11 @@ environment variable before compilation, or by adding the
 
 This project spun off of
 [therubyracer](http://github.com/cowboyd/therubyracer) which depends
-on having a specific version of v8 to compile and run against.
+on having a specific version of V8 to compile and run against.
 However, actually delivering that version reliably to all the
 different platforms proved to be a challenge to say the least.
 
-We got tired of waiting 5 minutes for v8 to compile every time we
+We got tired of waiting 5 minutes for V8 to compile every time we
 installed that gem.
 
 ### Sponsored by

data/Rakefile

@@ -51,7 +51,7 @@ task :binary => :compile do
 end
 
 namespace :build do
-  ['x86_64-linux', 'x86-linux', 'x86_64-freebsd10'].each do |arch|
+  ['x86_64-linux', 'x86-linux', 'armhf-linux', 'x86_64-freebsd-10'].each do |arch|
     desc "build binary gem for #{arch}"
     task arch do
       arch_dir = Pathname(__FILE__).dirname.join("release/#{arch}")

data/ext/libv8/builder.rb

@@ -108,7 +108,7 @@ module Libv8
           system "gclient fetch" or fail "could not fetch v8 build dependencies commits"
         end
         Dir.chdir('v8') do
-          unless system "git checkout #{source_version} -- ."
+          unless system "git checkout #{source_version}"
             fail "unable to checkout source for v8 #{source_version}"
           end
           system "gclient sync" or fail "could not sync v8 build dependencies"

data/ext/libv8/compiler/apple_llvm.rb

@@ -2,17 +2,18 @@ module Libv8
   module Compiler
     class AppleLLVM < Clang
       LLVM_VERSION_REGEXP = /Apple LLVM version (\d+\.\d+(\.\d+)*) \(/i
+      REQUIRED_VERSION = '4.3'
 
       def name
         'Apple LLVM'
       end
 
-      def compatible?
-        version >= '4.3' unless version.nil?
-      end
-
       private
 
+      def required_version
+        REQUIRED_VERSION
+      end
+
       def version_regexp
         LLVM_VERSION_REGEXP
       end

data/ext/libv8/compiler/clang.rb

@@ -2,17 +2,18 @@ module Libv8
   module Compiler
     class Clang < GenericCompiler
       CLANG_VERSION_REGEXP = /clang version (\d+\.\d+(\.\d+)*) \(/i
+      REQUIRED_VERSION = '3.1'
 
       def name
         'clang'
       end
 
-      def compatible?
-        version >= '3.1' unless version.nil?
-      end
-
       private
 
+      def required_version
+        REQUIRED_VERSION
+      end
+
       def version_regexp
         CLANG_VERSION_REGEXP
       end

data/ext/libv8/compiler/gcc.rb

@@ -2,17 +2,18 @@ module Libv8
   module Compiler
     class GCC < GenericCompiler
       GCC_VERSION_REGEXP = /gcc version (\d+\.\d+(\.\d+)*)/i
+      REQUIRED_VERSION = '4.7'
 
       def name
         'GCC'
       end
 
-      def compatible?
-        version > '4.7' unless version.nil?
-      end
-
       private
 
+      def required_version
+        REQUIRED_VERSION
+      end
+
       def version_regexp
         GCC_VERSION_REGEXP
       end

data/ext/libv8/compiler/generic_compiler.rb

@@ -27,7 +27,9 @@ module Libv8
       end
 
       def compatible?
-        false
+        return false unless required_version && !version.nil?
+
+        (string_to_semver(version) <=> string_to_semver(required_version)) > -1
       end
 
       def call(*arguments)
@@ -51,6 +53,14 @@ module Libv8
       def target_regexp
         GENERIC_TARGET_REGEXP
       end
+
+      def required_version
+        nil
+      end
+
+      def string_to_semver(version)
+        version.split('.').map(&:to_i)
+      end
     end
   end
 end

data/lib/libv8/version.rb

@@ -1,3 +1,3 @@
 module Libv8
-  VERSION = "5.1.281.59.1"
+  VERSION = "5.2.361.43.0"
 end

data/patches/0001-Build-a-standalone-static-library.patch

@@ -0,0 +1,26 @@
+From 84470a0b6c696f42aa7f86ba5ffd22b17e0047c4 Mon Sep 17 00:00:00 2001
+From: Petko Bordjukov <bordjukov@gmail.com>
+Date: Mon, 25 Jul 2016 00:00:19 +0300
+Subject: [PATCH 1/4] Build a standalone static library
+
+---
+ gypfiles/standalone.gypi | 3 +++
+ 1 file changed, 3 insertions(+)
+
+diff --git a/gypfiles/standalone.gypi b/gypfiles/standalone.gypi
+index 630c45e..1a69c6f 100644
+--- a/gypfiles/standalone.gypi
++++ b/gypfiles/standalone.gypi
+@@ -488,6 +488,9 @@
+       }],  # fastbuild!=0
+     ],
+     'target_conditions': [
++      ['_type=="static_library"', {
++        'standalone_static_library': 1,
++      }],
+       ['v8_code == 0', {
+         'defines!': [
+           'DEBUG',
+-- 
+2.9.0
+

data/patches/{0002-Disable-building-tests.patch → 0002-Don-t-compile-unnecessary-stuff.patch}

@@ -1,51 +1,48 @@
-From a27a92cbe9db50bceedfffeb43d12345209e4d75 Mon Sep 17 00:00:00 2001
+From 930d596106fb3289f6aa37cdf8abaec3106c31b9 Mon Sep 17 00:00:00 2001
 From: Petko Bordjukov <bordjukov@gmail.com>
-Date: Fri, 10 Jun 2016 08:57:29 +0300
-Subject: [PATCH 2/3] Disable building tests
+Date: Mon, 25 Jul 2016 00:04:07 +0300
+Subject: [PATCH 2/4] Don't compile unnecessary stuff
 
 ---
- Makefile      |  6 ++----
- build/all.gyp | 27 ---------------------------
- 2 files changed, 2 insertions(+), 31 deletions(-)
+ Makefile         |  7 +------
+ gypfiles/all.gyp | 32 --------------------------------
+ 2 files changed, 1 insertion(+), 38 deletions(-)
 
 diff --git a/Makefile b/Makefile
-index a0c08a6..10b2924 100644
+index 0f8f93a..68778e9 100644
 --- a/Makefile
 +++ b/Makefile
-@@ -258,11 +258,9 @@ NACL_ARCHES = nacl_ia32 nacl_x64
- GYPFILES = third_party/icu/icu.gypi third_party/icu/icu.gyp \
- 	   build/shim_headers.gypi build/features.gypi build/standalone.gypi \
- 	   build/toolchain.gypi build/all.gyp build/mac/asan.gyp \
+@@ -264,12 +264,7 @@ GYPFILES = third_party/icu/icu.gypi third_party/icu/icu.gyp \
+ 	   gypfiles/shim_headers.gypi gypfiles/features.gypi \
+            gypfiles/standalone.gypi \
+ 	   gypfiles/toolchain.gypi gypfiles/all.gyp gypfiles/mac/asan.gyp \
 -	   test/cctest/cctest.gyp test/fuzzer/fuzzer.gyp \
--	   test/unittests/unittests.gyp tools/gyp/v8.gyp \
+-	   test/unittests/unittests.gyp src/v8.gyp \
 -	   tools/parser-shell.gyp testing/gmock.gyp testing/gtest.gyp \
-+	   tools/gyp/v8.gyp tools/parser-shell.gyp \
- 	   buildtools/third_party/libc++abi/libc++abi.gyp \
+-	   buildtools/third_party/libc++abi/libc++abi.gyp \
 -	   buildtools/third_party/libc++/libc++.gyp samples/samples.gyp \
-+	   buildtools/third_party/libc++/libc++.gyp \
- 	   src/third_party/vtune/v8vtune.gyp src/d8.gyp
+-	   src/third_party/vtune/v8vtune.gyp src/d8.gyp
++	   src/v8.gyp tools/parser-shell.gyp src/d8.gyp
  
  # If vtunejit=on, the v8vtune.gyp will be appended.
-diff --git a/build/all.gyp b/build/all.gyp
-index feaf4fe..96820a0 100644
---- a/build/all.gyp
-+++ b/build/all.gyp
-@@ -8,11 +8,7 @@
-       'target_name': 'All',
-       'type': 'none',
-       'dependencies': [
--        '../samples/samples.gyp:*',
-         '../src/d8.gyp:d8',
--        '../test/cctest/cctest.gyp:*',
--        '../test/fuzzer/fuzzer.gyp:*',
--        '../test/unittests/unittests.gyp:*',
-       ],
-       'conditions': [
-         ['component!="shared_library"', {
-@@ -20,29 +16,6 @@
+ ifeq ($(vtunejit), on)
+diff --git a/gypfiles/all.gyp b/gypfiles/all.gyp
+index ff1bea4..96820a0 100644
+--- a/gypfiles/all.gyp
++++ b/gypfiles/all.gyp
+@@ -16,38 +16,6 @@
              '../tools/parser-shell.gyp:parser-shell',
            ],
          }],
+-        # These items don't compile for Android on Mac.
+-        ['host_os!="mac" or OS!="android"', {
+-          'dependencies': [
+-            '../samples/samples.gyp:*',
+-            '../test/cctest/cctest.gyp:*',
+-            '../test/fuzzer/fuzzer.gyp:*',
+-            '../test/unittests/unittests.gyp:*',
+-            ],
+-        }],
 -        ['test_isolation_mode != "noop"', {
 -          'dependencies': [
 -            '../test/bot_default.gyp:*',
@@ -73,5 +70,5 @@ index feaf4fe..96820a0 100644
      }
    ]
 -- 
-2.8.3
+2.9.0
 

data/patches/0003-Use-the-fPIC-flag-for-the-static-library.patch

@@ -1,17 +1,17 @@
-From 8c92844ee381fc2074204dd52ca3668e3c6fe89f Mon Sep 17 00:00:00 2001
+From 8873d0ee6e1304e256bf3181b287e189bc68a9c8 Mon Sep 17 00:00:00 2001
 From: Petko Bordjukov <bordjukov@gmail.com>
-Date: Fri, 10 Jun 2016 08:58:29 +0300
-Subject: [PATCH 3/3] Use the -fPIC flag for the static library
+Date: Mon, 25 Jul 2016 00:05:47 +0300
+Subject: [PATCH 3/4] Use the -fPIC flag for the static library
 
 ---
- build/standalone.gypi | 2 +-
+ gypfiles/standalone.gypi | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)
 
-diff --git a/build/standalone.gypi b/build/standalone.gypi
-index 4f10909..3a67af3 100644
---- a/build/standalone.gypi
-+++ b/build/standalone.gypi
-@@ -728,7 +728,7 @@
+diff --git a/gypfiles/standalone.gypi b/gypfiles/standalone.gypi
+index 1a69c6f..b19304f 100644
+--- a/gypfiles/standalone.gypi
++++ b/gypfiles/standalone.gypi
+@@ -744,7 +744,7 @@
            [ 'visibility=="hidden" and v8_enable_backtrace==0', {
              'cflags': [ '-fvisibility=hidden' ],
            }],
@@ -21,5 +21,5 @@ index 4f10909..3a67af3 100644
            }],
            [ 'clang==0 and coverage==1', {
 -- 
-2.8.3
+2.9.0
 

data/patches/0004-Reinterpret-thread-hash-for-FreeBSD-too.patch

@@ -0,0 +1,25 @@
+From 0f376ac8f77980b86bfbfb55e3a1714201a0f78f Mon Sep 17 00:00:00 2001
+From: Petko Bordjukov <bordjukov@gmail.com>
+Date: Wed, 27 Jul 2016 06:20:13 +0300
+Subject: [PATCH 4/4] Reinterpret thread hash for FreeBSD, too
+
+---
+ src/profiler/sampler.cc | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/src/profiler/sampler.cc b/src/profiler/sampler.cc
+index ae47dca..4eb8780 100644
+--- a/src/profiler/sampler.cc
++++ b/src/profiler/sampler.cc
+@@ -210,7 +210,7 @@ void* ThreadKey(pthread_t thread_id) {
+ 
+ // Returns hash value for hash map.
+ uint32_t ThreadHash(pthread_t thread_id) {
+-#if V8_OS_MACOSX
++#if (V8_OS_MACOSX || V8_OS_FREEBSD)
+   return static_cast<uint32_t>(reinterpret_cast<intptr_t>(thread_id));
+ #else
+   return static_cast<uint32_t>(thread_id);
+-- 
+2.9.0
+