From f50e8b221b9d33d2bd8e6eb07fdbef6a46ad9016 Mon Sep 17 00:00:00 2001 From: Adam Cozzette Date: Tue, 27 Dec 2022 18:18:55 -0800 Subject: [PATCH] Update staleness_test() error message to include diff PiperOrigin-RevId: 498069301 --- cmake/staleness_test_lib.py | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/cmake/staleness_test_lib.py b/cmake/staleness_test_lib.py index 171d2be0fc..4887f24855 100644 --- a/cmake/staleness_test_lib.py +++ b/cmake/staleness_test_lib.py @@ -33,6 +33,7 @@ This code is used by test scripts generated from staleness_test() rules. from __future__ import absolute_import from __future__ import print_function +import difflib import sys import os from shutil import copyfile @@ -171,7 +172,10 @@ def CheckFilesMatch(config): continue for pair in stale_files: - diff_errors.append("File %s is out of date" % pair.target) + with open(pair.generated) as g, open(pair.target) as t: + diff = ''.join(difflib.unified_diff(g.read().splitlines(keepends=True), + t.read().splitlines(keepends=True))) + diff_errors.append("File %s is out of date:\n%s" % (pair.target, diff)) if diff_errors: error_msg = "Files out of date!\n\n"