The following three tests started failing after https://trac.webkit.org/changeset/271805/webkit landed: [170/1960] webkitpy.common.checkout.checkout_unittest.CommitMessageForThisCommitTest.test_commit_message_for_changelogs_with_different_messages failed: Traceback (most recent call last): File "/Volumes/Data/slave/catalina-release-tests-wk2/build/Tools/Scripts/webkitpy/common/checkout/checkout_unittest.py", line 281, in test_commit_message_for_changelogs_with_different_messages self.assertMultiLineEqual(commit_message.message(), expected_commit_message) AssertionError: u"SECTORDER_FLAGS should be defined in target's xcconfig file, not Base.xcconfig [truncated]... != u"SECTORDER_FLAGS should be defined in target's xcconfig file, not Base.xcconfig [truncated]... SECTORDER_FLAGS should be defined in target's xcconfig file, not Base.xcconfig <http://webkit.org/b/135006> Patch by David Kilzer <ddkilzer@apple.com> on 2014-07-17 Reviewed by Darin Adler. + Source/WebKitLegacy: - Source/WebKitLegacy: * WebKit.xcodeproj/project.pbxproj: Remove references to unused WebKit.xcconfig file. LayoutTests: + Filler change. * Path/To/Complicated/File: Added. [232/1960] webkitpy.common.checkout.checkout_unittest.CommitMessageForThisCommitTest.test_commit_message_for_changelogs_with_same_messages failed: Traceback (most recent call last): File "/Volumes/Data/slave/catalina-release-tests-wk2/build/Tools/Scripts/webkitpy/common/checkout/checkout_unittest.py", line 253, in test_commit_message_for_changelogs_with_same_messages self.assertMultiLineEqual(commit_message.message(), expected_commit_message) AssertionError: u"SECTORDER_FLAGS should be defined in target's xcconfig file, not Base.xcconfig [truncated]... != u"SECTORDER_FLAGS should be defined in target's xcconfig file, not Base.xcconfig [truncated]... SECTORDER_FLAGS should be defined in target's xcconfig file, not Base.xcconfig <http://webkit.org/b/135006> Patch by David Kilzer <ddkilzer@apple.com> on 2014-07-17 Reviewed by Darin Adler. + Source/WebKitLegacy: - Source/WebKitLegacy: * WebKit.xcodeproj/project.pbxproj: Remove references to unused WebKit.xcconfig file. LayoutTests: + * Path/To/Complicated/File: Added. [281/1960] webkitpy.common.checkout.checkout_unittest.CommitMessageForThisCommitTest.test_commit_message_for_unreviewed_changelogs_with_different_messages failed: Traceback (most recent call last): File "/Volumes/Data/slave/catalina-release-tests-wk2/build/Tools/Scripts/webkitpy/common/checkout/checkout_unittest.py", line 207, in test_commit_message_for_unreviewed_changelogs_with_different_messages self.assertMultiLineEqual(commit_message.message(), expected_commit_message) AssertionError: u'Unreviewed build fix to un-break webkit-patch land.\n\n\nTools:\nMove commit_m [truncated]... != u'Unreviewed build fix to un-break webkit-patch land.\n\nTools:\n\nMove commit_m [truncated]... Unreviewed build fix to un-break webkit-patch land. + Tools: - Tools: Move commit_message_for_this_commit from scm to checkout https://bugs.webkit.org/show_bug.cgi?id=36629 * Scripts/webkitpy/common/checkout/api.py: import scm.CommitMessage LayoutTests: + Second part of this complicated change by me, Fr\xe9d\xe9ric Wang! * Path/To/Complicated/File: Added.
Sounds like we should roll out the change and re-land it with test fixes. Why didn’t hose failures show up in EWS?
Reverted r271805 with https://trac.webkit.org/changeset/271865.
<rdar://problem/73590660>
(In reply to Darin Adler from comment #1) > Sounds like we should roll out the change and re-land it with test fixes. > Why didn’t hose failures show up in EWS? webkitpy EWS tests only get triggered for changes in specific directories, apparently commit-log-editor is not included. Filed https://bugs.webkit.org/show_bug.cgi?id=220953
I'm sorry for breaking the tests. Thank you very much.