Hello community,

here is the log from the commit of package yast2-ruby-bindings for 
openSUSE:Factory checked in at 2017-03-15 01:06:12
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/yast2-ruby-bindings (Old)
 and      /work/SRC/openSUSE:Factory/.yast2-ruby-bindings.new (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "yast2-ruby-bindings"

Wed Mar 15 01:06:12 2017 rev:77 rq:479296 version:3.2.7

Changes:
--------
--- /work/SRC/openSUSE:Factory/yast2-ruby-bindings/yast2-ruby-bindings.changes  
2017-01-15 11:02:13.473131086 +0100
+++ 
/work/SRC/openSUSE:Factory/.yast2-ruby-bindings.new/yast2-ruby-bindings.changes 
    2017-03-15 02:00:02.842433784 +0100
@@ -1,0 +2,17 @@
+Tue Mar 14 16:25:59 UTC 2017 - [email protected]
+
+- Use tmux instead of screen to run the Ncurses integration test
+  in headless systems (skip the test if tmux is not available).
+  The version of screen introduced to fix bsc#1021743 was causing
+  the test to fail.
+- 3.2.7
+
+-------------------------------------------------------------------
+Thu Mar  9 15:57:00 UTC 2017 - [email protected]
+
+- Display standard output and error channels in case of failure
+  running the Ncurses integration tests in a headless system.
+  Needed to debug the error produced by the fix to bsc#1021743
+- 3.2.6
+
+-------------------------------------------------------------------

Old:
----
  yast2-ruby-bindings-3.2.5.tar.bz2

New:
----
  yast2-ruby-bindings-3.2.7.tar.bz2

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ yast2-ruby-bindings.spec ++++++
--- /var/tmp/diff_new_pack.NxiJ0z/_old  2017-03-15 02:00:03.838292993 +0100
+++ /var/tmp/diff_new_pack.NxiJ0z/_new  2017-03-15 02:00:03.842292427 +0100
@@ -17,7 +17,7 @@
 
 
 Name:           yast2-ruby-bindings
-Version:        3.2.5
+Version:        3.2.7
 Release:        0
 Url:            https://github.com/yast/yast-ruby-bindings
 BuildRoot:      %{_tmppath}/%{name}-%{version}-build
@@ -46,8 +46,9 @@
 # The test suite includes a regression test (std_streams_spec.rb) for a
 # libyui-ncurses bug fixed in 2.47.3
 BuildRequires:  libyui-ncurses >= 2.47.3
-# The mentioned test requires screen in order to be executed in headless 
systems
-BuildRequires:  screen
+# The mentioned test requires to check if tmux is there, because tmux is
+# needed to execute the test in headless systems
+BuildRequires:  which
 
 # only a soft dependency, the Ruby debugger is optional
 Suggests:       rubygem(%{rb_default_ruby_abi}:byebug)

++++++ yast2-ruby-bindings-3.2.5.tar.bz2 -> yast2-ruby-bindings-3.2.7.tar.bz2 
++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/yast2-ruby-bindings-3.2.5/.travis.yml 
new/yast2-ruby-bindings-3.2.7/.travis.yml
--- old/yast2-ruby-bindings-3.2.5/.travis.yml   2017-01-10 13:21:49.563730000 
+0100
+++ new/yast2-ruby-bindings-3.2.7/.travis.yml   2017-03-14 17:47:00.100198654 
+0100
@@ -1,5 +1,5 @@
 sudo: required
-language: ruby
+language: bash
 services:
   - docker
 
@@ -8,4 +8,4 @@
 script:
   # the "yast-travis-cpp" script is included in the base yastdevel/cpp-tw image
   # see https://github.com/yast/docker-yast-cpp-tw/blob/master/yast-travis-cpp
-  - docker run -it -e TRAVIS_JOB_ID="$TRAVIS_JOB_ID" yast-ruby-bindings-image 
yast-travis-cpp
+  - docker run -it -e TRAVIS=1 -e TRAVIS_JOB_ID="$TRAVIS_JOB_ID" 
yast-ruby-bindings-image yast-travis-cpp
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/yast2-ruby-bindings-3.2.5/Dockerfile 
new/yast2-ruby-bindings-3.2.7/Dockerfile
--- old/yast2-ruby-bindings-3.2.5/Dockerfile    2017-01-10 13:21:49.563730000 
+0100
+++ new/yast2-ruby-bindings-3.2.7/Dockerfile    2017-03-14 17:47:00.112198654 
+0100
@@ -1,2 +1,5 @@
-FROM yastdevel/cpp-tw
-COPY . /tmp/sources
+FROM yastdevel/cpp
+# Install tmux to make sure the libyui+YaST integration tests are run
+RUN zypper --non-interactive in tmux
+
+COPY . /usr/src/app
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-ruby-bindings-3.2.5/package/yast2-ruby-bindings.changes 
new/yast2-ruby-bindings-3.2.7/package/yast2-ruby-bindings.changes
--- old/yast2-ruby-bindings-3.2.5/package/yast2-ruby-bindings.changes   
2017-01-10 13:21:49.567730000 +0100
+++ new/yast2-ruby-bindings-3.2.7/package/yast2-ruby-bindings.changes   
2017-03-14 17:47:00.112198654 +0100
@@ -1,4 +1,21 @@
 -------------------------------------------------------------------
+Tue Mar 14 16:25:59 UTC 2017 - [email protected]
+
+- Use tmux instead of screen to run the Ncurses integration test
+  in headless systems (skip the test if tmux is not available).
+  The version of screen introduced to fix bsc#1021743 was causing
+  the test to fail.
+- 3.2.7
+
+-------------------------------------------------------------------
+Thu Mar  9 15:57:00 UTC 2017 - [email protected]
+
+- Display standard output and error channels in case of failure
+  running the Ncurses integration tests in a headless system.
+  Needed to debug the error produced by the fix to bsc#1021743
+- 3.2.6
+
+-------------------------------------------------------------------
 Mon Jan  9 14:18:30 UTC 2017 - [email protected]
 
 - allow in Yast::SCR and Yast::WFM to have string as first
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-ruby-bindings-3.2.5/package/yast2-ruby-bindings.spec 
new/yast2-ruby-bindings-3.2.7/package/yast2-ruby-bindings.spec
--- old/yast2-ruby-bindings-3.2.5/package/yast2-ruby-bindings.spec      
2017-01-10 13:21:49.567730000 +0100
+++ new/yast2-ruby-bindings-3.2.7/package/yast2-ruby-bindings.spec      
2017-03-14 17:47:00.112198654 +0100
@@ -17,7 +17,7 @@
 
 
 Name:           yast2-ruby-bindings
-Version:        3.2.5
+Version:        3.2.7
 Url:            https://github.com/yast/yast-ruby-bindings
 Release:        0
 BuildRoot:      %{_tmppath}/%{name}-%{version}-build
@@ -46,8 +46,9 @@
 # The test suite includes a regression test (std_streams_spec.rb) for a
 # libyui-ncurses bug fixed in 2.47.3
 BuildRequires:  libyui-ncurses >= 2.47.3
-# The mentioned test requires screen in order to be executed in headless 
systems
-BuildRequires:  screen
+# The mentioned test requires to check if tmux is there, because tmux is
+# needed to execute the test in headless systems
+BuildRequires:  which
 
 # only a soft dependency, the Ruby debugger is optional
 Suggests:       rubygem(%{rb_default_ruby_abi}:byebug)
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/yast2-ruby-bindings-3.2.5/tests/integration/run.rb 
new/yast2-ruby-bindings-3.2.7/tests/integration/run.rb
--- old/yast2-ruby-bindings-3.2.5/tests/integration/run.rb      2017-01-10 
13:21:49.575730000 +0100
+++ new/yast2-ruby-bindings-3.2.7/tests/integration/run.rb      2017-03-14 
17:47:00.136198654 +0100
@@ -7,14 +7,39 @@
 #   rspec std_streams_spec.rb
 # but headless systems like jenkins need this script to fake the screen
 
+def tmux_available?
+  system "which tmux >/dev/null 2>&1"
+end
+
+# If tmux is not available, just skip this without failing
+if !tmux_available?
+  puts "Test skipped."
+  exit true
+end
+
+RESULT = "/tmp/exit".freeze
+OUTPUT = "/tmp/test_cmd_output".freeze
+
+def cleanup
+  [RESULT, OUTPUT].each do |file|
+    File.delete(file) if File.exist?(file)
+  end
+end
+
 test = File.dirname(__FILE__) + "/std_streams_spec.rb"
-cmd = "rspec #{test}"
+cmd = "rspec #{test} >#{OUTPUT} 2>&1"
 
-`screen -D -m sh -c '#{cmd}; echo \$? > /tmp/exit'`
-if File.read("/tmp/exit") != "0\n"
-  puts "Test failed: '#{cmd}'. Rerun manually to see the cause."
-  exit false
-else
+`tmux -c '#{cmd}; echo \$? > #{RESULT}'`
+if File.exist?(RESULT) && File.read(RESULT) == "0\n"
   puts "Test succeeded."
+  cleanup
   exit true
+else
+  puts "Test failed: '#{cmd}'."
+  if File.exist?(OUTPUT)
+    puts "Output was:"
+    puts File.read(OUTPUT)
+  end
+  cleanup
+  exit false
 end
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-ruby-bindings-3.2.5/tests/integration/std_streams_spec.rb 
new/yast2-ruby-bindings-3.2.7/tests/integration/std_streams_spec.rb
--- old/yast2-ruby-bindings-3.2.5/tests/integration/std_streams_spec.rb 
2017-01-10 13:21:49.575730000 +0100
+++ new/yast2-ruby-bindings-3.2.7/tests/integration/std_streams_spec.rb 
2017-03-14 17:47:00.136198654 +0100
@@ -1,8 +1,5 @@
 #! /usr/bin/env rspec
 
-# We do not have a proper ncurses in travis at the moment
-exit 0 if ENV["TRAVIS"]
-
 require_relative "../test_helper"
 require "yast/ui_shortcuts"
 


Reply via email to