Hello community,

here is the log from the commit of package obs-service-source_validator for 
openSUSE:Factory checked in at 2015-06-12 20:29:35
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/obs-service-source_validator (Old)
 and      /work/SRC/openSUSE:Factory/.obs-service-source_validator.new (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "obs-service-source_validator"

Changes:
--------
--- 
/work/SRC/openSUSE:Factory/obs-service-source_validator/obs-service-source_validator.changes
        2015-05-16 07:12:21.000000000 +0200
+++ 
/work/SRC/openSUSE:Factory/.obs-service-source_validator.new/obs-service-source_validator.changes
   2015-06-12 20:29:36.000000000 +0200
@@ -1,0 +2,16 @@
+Thu Jun 11 08:58:17 UTC 2015 - [email protected]
+
+- Update to version 0.5+git20150611.b85a5d4:
+  + unbreak source validator with packages using source services again
+    * Not all _* files are xml
+    * Don't complain about service generated files
+
+-------------------------------------------------------------------
+Wed May 27 13:54:35 UTC 2015 - [email protected]
+
+- Update to version 0.5+git20150527.6e08ec8:
+  + baselibs.conf: ignore lines starting with #; comments might be welcome
+  + baselibs.conf: 'arch' is also a valid keyword
+  + Avoid trashing _service file
+
+-------------------------------------------------------------------

Old:
----
  obs-service-source_validator-0.5+git20150513.7c2f7ae.tar.bz2

New:
----
  obs-service-source_validator-0.5+git20150611.b85a5d4.tar.bz2

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

Other differences:
------------------
++++++ obs-service-source_validator.spec ++++++
--- /var/tmp/diff_new_pack.oYujfy/_old  2015-06-12 20:29:37.000000000 +0200
+++ /var/tmp/diff_new_pack.oYujfy/_new  2015-06-12 20:29:37.000000000 +0200
@@ -20,7 +20,7 @@
 Summary:        An OBS source service: running all the osc source-validator 
checks
 License:        GPL-2.0+
 Group:          Development/Tools/Building
-Version:        0.5+git20150513.7c2f7ae
+Version:        0.5+git20150611.b85a5d4
 Release:        0
 # use osc service dr to update
 Source:         %{name}-%{version}.tar.bz2

++++++ _servicedata ++++++
--- /var/tmp/diff_new_pack.oYujfy/_old  2015-06-12 20:29:37.000000000 +0200
+++ /var/tmp/diff_new_pack.oYujfy/_new  2015-06-12 20:29:37.000000000 +0200
@@ -1,4 +1,4 @@
 <servicedata>
 <service name="tar_scm">
             <param 
name="url">git://github.com/openSUSE/obs-service-source_validator.git</param>
-          <param 
name="changesrevision">7c2f7ae62322ff5293567d70bfa4904568296a11</param></service></servicedata>
\ No newline at end of file
+          <param 
name="changesrevision">b85a5d4b92edd871e77b4af202967ee6290e6d88</param></service></servicedata>
\ No newline at end of file

++++++ obs-service-source_validator-0.5+git20150513.7c2f7ae.tar.bz2 -> 
obs-service-source_validator-0.5+git20150611.b85a5d4.tar.bz2 ++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/obs-service-source_validator-0.5+git20150513.7c2f7ae/20-files-present-and-referenced
 
new/obs-service-source_validator-0.5+git20150611.b85a5d4/20-files-present-and-referenced
--- 
old/obs-service-source_validator-0.5+git20150513.7c2f7ae/20-files-present-and-referenced
    2015-05-13 10:54:32.000000000 +0200
+++ 
new/obs-service-source_validator-0.5+git20150611.b85a5d4/20-files-present-and-referenced
    2015-06-11 10:58:17.000000000 +0200
@@ -176,23 +176,27 @@
 
 # XML validate files starting with _..
 if [ -x $(type -p xmllint) ]; then
-    # Check if _service is sane
-    if [ -f $DIR_TO_CHECK/_service ]; then
-        xmllint --format $DIR_TO_CHECK/_service > $TMPDIR/_service
-
-        if egrep -q "service .*mode=." $TMPDIR/_service \
-                && ! egrep -q "service .*mode=.(disabled|localrun)" \
-                $TMPDIR/_service; then
-            echo "(W) openSUSE: projects only allow 'disabled or 'localrun' 
services."
-        fi
-    fi
-
-    for i in $DIR_TO_CHECK/_*; do
+    for i in $DIR_TO_CHECK/_service; do
         test -f $i || continue
+        BASE=${i##*/}
+
         xmllint --format $i >/dev/null || {
             echo "(E) $(basename $i) is not valid XML"
             RETURN=2
+            continue
         }
+
+        # Check if _service is sane
+        if [ "$BASE" = "_service" ]; then
+            xmllint --format $i > $TMPDIR/_service
+
+            if egrep -q "service .*mode=." $TMPDIR/_service \
+                    && ! egrep -q "service .*mode=.(disabled|localrun)" \
+                    $TMPDIR/_service; then
+                echo "(W) openSUSE: projects only allow 'disabled or 
'localrun' services."
+            fi
+        fi
+
     done
 fi
 
@@ -216,9 +220,11 @@
        fi
 }
 
-for HASTOBETHER in `cat $TMPDIR/sources` ; do
-    check_tracked "$HASTOBETHER" || RETURN=2
-done
+if ! test -f "$DIR_TO_CHECK/_service"; then
+    for HASTOBETHER in `cat $TMPDIR/sources` ; do
+        check_tracked "$HASTOBETHER" || RETURN=2
+    done
+fi
 
 #
 # Verify GPG keys
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/obs-service-source_validator-0.5+git20150513.7c2f7ae/70-baselibs 
new/obs-service-source_validator-0.5+git20150611.b85a5d4/70-baselibs
--- old/obs-service-source_validator-0.5+git20150513.7c2f7ae/70-baselibs        
2015-05-13 10:47:14.000000000 +0200
+++ new/obs-service-source_validator-0.5+git20150611.b85a5d4/70-baselibs        
2015-06-11 10:58:17.000000000 +0200
@@ -21,8 +21,8 @@
 
 BUILTBINARIES=($(rpm -q --qf "%{name}\n" --specfile $DIR_TO_CHECK/*.spec))
 # add 'targettype' as a 'known keyword' to not trip over it
-BUILTBINARIES+=('targettype')
-BASELIBSREF=$(grep -o "^\\S\+" $DIR_TO_CHECK/baselibs.conf)
+BUILTBINARIES+=('targettype' 'arch')
+BASELIBSREF=$(grep -o "^[a-zA-Z0-9.+_-]\+" $DIR_TO_CHECK/baselibs.conf)
 
 RETURN=0
 for rpm in $BASELIBSREF; do


Reply via email to