[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#769049: unblock: ruby-elasticsearch/1.0.5-1



Package: release.debian.org
Severity: normal
User: release.debian.org@packages.debian.org
Usertags: unblock

Please unblock package ruby-elasticsearch.

This package has a RC bug (#768613) filed against it due to a dependency
that is not currently in testing.  The dependency is for an optional 
transport-layer plugin that doesn't affect any functionality in the 
package.  One unit test (out of 127) has also been disabled.

I would love to see this package stay in testing and not be removed due
to a broken, optional dependency.  No source changes were made, and
the binary packages are unchanged

ruby-elasticsearch (1.0.5-2) unstable; urgency=medium

  * Remove build/test dependency on ruby-patron.  The ruby-patron
    package is not currently in testing and is only one of a handful
    of transport-layer adapters and so does not affect any functionality.
    (Closes: #768613)

 -- Tim Potter <tpot@hp.com>  Tue, 11 Nov 2014 10:21:11 +1100

Attached are debdiffs for the source and binary packages.

unblock ruby-elasticsearch/1.0.5-1

-- System Information:
Debian Release: jessie/sid
  APT prefers unstable
  APT policy: (500, 'unstable')
Architecture: amd64 (x86_64)

Kernel: Linux 3.15.3-tinycore64 (SMP w/8 CPU cores)
Locale: LANG=C, LC_CTYPE=C (charmap=ANSI_X3.4-1968)
Shell: /bin/sh linked to /bin/dash
diff -Nru ruby-elasticsearch-1.0.5/debian/changelog ruby-elasticsearch-1.0.5/debian/changelog
--- ruby-elasticsearch-1.0.5/debian/changelog	2014-11-11 10:48:47.000000000 +1100
+++ ruby-elasticsearch-1.0.5/debian/changelog	2014-11-11 10:51:21.000000000 +1100
@@ -1,3 +1,12 @@
+ruby-elasticsearch (1.0.5-2) unstable; urgency=medium
+
+  * Remove build/test dependency on ruby-patron.  The ruby-patron
+    package is not currently in testing and is only one of a handful
+    of transport-layer adapters and so does not affect any functionality.
+    (Closes: #768613)
+
+ -- Tim Potter <tpot@hp.com>  Tue, 11 Nov 2014 10:21:11 +1100
+
 ruby-elasticsearch (1.0.5-1) unstable; urgency=low

   * Initial release (Closes: #760806)
diff -Nru ruby-elasticsearch-1.0.5/debian/control ruby-elasticsearch-1.0.5/debian/control
--- ruby-elasticsearch-1.0.5/debian/control	2014-11-11 10:48:47.000000000 +1100
+++ ruby-elasticsearch-1.0.5/debian/control	2014-11-11 10:51:21.000000000 +1100
@@ -14,7 +14,6 @@
                ruby-jsonify,
                ruby-mocha,
                ruby-multi-json,
-               ruby-patron,
                ruby-shoulda-context,
                ruby-test-unit
 Standards-Version: 3.9.6
diff -Nru ruby-elasticsearch-1.0.5/debian/patches/disable-patron-test.patch ruby-elasticsearch-1.0.5/debian/patches/disable-patron-test.patch
--- ruby-elasticsearch-1.0.5/debian/patches/disable-patron-test.patch	1970-01-01 10:00:00.000000000 +1000
+++ ruby-elasticsearch-1.0.5/debian/patches/disable-patron-test.patch	2014-11-11 10:51:21.000000000 +1100
@@ -0,0 +1,28 @@
+Description: Remove semi-bogus build/test dependency on ruby-patron
+ The ruby-patron package is currently not in testing, and looks like
+ it is no longer maintained.  This patch disables the build and test
+ dependency on ruby-patron as it's an optional transport-layer plugin.
+Author: Tim Potter <tpot@hp.com>
+Last-Update: 2014-11-11
+---
+This patch header follows DEP-3: http://dep.debian.net/deps/dep3/
+Index: ruby-elasticsearch/elasticsearch-transport/test/unit/client_test.rb
+===================================================================
+--- ruby-elasticsearch.orig/elasticsearch-transport/test/unit/client_test.rb
++++ ruby-elasticsearch/elasticsearch-transport/test/unit/client_test.rb
+@@ -193,15 +193,6 @@ class Elasticsearch::Transport::ClientTe
+
+         assert_includes handlers, Faraday::Adapter::Typhoeus
+       end
+-
+-      should "detect the adapter" do
+-        require 'patron'; load 'patron.rb'
+-
+-        c = Elasticsearch::Transport::Client.new
+-        handlers = c.transport.connections.all.first.connection.builder.handlers
+-
+-        assert_includes handlers, Faraday::Adapter::Patron
+-      end
+     end
+
+   end
diff -Nru ruby-elasticsearch-1.0.5/debian/patches/series ruby-elasticsearch-1.0.5/debian/patches/series
--- ruby-elasticsearch-1.0.5/debian/patches/series	2014-11-11 10:48:47.000000000 +1100
+++ ruby-elasticsearch-1.0.5/debian/patches/series	2014-11-11 10:51:21.000000000 +1100
@@ -1 +1,2 @@
 really-disable-ruby-prof.patch
+disable-patron-test.patch
File lists identical (after any substitutions)

Control files: lines which differ (wdiff format)
------------------------------------------------
Depends: ruby | ruby-interpreter, ruby-elasticsearch-api (= [-1.0.5-1),-] {+1.0.5-2),+} ruby-elasticsearch-transport (= [-1.0.5-1)-] {+1.0.5-2)+}
Version: [-1.0.5-1-] {+1.0.5-2+}
File lists identical (after any substitutions)

Control files: lines which differ (wdiff format)
------------------------------------------------
Version: [-1.0.5-1-] {+1.0.5-2+}
File lists identical (after any substitutions)

Control files: lines which differ (wdiff format)
------------------------------------------------
Version: [-1.0.5-1-] {+1.0.5-2+}

Reply to: