Treat security headers as boring for the purposes of API change reviewHEADdev

Treat the new header as if it were part of the copyright header.

Pick-to: 6.10
Task-number: QTBUG-135177
Change-Id: I6a21d0a966d05308a5d35641561c8c9287d8c03f
Reviewed-by: Cristian Maureira-Fredes <[email protected]>

1 files changed, 9 insertions, 2 deletions

diff --git a/scripts/api-review/resetboring.py b/scripts/api-review/resetboring.py
index 29f345b4..69225e6b 100755
--- a/scripts/api-review/resetboring.py
+++ b/scripts/api-review/resetboring.py
@@ -114,10 +114,17 @@ class Selector(object): # Select interesting changes, discard boring.
     # scanning *this* file aren't mislead by them !
     @staticmethod
     def __end_copyright(seq, oldMarker='_'.join(('$QT', 'END', 'LICENSE$')),
-                        spdxHeader='-'.join(('SPDX', 'License', 'Identifier:'))):
-        """Line number just after the end of the copyright banner"""
+                        spdxHeader='-'.join(('SPDX', 'License', 'Identifier:')),
+                        secHeader='-'.join(('Qt', 'Security'))):
+        """Line number just after the end of the copyright banner.
+
+        For these purposes, treat the security header as part of the banner. It
+        can be changed without affecting source or binary compatibility.
+        """
         for i, line in enumerate(seq, 1):
             if spdxHeader in line:
+                if i < len(seq) and secHeader in seq[i]:
+                    return i + 1
                 return i
             if oldMarker in line:
                 return i + 2