[SCM] ci-tooling packaging branch, master, updated. 9898d033e20fe2ff6c98fb0e3a830884b03a3fdd

[Harald Sitter]

Gitweb-URL: http://git.debian.org/?p=pkg-kde/ci-tooling.git;a=commitdiff;h=1750fcc

The following commit has been merged in the master branch:
commit 1750fccc8755449ae7c5b5fae0ea14d1ff1ad468
Author: Harald Sitter <sitter at kde.org>
Date:   Thu Oct 8 12:47:13 2015 +0200

    reduce verbosity of source publisher
---
 kci/source_publisher.rb | 17 +++--------------
 1 file changed, 3 insertions(+), 14 deletions(-)

diff --git a/kci/source_publisher.rb b/kci/source_publisher.rb
index 5c4db3a..f6e6bea 100755
--- a/kci/source_publisher.rb
+++ b/kci/source_publisher.rb
@@ -25,7 +25,6 @@ class SourcePublisher
     fail_count = 30 # This is ~= minutes
 
     until source
-      puts 'no source yet'
       sleep(60)
       fail_count -= 1
       if fail_count <= 0
@@ -43,12 +42,6 @@ class SourcePublisher
       has_failed = false
 
       source.getBuilds.each do |build|
-        state = build.buildstate
-        puts '------'
-        puts state
-        puts source.source_package_version
-        puts build.arch_tag
-
         case state
         when *WAITING_STATES
           needs_wait = true
@@ -61,8 +54,6 @@ class SourcePublisher
           fail "Build state '#{build.buildstate}' is not being handled"
         end
       end
-      puts "+++++"
-      puts "going to fail: #{has_failed}. going to wait: #{needs_wait} (waiting for #{sleep_time})"
 
       if needs_wait
         puts "Sleeping for #{sleep_time} seconds....."
@@ -71,7 +62,7 @@ class SourcePublisher
       end
 
       if has_failed
-        puts "Got a build failure"
+        puts 'Got a build failure'
         print_state(source)
         return false
       end
@@ -87,8 +78,6 @@ class SourcePublisher
       has_pending = false
       source_id = File.basename(source.self_link)
       build_summary = @ppa.getBuildSummariesForSourceIds(:source_ids => source_id)
-      require 'pp'
-      pp build_summary
       status = build_summary[source_id]['status']
       case status
       when 'FULLYBUILT_PENDING'
@@ -98,10 +87,10 @@ class SourcePublisher
       else
         # FIXME: fail?
         puts "Something very terrible happened as the overall state is #{status}, which was not expected at all"
-        print_state(get_source())
+        print_state(get_source)
         return false
       end
-      if not has_pending
+      unless has_pending
         puts 'All things are published, hooray!'
         break
       end

-- 
ci-tooling packaging