Hello community,

here is the log from the commit of package linuxrc-devtools for 
openSUSE:Factory checked in at 2018-01-13 21:46:32
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/linuxrc-devtools (Old)
 and      /work/SRC/openSUSE:Factory/.linuxrc-devtools.new (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "linuxrc-devtools"

Sat Jan 13 21:46:32 2018 rev:7 rq:562915 version:0.14

Changes:
--------
--- /work/SRC/openSUSE:Factory/linuxrc-devtools/linuxrc-devtools.changes        
2017-09-21 12:35:07.756866034 +0200
+++ /work/SRC/openSUSE:Factory/.linuxrc-devtools.new/linuxrc-devtools.changes   
2018-01-13 21:46:51.660274719 +0100
@@ -1,0 +2,7 @@
+Tue Jan 9 14:58:21 UTC 2018 - [email protected]
+
+- merge gh#openSUSE/linuxrc-devtools#15
+- more complete OBS state check
+- 0.14
+
+--------------------------------------------------------------------

Old:
----
  linuxrc-devtools-0.13.tar.xz

New:
----
  linuxrc-devtools-0.14.tar.xz

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

Other differences:
------------------
++++++ linuxrc-devtools.spec ++++++
--- /var/tmp/diff_new_pack.VleKFB/_old  2018-01-13 21:46:52.228248241 +0100
+++ /var/tmp/diff_new_pack.VleKFB/_new  2018-01-13 21:46:52.232248054 +0100
@@ -1,7 +1,7 @@
 #
 # spec file for package linuxrc-devtools
 #
-# Copyright (c) 2017 SUSE LINUX GmbH, Nuernberg, Germany.
+# Copyright (c) 2018 SUSE LINUX GmbH, Nuernberg, Germany.
 #
 # All modifications and additions to the file contributed by third parties
 # remain the property of their copyright owners, unless otherwise agreed
@@ -17,7 +17,7 @@
 
 
 Name:           linuxrc-devtools
-Version:        0.13
+Version:        0.14
 Release:        0
 Source:         %{name}-%{version}.tar.xz
 

++++++ linuxrc-devtools-0.13.tar.xz -> linuxrc-devtools-0.14.tar.xz ++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/linuxrc-devtools-0.13/VERSION 
new/linuxrc-devtools-0.14/VERSION
--- old/linuxrc-devtools-0.13/VERSION   2017-09-15 08:36:25.000000000 +0200
+++ new/linuxrc-devtools-0.14/VERSION   2018-01-09 15:58:21.000000000 +0100
@@ -1 +1 @@
-0.13
+0.14
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/linuxrc-devtools-0.13/changelog 
new/linuxrc-devtools-0.14/changelog
--- old/linuxrc-devtools-0.13/changelog 2017-09-15 08:36:25.000000000 +0200
+++ new/linuxrc-devtools-0.14/changelog 2018-01-09 15:58:21.000000000 +0100
@@ -1,3 +1,7 @@
+2018-01-09:    0.14
+       - merge gh#openSUSE/linuxrc-devtools#15
+       - more complete OBS state check
+
 2017-09-15:    0.13
        - merge gh#openSUSE/linuxrc-devtools#14
        - don't query branch from git
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/linuxrc-devtools-0.13/tobs 
new/linuxrc-devtools-0.14/tobs
--- old/linuxrc-devtools-0.13/tobs      2017-09-15 08:36:25.000000000 +0200
+++ new/linuxrc-devtools-0.14/tobs      2018-01-09 15:58:21.000000000 +0100
@@ -644,6 +644,17 @@
 
 
 # - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
+# Wait for OBS build results and submit package if we got a positive result.
+#
+# We loop until we see a positive build result or we're sure the OBS is not
+# building anymore.
+#
+# The package build is considered ok if there's at least a single positive
+# build at this point. If not, it has failed.
+#
+# Note that failed builds do not worry us as many projects are set up to
+# build several targets and not all of them are expected to work.
+#
 sub do_sr
 {
   my $bs = $config->{bs};
@@ -682,6 +693,15 @@
     do {
       sleep $delay;
       if(open my $p, "osc -A https://$bs r --csv $bs_prefix$config->{prj} 
$s[1] |") {
+        # sample line:
+        #
+        # openSUSE_Factory|x86_64|unpublished|False|succeeded|
+        #
+        # field 0, 1, 2: not relevant
+        # field 3: False or True; if True, state is going to change
+        # field 4: status
+        # field 5: sometimes a 6th field is added with the real state
+        #
         while (<$p>) {
           chomp;
           my @i = split /\|/;
@@ -698,12 +718,20 @@
             $i[3] eq "False" &&
             (
               $i[4] eq "failed" ||
+              $i[4] eq "unresolvable" ||
+              $i[4] eq "broken" ||
               $i[4] eq "finished" && $i[5] eq "failed"
             )
           ) {
             $failed = 1;
           }
-          else {
+          elsif(
+            $i[3] eq "False" &&
+            !(
+              $i[4] eq "excluded" ||
+              $i[4] eq "disabled"
+            )
+          ) {
             $building = 1;
           }
         }


Reply via email to