Commit queue needs to generate a commit message (from the ChangeLog files). For this it needs the find the list of modified ChangeLog files.
Created attachment 392274 [details] Patch
Comment on attachment 392274 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=392274&action=review > Tools/BuildSlaveSupport/ews-build/steps.py:2449 > + def _status_regexp(self, expected_types): This comes from https://trac.webkit.org/browser/webkit/trunk/Tools/Scripts/webkitpy/common/checkout/scm/git.py#L176
Sample runs: https://ews-build.webkit-uat.org/#/builders/26/builds/1042 https://ews-build.webkit-uat.org/#/builders/26/builds/1041 https://ews-build.webkit-uat.org/#/builders/26/builds/1040
Comment on attachment 392274 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=392274&action=review > Tools/BuildSlaveSupport/ews-build/steps.py:2423 > + command = ['git', 'diff', '-r', '--name-status', '--no-renames', '--no-ext-diff', '--full-index'] Commit queue uses git? That surprises me. I would expect it to be native svn. > Tools/BuildSlaveSupport/ews-build/steps.py:2442 > + modified_changelogs = self.extract_changelogs(log_text, self._status_regexp('ADM')) Do we ever expect this to not be 'Added, Deleted, Modified? Might be better to just create and compile the regex ahead of time. > Tools/BuildSlaveSupport/ews-build/steps_unittest.py:3035 > +M Source/WebCore/layout/blockformatting/BlockMarginCollapse.cpp''') + Test added and deleted?
Created attachment 392290 [details] Patch
View in context: https://bugs.webkit.org/attachment.cgi?id=392274&action=review >> Tools/BuildSlaveSupport/ews-build/steps.py:2423 >> + command = ['git', 'diff', '-r', '--name-status', '--no-renames', '--no-ext-diff', '--full-index'] > > Commit queue uses git? That surprises me. I would expect it to be native svn. Yes, all EWS bots (including commit-queue on old ews) use git. e.g.: https://bugs.webkit.org/show_bug.cgi?id=207727#c10 >> Tools/BuildSlaveSupport/ews-build/steps.py:2442 >> + modified_changelogs = self.extract_changelogs(log_text, self._status_regexp('ADM')) > > Do we ever expect this to not be 'Added, Deleted, Modified? Might be better to just create and compile the regex ahead of time. compiling wouldn't change much, this is very simple regex, and used only once per build. >> Tools/BuildSlaveSupport/ews-build/steps_unittest.py:3035 >> +M Source/WebCore/layout/blockformatting/BlockMarginCollapse.cpp''') + > > Test added and deleted? Added unit-tests for ChangeLog addition. We don't need to consider ChangeLog deletion (otherwise the next step of generating commit message will fail for deleted files). Updated steps.py to look at only addition and modification.
Committed r257794: <https://trac.webkit.org/changeset/257794>
<rdar://problem/60007220>
Just for reference, output from old commit-queue when the patch doesn't have any ChangeLog (pretty unreadable): https://bugs.webkit.org/show_bug.cgi?id=208495#c10