Run the CI against various version of Taglib

Add Rake tasks to build various versions of Taglib.
Get Travis to build Taglib in the before_script step.
Update some unit tests to reflect some changes in taglib.

Author: Sigill

.travis.yml

@@ -1,9 +1,18 @@
 language: ruby
-rvm:
-  - 2.2.7
-  - 2.3.4
-  - 2.4.1
-addons:
-  apt:
-    packages:
-    - libtag1-dev
+env:
+  global:
+    - PLATFORM=x86_64-linux-gnu
+    - MAKEFLAGS="-j2"
+matrix:
+  include:
+    - rvm: 2.2.7
+      env: TAGLIB_VERSION=1.7.2 TAGLIB_DIR=$TRAVIS_BUILD_DIR/tmp/x86_64-linux-gnu/taglib-1.7.2 LD_LIBRARY_PATH=$TRAVIS_BUILD_DIR/tmp/x86_64-linux-gnu/taglib-1.7.2/lib:$LD_LIBRARY_PATH
+    - rvm: 2.2.7
+      env: TAGLIB_VERSION=1.8 TAGLIB_DIR=$TRAVIS_BUILD_DIR/tmp/x86_64-linux-gnu/taglib-1.8 LD_LIBRARY_PATH=$TRAVIS_BUILD_DIR/tmp/x86_64-linux-gnu/taglib-1.8/lib:$LD_LIBRARY_PATH
+    - rvm: 2.3.4
+      env: TAGLIB_VERSION=1.9.1 TAGLIB_DIR=$TRAVIS_BUILD_DIR/tmp/x86_64-linux-gnu/taglib-1.9.1 LD_LIBRARY_PATH=$TRAVIS_BUILD_DIR/tmp/x86_64-linux-gnu/taglib-1.9.1/lib:$LD_LIBRARY_PATH
+    - rvm: 2.4.1
+      env: TAGLIB_VERSION=1.10 TAGLIB_DIR=$TRAVIS_BUILD_DIR/tmp/x86_64-linux-gnu/taglib-1.10 LD_LIBRARY_PATH=$TRAVIS_BUILD_DIR/tmp/x86_64-linux-gnu/taglib-1.10/lib:$LD_LIBRARY_PATH
+    - rvm: 2.4.1
+      env: TAGLIB_VERSION=1.11.1 TAGLIB_DIR=$TRAVIS_BUILD_DIR/tmp/x86_64-linux-gnu/taglib-1.11.1 LD_LIBRARY_PATH=$TRAVIS_BUILD_DIR/tmp/x86_64-linux-gnu/taglib-1.11.1/lib:$LD_LIBRARY_PATH
+before_script: bundle exec rake vendor

README.md

@@ -95,11 +95,19 @@ Build and install gem into system gems:
 
     rake install
 
-The `swig` and `compile` rake tasks can use the `TAGLIB_DIR` environment
-variable to build against non-standard taglib installations. It is assumed
-that taglib headers are located at `$TAGLIB_DIR/include` and taglib libraries
-at `$TAGLIB_DIR/lib`. To run taglib-ruby with non-standard taglib
-installations, use the `LD_LIBRARY_PATH` env. variable.
+Build a specific version of Taglib:
+
+    PLATFORM=x86_64-linux TAGLIB_VERSION=1.9.1 rake vendor
+
+The above command will automatically download Taglig 1.9.1, build it and install it in `tmp/x86_64-linux/taglib-1.9.1`.
+
+The `swig` and `compile` tasks can then be executed against that specific version of Taglib by setting the `TAGLIB_DIR` environment variable to `$PWD/tmp/x86_64-linux/taglib-1.9.1` (it is assumed that taglib headers are located at `$TAGLIB_DIR/include` and taglib libraries at `$TAGLIB_DIR/lib`).
+
+The `test` task can then be run for that version of Taglib by adding `$PWD/tmp/x86_64-linux/taglib-1.9.1/lib` to the `LD_LIBRARY_PATH` environment variable.
+
+To do everything in one command:
+
+    PLATFORM=x86_64-linux TAGLIB_VERSION=1.7.2 TAGLIB_DIR=$PWD/tmp/x86_64-linux/taglib-1.7.2 LD_LIBRARY_PATH=$PWD/tmp/x86_64-linux/taglib-1.7.2/lib rake vendor compile test
 
 ### Workflow
 

tasks/ext.rake

@@ -1,19 +1,28 @@
 # Extension tasks and cross-compiling
 
 host = 'i686-w64-mingw32'
-$plat = 'i386-mingw32'
+$plat = ENV['PLATFORM'] || 'i386-mingw32'
 
-tmp = "#{Dir.pwd}/tmp/#{$plat}"
+
+taglib_version = ENV['TAGLIB_VERSION'] || '1.9.1'
+taglib = "taglib-#{taglib_version}"
+
+tmp = "#{Dir.pwd}/tmp"
+tmp_arch = "#{tmp}/#{$plat}"
 toolchain_file = "#{Dir.pwd}/ext/win.cmake"
-install_dir = "#{tmp}/install"
+install_dir = "#{tmp_arch}/#{taglib}"
 install_dll = "#{install_dir}/bin/libtag.dll"
+install_so = "#{install_dir}/lib/libtag.so"
 $cross_config_options = ["--with-opt-dir=#{install_dir}"]
 
-taglib_version = '1.9.1'
-taglib = "taglib-#{taglib_version}"
-taglib_url = "https://github.com/taglib/taglib/releases/download/v#{taglib_version}/#{taglib}.tar.gz"
-# WITH_MP4, WITH_ASF only needed with taglib 1.7, will be default in 1.8
-taglib_options = "-DCMAKE_BUILD_TYPE=Release -DWITH_MP4=ON -DWITH_ASF=ON"
+taglib_url = "https://github.com/taglib/taglib/archive/v#{taglib_version}.tar.gz"
+taglib_options = [ "-DCMAKE_BUILD_TYPE=Release",
+                   "-DBUILD_EXAMPLES=OFF",
+                   "-DBUILD_TESTS=OFF",
+                   "-DBUILD_BINDINGS=OFF", # 1.11 builds bindings by default
+                   "-DBUILD_SHARED_LIBS=ON", # 1.11 builds static by default
+                   "-DWITH_MP4=ON",# WITH_MP4, WITH_ASF only needed with taglib 1.7, will be default in 1.8
+                   "-DWITH_ASF=ON"].join(' ')
 
 def configure_cross_compile(ext)
   ext.cross_compile = true
@@ -62,7 +71,7 @@ task :cross do
   ENV["CXX"] = "#{host}-g++"
 end
 
-file "tmp/#{$plat}/stage/lib/libtag.dll" => [install_dll] do |f|
+file "#{tmp_arch}/stage/lib/libtag.dll" => [install_dll] do |f|
   install install_dll, f
 end
 
@@ -74,6 +83,18 @@ file install_dll => ["#{tmp}/#{taglib}"] do
   end
 end
 
+task :vendor => [install_so]
+
+file install_so => ["#{tmp_arch}/#{taglib}", "#{tmp_arch}/#{taglib}-build", "#{tmp}/#{taglib}"] do
+  chdir "#{tmp_arch}/#{taglib}-build" do
+    sh %(cmake -DCMAKE_INSTALL_PREFIX=#{install_dir} #{taglib_options} #{tmp}/#{taglib})
+    sh "make install VERBOSE=1"
+  end
+end
+
+directory "#{tmp_arch}/#{taglib}"
+directory "#{tmp_arch}/#{taglib}-build"
+
 file "#{tmp}/#{taglib}" => ["#{tmp}/#{taglib}.tar.gz"] do
   chdir tmp do
     sh "tar xzf #{taglib}.tar.gz"
@@ -83,12 +104,9 @@ end
 file "#{tmp}/#{taglib}.tar.gz" => [tmp] do |t|
   require 'open-uri'
   puts "Downloading #{taglib_url}"
-  data = open(taglib_url).read()
-  break if data == nil
-  chdir tmp do
-    open(File.basename(t.name), 'wb') do |f|
-      f.write(data)
-    end
+
+  File.open(t.name, 'wb') do |f|
+    IO.copy_stream(open(taglib_url), f)
   end
 end
 

test/flac_file_test.rb

@@ -37,7 +37,14 @@ class FlacFileTest < Test::Unit::TestCase
 
       should "contain basic information" do
         assert_equal 1, @properties.length
-        assert_equal 212, @properties.bitrate
+
+        # taglib/taglib#557 changed the way bitrate is calculated.
+        if TagLib::TAGLIB_MAJOR_VERSION > 1 || (TagLib::TAGLIB_MAJOR_VERSION == 1 && TagLib::TAGLIB_MINOR_VERSION >= 10)
+          assert_equal 209, @properties.bitrate
+        else
+          assert_equal 212, @properties.bitrate
+        end
+
         assert_equal 44100, @properties.sample_rate
         assert_equal 1, @properties.channels
       end

test/id3v2_tag_test.rb

@@ -44,11 +44,20 @@ class TestID3v2Tag < Test::Unit::TestCase
     end
 
     should "have nil for string attributes" do
-      assert_nil @tag.title
-      assert_nil @tag.artist
-      assert_nil @tag.album
-      assert_nil @tag.comment
-      assert_nil @tag.genre
+      # taglib/taglib/8c6fe45: return an empty string instead of NULL
+      if TagLib::TAGLIB_MAJOR_VERSION > 1 || (TagLib::TAGLIB_MAJOR_VERSION == 1 && TagLib::TAGLIB_MINOR_VERSION >= 11)
+        assert_equal "", @tag.title
+        assert_equal "", @tag.artist
+        assert_equal "", @tag.album
+        assert_equal "", @tag.comment
+        assert_equal "", @tag.genre
+      else
+        assert_nil @tag.title
+        assert_nil @tag.artist
+        assert_nil @tag.album
+        assert_nil @tag.comment
+        assert_nil @tag.genre
+      end
     end
 
     should "have 0 for numeric attributes" do

test/mp4_file_test.rb

@@ -29,7 +29,14 @@ class MP4FileTest < Test::Unit::TestCase
 
       should "contain basic information" do
         assert_equal 1, @properties.length
-        assert_equal 54, @properties.bitrate
+
+        # taglib/taglib#558 changed the way bitrate is calculated.
+        if TagLib::TAGLIB_MAJOR_VERSION > 1 || (TagLib::TAGLIB_MAJOR_VERSION == 1 && TagLib::TAGLIB_MINOR_VERSION >= 10)
+          assert_equal 55, @properties.bitrate
+        else
+          assert_equal 54, @properties.bitrate
+        end
+
         assert_equal 44100, @properties.sample_rate
         # The test file is mono, this appears to be a TagLib bug
         assert_equal 2, @properties.channels

test/vorbis_file_test.rb

@@ -23,7 +23,14 @@ class TestVorbisFile < Test::Unit::TestCase
 
       should "contain basic information" do
         assert_equal 0, @properties.length # file is short
-        assert_equal 64, @properties.bitrate
+
+        # taglib/taglib#563 changed the way bitrate is calculated.
+        if TagLib::TAGLIB_MAJOR_VERSION > 1 || (TagLib::TAGLIB_MAJOR_VERSION == 1 && TagLib::TAGLIB_MINOR_VERSION >= 10)
+          assert_equal 222, @properties.bitrate
+        else
+          assert_equal 64, @properties.bitrate
+        end
+
         assert_equal 44100, @properties.sample_rate
         assert_equal 2, @properties.channels
       end