Improve patches_ignore detection

patches_base was always added to .spec when it contained string
'patches_ignore' (like rdopkg does in changelog) due to an overly relaxed
check.

I used regex to make sure it only triggers for proper patches_ignore comment.

No time was wasted on unit tests because all these hacks must be replaced with
proper magic comment parser without silly expectations and million special
cases.

Fixes: #122

Change-Id: Ieadab8f7549da95b6408533f068471c02c9e39ec

Author: Jakub Ruzicka

rdopkg/utils/specfile.py

@@ -305,8 +305,13 @@ def _create_new_patches_base(self, base):
     def set_patches_base(self, base):
         v, _ = self.get_patches_base()
 
-        if 'patches_ignore' in self.txt and (base is None or base == ''):
-            base = self.get_tag('Version', expand_macros=True)
+        if re.search("^#\s*patches_ignore\s*=\s*\S+", self.txt, flags=re.M):
+            # This is a temporary hack as patches_ignore currently requires
+            # explicit patches_base. This should be solved with a proper
+            # magic comment parser and using Version in filtration logic
+            # when no patches_base is defined.
+            if not base:
+                base = self.get_tag('Version', expand_macros=True)
         if base:
             regex = r'^#\s*patches_base*'
             if v is None and re.search(regex, self.txt, flags=re.M) is None: