pkgkde-mark-qt5-private-symbols: Do nothing when alternative is not specified.

2 files changed, 9 insertions, 1 deletions

diff --git a/debian/changelog b/debian/changelog
index 2199ee6..4b9b8d7 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -4,6 +4,8 @@ pkg-kde-tools (0.15.21) UNRELEASED; urgency=medium
   * pkgkde-mark-qt5-private-symbols:
     - Make --write-results option the default, add option --print-diff for the
       old behavior.
+    - Do not try to mark private symbols if alternative dependency is not
+      specified in the symbols file.
 
  -- Debian Qt/KDE Maintainers <debian-qt-kde@lists.debian.org>  Wed, 06 Jan 2016 20:52:49 +0300
 
diff --git a/pkgkde-mark-qt5-private-symbols b/pkgkde-mark-qt5-private-symbols
index a547453..56eedb6 100755
--- a/pkgkde-mark-qt5-private-symbols
+++ b/pkgkde-mark-qt5-private-symbols
@@ -36,6 +36,7 @@ from sys import stdout, stderr
 def process_symbols_file(symbols_file_path, write_results=False):
     old_lines = []
     new_lines = []
+    has_alternative = False
     with open(symbols_file_path) as symbols_file:
         for line in symbols_file:
             old_lines.append(line)
@@ -44,8 +45,13 @@ def process_symbols_file(symbols_file_path, write_results=False):
                 line = line[:-2]
             if '@Qt_5_PRIVATE_API' in line:
                 line += ' 1'
+            if line.startswith('| '):
+                has_alternative = True
             new_lines.append(line + '\n')
-    if write_results:
+    if not has_alternative:
+        print('Skipping file %s, no alternative dependency specified.' %
+              symbols_file_path)
+    elif write_results:
         with open(symbols_file_path, 'w') as symbols_file:
             for line in new_lines:
                 symbols_file.write(line)