Issue 4364: Allow ImageMagick's compare to exit with status 1

[lilypond.git] / scripts / build / output-distance.py
diff --git a/scripts/build/output-distance.py b/scripts/build/output-distance.py

index d9933efa083fe2792bd9b60ff7939400fb907482..49a659b73febd69fff0f79f01b0536b016cf7c1c 100755 (executable)
--- a/scripts/build/output-distance.py
+++ b/scripts/build/output-distance.py
@@ -89,7 +89,7 @@ def compare_png_images (old, new, dest_dir):
      system ('convert -depth 8 -crop %dx%d+0+0 %s %s/crop1.png' % (dims + (old, dir)))
      system ('convert -depth 8 -crop %dx%d+0+0 %s %s/crop2.png' % (dims + (new, dir)))
  
-    system ('compare -depth 8 -dissimilarity-threshold 1 %(dir)s/crop1.png %(dir)s/crop2.png %(dir)s/diff.png' % locals ())
+    system1 ('compare -depth 8 -dissimilarity-threshold 1 %(dir)s/crop1.png %(dir)s/crop2.png %(dir)s/diff.png' % locals ())
  
      system ("convert  -depth 8 %(dir)s/diff.png -blur 0x3 -negate -channel alpha,blue -type TrueColorMatte -fx 'intensity'    %(dir)s/matte.png" % locals ())
  
@@ -827,7 +827,7 @@ class ComparisonData:
                  re.sub (r'\\sourcefilename "([^"]+)"',
                          note_original, open (sf).read ())
              else:
-                print 'no source for', val
+                print 'no source for', val.file_names[1]
  
      def compare_trees (self, dir1, dir2):
          self.compare_directories (dir1, dir2)
@@ -988,9 +988,12 @@ class ComparisonData:
  
          summary += '<p>%d unchanged</p>' % len (unchanged)
  
+        me = sys.argv[0]
+
          html = '''<html>
  <head>
  <title>LilyPond regression test results</title>
+<meta name="author" content="This file was autogenerated by %(me)s">
  <script language="javascript" type="text/javascript">
  // <![CDATA[
      var rows = document.getElementsByTagName("tr");
@@ -1086,6 +1089,12 @@ def system (x):
      stat = os.system (x)
      assert stat == 0
  
+def system1 (x):
+# Allow exit status 0 and 1
+    print 'invoking', x
+    stat = os.system (x)
+    assert (stat == 0) or (stat == 256) # This return value convention is sick.
+
  
  def test_paired_files ():
      print paired_files (os.environ["HOME"] + "/src/lilypond/scripts/",