request_splitter: group_by(): provide required option and use in devel.

Staging packages with no devel project as a result of devel strategy seems
like the incorrect behavior.
This commit is contained in:
Jimmy Berry 2017-04-17 23:05:04 -05:00
parent 232219363c
commit 59eda5d82d

View File

@ -52,8 +52,10 @@ class RequestSplitter(object):
'contains("{requests}", concat(" ", ./action/target/@package, " "))'
.format(requests=requests))
def group_by(self, xpath):
def group_by(self, xpath, required=False):
self.groups.append(ET.XPath(xpath))
if required:
self.filter_add(xpath)
def filter_only(self):
ret = []
@ -401,7 +403,7 @@ class StrategyDevel(StrategyNone):
def apply(self, splitter):
super(StrategyDevel, self).apply(splitter)
splitter.group_by('./action/target/@devel_project')
splitter.group_by('./action/target/@devel_project', True)
def desirable(self, splitter):
groups = []