Hello community,

here is the log from the commit of package build-compare for openSUSE:Factory 
checked in at 2016-03-21 12:44:14
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/build-compare (Old)
 and      /work/SRC/openSUSE:Factory/.build-compare.new (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "build-compare"

Changes:
--------
--- /work/SRC/openSUSE:Factory/build-compare/build-compare.changes      
2016-01-16 11:55:37.000000000 +0100
+++ /work/SRC/openSUSE:Factory/.build-compare.new/build-compare.changes 
2016-03-21 12:44:16.000000000 +0100
@@ -1,0 +2,35 @@
+Tue Mar  8 18:00:40 UTC 2016 - [email protected]
+
+- Remove hacks from /var/adm/update-scripts|messages case
+
+-------------------------------------------------------------------
+Tue Mar  8 16:48:48 UTC 2016 - [email protected]
+
+- Handle /var/adm/update-messages in scripts and filelist
+
+-------------------------------------------------------------------
+Mon Mar  7 11:56:28 UTC 2016 - [email protected]
+
+- Whitespace in javadoc case
+- Whitespace changes in html case
+- Move case kde/gtk html up in the case list
+- Move case rdoc html up in the case list
+- Add option -a to same-build-result.sh and loop through all rpms
+- Rework sorting of rpm files in same-build-result.sh
+- Better diagnostic if number of subpackages changes
+- Create tmpfiles earlier in same-build-result.sh
+- Handle all javadoc html files
+
+-------------------------------------------------------------------
+Fri Mar  4 18:50:32 UTC 2016 - [email protected]
+
+- Handle yet another variant of elc timestamps
+
+-------------------------------------------------------------------
+Thu Mar  3 12:44:04 UTC 2016 - [email protected]
+
+- Handle timestamp in dvi files
+- Handle Last-modified HTML META tag
+- Handle yet another variant of javadoc timestamps
+
+-------------------------------------------------------------------

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

Other differences:
------------------
++++++ build-compare.spec ++++++
--- /var/tmp/diff_new_pack.ZqZ2iY/_old  2016-03-21 12:44:17.000000000 +0100
+++ /var/tmp/diff_new_pack.ZqZ2iY/_new  2016-03-21 12:44:17.000000000 +0100
@@ -21,7 +21,7 @@
 License:        GPL-2.0+
 Group:          Development/Tools/Building
 Url:            https://github.com/openSUSE/build-compare
-Version:        20160104T085658.0b929c8
+Version:        20160308T194108.4a952dd
 Release:        0
 Source1:        COPYING
 Source2:        same-build-result.sh

++++++ functions.sh ++++++
--- /var/tmp/diff_new_pack.ZqZ2iY/_old  2016-03-21 12:44:17.000000000 +0100
+++ /var/tmp/diff_new_pack.ZqZ2iY/_new  2016-03-21 12:44:17.000000000 +0100
@@ -26,6 +26,7 @@
   sed -e "
   
/\(\/boot\|\/lib\/modules\|\/lib\/firmware\|\/usr\/src\|$version_release_old_regex_l\$\)/{s,$version_release_old_regex_l,@VERSION@-@RELEASE_LONG@,g;s,$version_release_old_regex_s,@VERSION@-@RELEASE_SHORT@,g}
   
s/\(\/var\/adm\/update-scripts\/\)${name_ver_rel_old_regex_l}\([^[:blank:]]\+\)/\1@NAME_VER_REL@\2/g
+  
s/\(\/var\/adm\/update-messages\/\)${name_ver_rel_old_regex_l}\([^[:blank:]]\+\)/\1@NAME_VER_REL@\2/g
   "
 }
 function trim_release_new()
@@ -33,16 +34,17 @@
   sed -e "
   
/\(\/boot\|\/lib\/modules\|\/lib\/firmware\|\/usr\/src\|$version_release_new_regex_l\$\)/{s,$version_release_new_regex_l,@VERSION@-@RELEASE_LONG@,g;s,$version_release_new_regex_s,@VERSION@-@RELEASE_SHORT@,g}
   
s/\(\/var\/adm\/update-scripts\/\)${name_ver_rel_new_regex_l}\([^[:blank:]]\+\)/\1@NAME_VER_REL@\2/g
+  
s/\(\/var\/adm\/update-messages\/\)${name_ver_rel_new_regex_l}\([^[:blank:]]\+\)/\1@NAME_VER_REL@\2/g
   "
 }
 # Get single directory or filename with long or short release string
 function grep_release_old()
 {
-  grep -E 
"(/boot|/lib/modules|/lib/firmware|/usr/src)/[^/]*(${version_release_old_regex_l}(\$|[^/]+\$)|${version_release_old_regex_s}(\$|[^/]+\$))"
+  grep -E 
"(/boot|/lib/modules|/lib/firmware|/usr/src|/var/adm/update-scripts)/[^/]*(${version_release_old_regex_l}(\$|[^/]+\$)|${version_release_old_regex_s}(\$|[^/]+\$))"
 }
 function grep_release_new()
 {
-  grep -E 
"(/boot|/lib/modules|/lib/firmware|/usr/src)/[^/]*(${version_release_new_regex_l}(\$|[^/]+\$)|${version_release_new_regex_s}(\$|[^/]+\$))"
+  grep -E 
"(/boot|/lib/modules|/lib/firmware|/usr/src|/var/adm/update-scripts)/[^/]*(${version_release_new_regex_l}(\$|[^/]+\$)|${version_release_new_regex_s}(\$|[^/]+\$))"
 }
 
 function check_provides()

++++++ pkg-diff.sh ++++++
--- /var/tmp/diff_new_pack.ZqZ2iY/_old  2016-03-21 12:44:17.000000000 +0100
+++ /var/tmp/diff_new_pack.ZqZ2iY/_new  2016-03-21 12:44:17.000000000 +0100
@@ -440,6 +440,31 @@
         perl -e "open fh, '+<', 'old/$file'; seek fh, 4, SEEK_SET; print fh 
'0000';"
         perl -e "open fh, '+<', 'new/$file'; seek fh, 4, SEEK_SET; print fh 
'0000';"
         ;;
+      *.dvi)
+      # Opcodes 247: pre; i[1], num[4], den[4], mag[4], k[1], x[k]
+        perl -e "
+        my \$rec;
+        open fh, '+<', 'old/$file';
+        my \$dummy = read fh, \$rec, 15;
+        (\$pre, \$i, \$num, \$den, \$mag, \$k) = unpack('C2 N3 C', \$rec);
+        seek fh, 15, SEEK_SET;
+        while (\$k > 0) {
+          print fh '0';
+          \$k--;
+        }
+        "
+        perl -e "
+        my \$rec;
+        open fh, '+<', 'new/$file';
+        my \$dummy = read fh, \$rec, 15;
+        (\$pre, \$i, \$num, \$den, \$mag, \$k) = unpack('C2 N3 C', \$rec);
+        seek fh, 15, SEEK_SET;
+        while (\$k > 0) {
+          print fh '0';
+          \$k--;
+        }
+        "
+      ;;
      *.bz2)
         bunzip2 -c old/$file > old/${file/.bz2/}
         bunzip2 -c new/$file > new/${file/.bz2/}
@@ -474,69 +499,85 @@
          sed -i -e "s,POT-Creation-Date: ....-..-.. 
..:..+....,POT-Creation-Date: 1970-01-01 00:00+0000," $f
        done
        ;;
-     /usr/share/doc/packages/*/*.html|\
-     /usr/share/doc/packages/*/*/*.html|\
-     /usr/share/doc/*/html/*.html|\
-     /usr/share/doc/kde/HTML/*/*/*.html)
-       for f in old/$file new/$file; do
+    */rdoc/files/*.html)
+      # ruby documentation
+      # <td>Mon Sep 20 19:02:43 +0000 2010</td>
+      for f in old/$file new/$file; do
+        sed -i -e 's%<td>[A-Z][a-z][a-z] [A-Z][a-z][a-z] [0-9]\+ 
[0-9]\+:[0-9]\+:[0-9]\+ +0000 201[0-9]</td>%<td>Mon Sep 20 19:02:43 +0000 
2010</td>%g' $f
+      done
+      strip_numbered_anchors
+    ;;
+    
/usr/share/doc/kde/HTML/*/*/index.cache|/usr/share/doc/kde/HTML/*/*/*/index.cache|\
+    /usr/share/gtk-doc/html/*/*.html|/usr/share/gtk-doc/html/*/*.devhelp2)
+      # various kde and gtk packages
+      strip_numbered_anchors
+    ;;
+    /usr/share/doc/packages/*/*.html|\
+    /usr/share/doc/packages/*/*/*.html|\
+    /usr/share/doc/*/html/*.html|\
+    /usr/share/doc/kde/HTML/*/*/*.html)
+      for f in old/$file new/$file; do
         sed -i -e '
-        s|<!-- Created on [^,]\+, [0-9]\+ [0-9]\+ by texi2html [0-9\.]\+ 
-->|<!-- Created on July, 14 2015 by texi2html 1.78 -->|
-        s|<!-- Created on [^,]\+, [0-9]\+ by texi2html [0-9\.]\+$|<!-- Created 
on October 1, 2015 by texi2html 5.0|
-        s|^<!-- Created on .*, 20.. by texi2html .\...|<!-- Created on August 
7, 2009 by texi2html 1.82|
-        s|This document was generated by <em>Autobuild</em> on <em>[^,]\+, 
[0-9]\+ [0-9]\+</em> using <a 
href="http://www.nongnu.org/texi2html/";><em>texi2html [0-9\.]\+</em></a>.|This 
document was generated by <em>Autobuild</em> on <em>July, 15 2015</em> using <a 
href="http://www.nongnu.org/texi2html/";><em>texi2html 1.78</em></a>.|
-        s|^ *This document was generated by <em>Autobuild</em> on <em>.*, 
20..</em> using <a href="http://www.nongnu.org/texi2html/";><em>texi2html 
.\...</em></a>.$|  This document was generated by <em>Autobuild</em> on 
<em>August 7, 2009</em> using <a 
href="http://www.nongnu.org/texi2html/";><em>texi2html 1.82</em></a>.|
-        s|^ *This document was generated on <i>[a-zA-Z]\+ [0-9]\+, [0-9]\+</i> 
using <a href="http://www.nongnu.org/texi2html/";><i>texi2html 
[0-9\.]\+</i></a>.|  This document was generated on <i>October 1, 2015</i> 
using <a href="http://www.nongnu.org/texi2html/";><i>texi2html 5.0</i></a>.|
-        s|Generated on ... ... [0-9]* [0-9]*:[0-9][0-9]:[0-9][0-9] 
20[0-9][0-9] for |Generated on Mon May 10 20:45:00 2010 for |
-        s|Generated on ... ... [0-9]* 20[0-9][0-9] 
[0-9]*:[0-9][0-9]:[0-9][0-9] for |Generated on Mon May 10 20:45:00 2010 for |
-        ' $f
-       done
-       strip_numbered_anchors
-       ;;
-     /usr/share/javadoc/*.html |\
-     /usr/share/javadoc/*/*.html|/usr/share/javadoc/*/*/*.html)
-       strip_numbered_anchors
-       # There are more timestamps in html, so far we handle only some 
primitive versions.
-       for f in old/$file new/$file; do
-         # Javadoc:
-          # <head>
-          # <!-- Generated by javadoc (version 1.7.0_75) on Tue Feb 03 
02:20:12 GMT 2015 -->
-          # <!-- Generated by javadoc on Tue Feb 03 00:02:48 GMT 2015 -->
-          # <meta name="date" content="2015-02-03">
-          # </head>
-          sed -i -e '
-            /^<head>/{
-              : next
-              n
-              /^<\/head>/{
-                b end_head
-              }
-              s/^\(<!-- Generated by javadoc\) \((\(build\|version\) 
[0-9._]\+) on ... ... .. ..:..:.. \(GMT\|UTC\) ....\) \(-->\)/\1 
some-date-removed-by-build-compare \5/
-              t next
-              s/^\(<!-- Generated by javadoc\) \(on ... ... .. ..:..:.. 
\(GMT\|UTC\) ....\) \(-->\)/\1 some-date-removed-by-build-compare \3/
-              t next
-              s/^<meta name="date" 
content="[0-9]\{4\}-[0-9]\{2\}-[0-9]\{2\}">/<meta name="date" 
content="some-date-removed-by-build-compare">/
-              b next
-            }
-            : end_head
+          s|META NAME="Last-modified" CONTENT="[^"]\+"|META 
NAME="Last-modified" CONTENT="Thu Mar  3 10:32:44 2016"|
+          s|<!-- Created on [^,]\+, [0-9]\+ [0-9]\+ by texi2html [0-9\.]\+ 
-->|<!-- Created on July, 14 2015 by texi2html 1.78 -->|
+          s|<!-- Created on [^,]\+, [0-9]\+ by texi2html [0-9\.]\+$|<!-- 
Created on October 1, 2015 by texi2html 5.0|
+          s|^<!-- Created on .*, 20.. by texi2html .\...|<!-- Created on 
August 7, 2009 by texi2html 1.82|
+          s|This document was generated by <em>Autobuild</em> on <em>[^,]\+, 
[0-9]\+ [0-9]\+</em> using <a 
href="http://www.nongnu.org/texi2html/";><em>texi2html [0-9\.]\+</em></a>.|This 
document was generated by <em>Autobuild</em> on <em>July, 15 2015</em> using <a 
href="http://www.nongnu.org/texi2html/";><em>texi2html 1.78</em></a>.|
+          s|^ *This document was generated by <em>Autobuild</em> on <em>.*, 
20..</em> using <a href="http://www.nongnu.org/texi2html/";><em>texi2html 
.\...</em></a>.$|  This document was generated by <em>Autobuild</em> on 
<em>August 7, 2009</em> using <a 
href="http://www.nongnu.org/texi2html/";><em>texi2html 1.82</em></a>.|
+          s|^ *This document was generated on <i>[a-zA-Z]\+ [0-9]\+, 
[0-9]\+</i> using <a href="http://www.nongnu.org/texi2html/";><i>texi2html 
[0-9\.]\+</i></a>.|  This document was generated on <i>October 1, 2015</i> 
using <a href="http://www.nongnu.org/texi2html/";><i>texi2html 5.0</i></a>.|
+          s|Generated on ... ... [0-9]* [0-9]*:[0-9][0-9]:[0-9][0-9] 
20[0-9][0-9] for |Generated on Mon May 10 20:45:00 2010 for |
+          s|Generated on ... ... [0-9]* 20[0-9][0-9] 
[0-9]*:[0-9][0-9]:[0-9][0-9] for |Generated on Mon May 10 20:45:00 2010 for |
           ' $f
-         # Gjdoc HtmlDoclet:
-        sed -i -e 's%Generated by Gjdoc HtmlDoclet [0-9,.]*, part of <a 
href="http://www.gnu.org/software/classpath/cp-tools/"; title="" 
target="_top">GNU Classpath Tools</a>, on .*, 20.. [0-9]*:..:.. \(a\|p\)\.m\. 
GMT.%Generated by Gjdoc.%' $f
-        sed -i -e 's%<!DOCTYPE html PUBLIC "-//gnu.org///DTD XHTML 1.1 plus 
Target 1.0//EN"\(.*\)GNU Classpath Tools</a>, on [A-Z][a-z]* [0-9]*, 20?? 
[0-9]*:??:?? \(a|p\)\.m\. GMT.</p>%<!DOCTYPE html PUBLIC "-//gnu.org///DTD 
XHTML 1.1 plus Target 1.0//EN"\1GNU Classpath Tools</a>, on January 1, 2009 
0:00:00 a.m. GMT.</p>%' $f
-        sed -i -e 's%<!DOCTYPE html PUBLIC "-//gnu.org///DTD\(.*GNU Classpath 
Tools</a>\), on [a-zA-Z]* [0-9][0-9], 20.. [0-9]*:..:.. \(a\|p\)\.m\. 
GMT.</p>%<!DOCTYPE html PUBLIC "-//gnu.org///DTD\1,on May 1, 2010 1:11:42 p.m. 
GMT.</p>%' $f
-        # deprecated-list is randomly ordered, sort it for comparison
-        case $f in
-          */deprecated-list.html)
-            sort -o $f $f
-            ;;
-        esac
-       done
-       ;;
-     /usr/share/javadoc/gjdoc.properties |\
-     /usr/share/javadoc/*/gjdoc.properties)
-       for f in old/$file new/$file; do
-        sed -i -e 's|^#[A-Z][a-z]\{2\} [A-Z][a-z]\{2\} [0-9]\{2\} ..:..:.. GMT 
20..$|#Fri Jan 01 11:27:36 GMT 2009|' $f
-       done
-       ;;
+      done
+      strip_numbered_anchors
+    ;;
+    /usr/*/javadoc/*.html)
+      strip_numbered_anchors
+      # There are more timestamps in html, so far we handle only some 
primitive versions.
+      for f in old/$file new/$file; do
+        # Javadoc:
+        # <head>
+        # <!-- Generated by javadoc (version 1.7.0_75) on Tue Feb 03 02:20:12 
GMT 2015 -->
+        # <!-- Generated by javadoc on Tue Feb 03 00:02:48 GMT 2015 -->
+        # <!-- Generated by javadoc (1.8.0_72) on Thu Mar 03 12:50:28 GMT 2016 
-->
+        # <meta name="date" content="2015-02-03">
+        # </head>
+        sed -i -e '
+          /^<head>/{
+          : next
+          n
+          /^<\/head>/{
+          b end_head
+          }
+          s/^<!-- Generated by javadoc ([0-9._]\+) on ... ... .. ..:..:.. 
\(GMT\|UTC\) .... -->/<!-- Generated by javadoX (1.8.0_72) on Thu Mar 03 
12:50:28 GMT 2016 -->/
+          t next
+          s/^\(<!-- Generated by javadoc\) \((\(build\|version\) [0-9._]\+) on 
... ... .. ..:..:.. \(GMT\|UTC\) ....\) \(-->\)/\1 
some-date-removed-by-build-compare \5/
+          t next
+          s/^\(<!-- Generated by javadoc\) \(on ... ... .. ..:..:.. 
\(GMT\|UTC\) ....\) \(-->\)/\1 some-date-removed-by-build-compare \3/
+          t next
+          s/^<meta name="date" 
content="[0-9]\{4\}-[0-9]\{2\}-[0-9]\{2\}">/<meta name="date" 
content="some-date-removed-by-build-compare">/
+          b next
+          }
+          : end_head
+          ' $f
+        # Gjdoc HtmlDoclet:
+        sed -i -e 's%Generated by Gjdoc HtmlDoclet [0-9,.]*, part of <a 
href="http://www.gnu.org/software/classpath/cp-tools/"; title="" 
target="_top">GNU Classpath Tools</a>, on .*, 20.. [0-9]*:..:.. \(a\|p\)\.m\. 
GMT.%Generated by Gjdoc.%' $f
+        sed -i -e 's%<!DOCTYPE html PUBLIC "-//gnu.org///DTD XHTML 1.1 plus 
Target 1.0//EN"\(.*\)GNU Classpath Tools</a>, on [A-Z][a-z]* [0-9]*, 20?? 
[0-9]*:??:?? \(a|p\)\.m\. GMT.</p>%<!DOCTYPE html PUBLIC "-//gnu.org///DTD 
XHTML 1.1 plus Target 1.0//EN"\1GNU Classpath Tools</a>, on January 1, 2009 
0:00:00 a.m. GMT.</p>%' $f
+        sed -i -e 's%<!DOCTYPE html PUBLIC "-//gnu.org///DTD\(.*GNU Classpath 
Tools</a>\), on [a-zA-Z]* [0-9][0-9], 20.. [0-9]*:..:.. \(a\|p\)\.m\. 
GMT.</p>%<!DOCTYPE html PUBLIC "-//gnu.org///DTD\1,on May 1, 2010 1:11:42 p.m. 
GMT.</p>%' $f
+        # deprecated-list is randomly ordered, sort it for comparison
+        case $f in
+          */deprecated-list.html)
+            sort -o $f $f
+          ;;
+        esac
+      done
+    ;;
+    /usr/share/javadoc/gjdoc.properties |\
+    /usr/share/javadoc/*/gjdoc.properties)
+      for f in old/$file new/$file; do
+        sed -i -e 's|^#[A-Z][a-z]\{2\} [A-Z][a-z]\{2\} [0-9]\{2\} ..:..:.. GMT 
20..$|#Fri Jan 01 11:27:36 GMT 2009|' $f
+      done
+    ;;
      */fonts.scale|*/fonts.dir|*/encodings.dir)
        for f in old/$file new/$file; do
          # sort files before comparing
@@ -568,7 +609,12 @@
      *.elc)
        # emacs lisp files
        for f in old/$file new/$file; do
-         sed -i -e 's|Compiled by abuild@.* on ... ... .. ..:..:.. 
20..$|compiled by abuild@buildhost on Wed Jul 01 00:00:00 2009|' $f
+         sed -i -e '
+          s|^;;; .ompiled by abuild@.* on ... ... .. ..:..:.. ....|;;; 
compiled by abuild@buildhost on Wed Jul 01 00:00:00 2009|
+          s|^;;; from file .*\.el|;;; from file 
/home/abuild/rpmbuild/BUILD/anthy-9100h/src-util/elc.8411/anthy-azik.el|
+          s|^;;; emacs version .*|;;; emacs version 21.5  (beta34) "kale" 
XEmacs Lucid.|
+          s|^;;; bytecomp version .*|;;; bytecomp version 2.28 XEmacs; 
2009-08-09.|
+          ' $f
        done
        ;;
      /var/lib/texmf/web2c/*/*fmt |\
@@ -590,25 +636,12 @@
          sed -i -e 's|built by abuild@[a-z0-9]* on ... ... [0-9]* 
[0-9]*:[0-9][0-9]:[0-9][0-9] .* 20[0-9][0-9]|built by abuild@build42 on Thu May 
6 11:21:17 UTC 2010|' $f
        done
        ;;
-     
/usr/share/doc/kde/HTML/*/*/index.cache|/usr/share/doc/kde/HTML/*/*/*/index.cache|\
-     /usr/share/gtk-doc/html/*/*.html|/usr/share/gtk-doc/html/*/*.devhelp2)
-       # various kde and gtk packages
-       strip_numbered_anchors
-       ;;
     */created.rid)
        # ruby documentation
        # file just contains a timestamp and nothing else, so ignore it
        echo "Ignore $file"
        return 0
        ;;
-    */rdoc/files/*.html)
-       # ruby documentation
-       # <td>Mon Sep 20 19:02:43 +0000 2010</td>
-       for f in old/$file new/$file; do
-          sed -i -e 's%<td>[A-Z][a-z][a-z] [A-Z][a-z][a-z] [0-9]\+ 
[0-9]\+:[0-9]\+:[0-9]\+ +0000 201[0-9]</td>%<td>Mon Sep 20 19:02:43 +0000 
2010</td>%g' $f
-       done
-       strip_numbered_anchors
-       ;;
     */Linux*Env.Set.sh)
        # LibreOffice files, contains:
        # Generated on: Mon Apr 18 13:19:22 UTC 2011
@@ -622,19 +655,9 @@
       return 0
       ;;
     /var/adm/update-messages/*|/var/adm/update-scripts/*)
-      # encode version-release inside
-      oldfn=`echo "$file"|sed -e s/-$release2/-$release1/;`
-
       # fetchmsttfonts embeds the release number in the update shell script.
-      echo sed -i -e "s/-$release1/-$release2/g;" "old/$oldfn"
-      sed -i -e "s/-$release1/-$release2/g;" "old/$oldfn"
-
-      if ! diff -u old/$oldfn new/$file; then
-           echo "$oldfn is not same as $file"
-           return 1
-      fi
-      echo "$file and $oldfn are same"
-      return 0
+      sed -i "s/${name_ver_rel_old_regex_l}/@NAME_VER_REL@/" old/$file
+      sed -i "s/${name_ver_rel_new_regex_l}/@NAME_VER_REL@/" new/$file
       ;;
     *.ps)
       for f in "old/$file" "new/$file"; do

++++++ same-build-result.sh ++++++
--- /var/tmp/diff_new_pack.ZqZ2iY/_old  2016-03-21 12:44:17.000000000 +0100
+++ /var/tmp/diff_new_pack.ZqZ2iY/_new  2016-03-21 12:44:17.000000000 +0100
@@ -14,7 +14,20 @@
 CMPSCRIPT=${0%/*}/pkg-diff.sh
 SCMPSCRIPT=${0%/*}/srpm-check.sh
 
-check_all=1
+file1=`mktemp`
+file2=`mktemp`
+_x() {
+  rm -f ${file1} ${file2}
+}
+trap _x EXIT
+#
+check_all=
+if test "$1" = "-a"
+then
+  check_all="-a"
+  shift
+fi
+#
 OLDDIR="$1"
 shift
 NEWDIRS="$*"
@@ -32,7 +45,9 @@
 
 if test `find $NEWDIRS -name '*.rpm' -and ! -name '*.delta.rpm' | wc -l` != 
`find $OLDDIR -name '*.rpm' -and ! -name '*.delta.rpm' | wc -l`; then
    echo "different number of subpackages"
-   find $OLDDIR $NEWDIRS -name '*.rpm' -and ! -name '*.delta.rpm'
+   find $OLDDIR  -name '*.rpm' -and ! -name '*.delta.rpm' -print0 | xargs -0 
rpm -qp --qf '%{NAME}\n' | sort > ${file1}
+   find $NEWDIRS -name '*.rpm' -and ! -name '*.delta.rpm' -print0 | xargs -0 
rpm -qp --qf '%{NAME}\n' | sort > ${file2}
+   diff -u ${file1} ${file2}
    exit 1
 fi
 
@@ -59,8 +74,22 @@
 # problem: a package can contain both noarch and arch subpackages, so we have 
to 
 # take care of proper sorting of NEWRPMS, e.g. noarch/x.rpm and x86_64/w.rpm 
since OLDRPMS 
 # has all the packages in a single directory and would sort this as w.rpm, 
x.rpm.
-OLDRPMS=($(find "$OLDDIR" -type f -name \*rpm -a ! -name \*src.rpm  -a ! -name 
\*.delta.rpm|sort|grep -v -- -32bit-|grep -v -- -64bit-|grep -v -- 
'-x86-.*\.ia64\.rpm'))
-NEWRPMS=($(find $NEWDIRS -type f -name \*rpm -a ! -name \*src.rpm -a ! -name 
\*.delta.rpm|sort --field-separator=/ --key=7|grep -v -- -32bit-|grep -v -- 
-64bit-|grep -v -- '-x86-.*\.ia64\.rpm'))
+find $OLDDIR  -type f -name '*.rpm' \
+  -a ! -name '*src.rpm' \
+  -a ! -name '*.delta.rpm' \
+  -a ! -name '*-32bit-*' \
+  -a ! -name '*-64bit-*' \
+  -a ! -name '*-x86-*.ia64.rpm' \
+  > ${file1}
+find $NEWDIRS -type f -name '*.rpm' \
+  -a ! -name '*src.rpm' \
+  -a ! -name '*.delta.rpm' \
+  -a ! -name '*-32bit-*' \
+  -a ! -name '*-64bit-*' \
+  -a ! -name '*-x86-*.ia64.rpm' \
+  > ${file2}
+OLDRPMS=($( sort --field-separator=/ --key=` sed -n '1s@[^/]@@gp' ${file1} | 
wc -c ` ${file1} ))
+NEWRPMS=($( sort --field-separator=/ --key=` sed -n '1s@[^/]@@gp' ${file2} | 
wc -c ` ${file2} ))
 
 # Get version-release from first RPM and keep for rpmlint check
 # Remember to quote the "." for future regexes
@@ -84,11 +113,7 @@
       echo "skipping -debuginfo package"
     ;;
     *)
-      bash $CMPSCRIPT "$opac" "$npac" || SUCCESS=0
-      if test $SUCCESS -eq 0 -a -z "$check_all"; then
-        echo "differences between $opac and $npac"
-        exit 1
-      fi
+      bash $CMPSCRIPT $check_all "$opac" "$npac" || SUCCESS=0
     ;;
   esac
 done
@@ -110,8 +135,6 @@
 
 if test -n "$OTHERDIR"; then
   if test -e $OLDDIR/rpmlint.log -a -e $OTHERDIR/rpmlint.log; then
-    file1=`mktemp`
-    file2=`mktemp`
     echo "comparing $OLDDIR/rpmlint.log and $OTHERDIR/rpmlint.log"
     # Sort the files first since the order of messages is not deterministic
     # Remove release from files


Reply via email to