Attachment 922910 Details for Bug 857090 – support for rhtsrequires="..." in <skeleton>

[patch] support for rhtsrequires="..." in <skeleton>

patch (text/plain), 5.56 KB, created by Martin Žember on 2014-07-31 12:53:10 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Martin Žember

Created: 2014-07-31 12:53:10 UTC

Size: 5.56 KB

patch

obsolete

>diff --git a/Client/src/bkr/client/wizard.py b/Client/src/bkr/client/wizard.py
>index 61407dc..f15e2d5 100755
>--- a/Client/src/bkr/client/wizard.py
>+++ b/Client/src/bkr/client/wizard.py
>@@ -1,3 +1,4 @@
>+#!/usr/bin/python
> 
> # This program is free software; you can redistribute it and/or modify
> # it under the terms of the GNU General Public License as published by
>@@ -416,7 +417,7 @@
>         </license>
>     </licenses>
>     <skeletons>
>-        <skeleton name="skel1">
>+        <skeleton name="skel1" requires="gdb" rhtsrequires="lib.sh">
>             This is skeleton 1 example.
>         </skeleton>
>         <skeleton name="skel2">
>@@ -951,6 +952,10 @@ def __init__(self, argv=None, load_user_prefs=True):
>             action="append",
>             metavar="PACKAGES",
>             help="required packages [%s]" % self.pref.getPackage())
>+        groupMeta.add_option("-Q",
>+            dest="rhtsrequires",
>+            action="append",
>+            help="rhts-required libraries")
>         groupMeta.add_option("-t",
>             dest="time",
>             help="test time [%s]" % self.pref.getTime())
>@@ -1054,6 +1059,7 @@ def archs(self):        return [ self.opt.archs,        [] ]
>     def releases(self):     return [ self.opt.releases,     [] ]
>     def runfor(self):       return [ self.opt.runfor,       [self.pref.getPackage()] ]
>     def requires(self):     return [ self.opt.requires,     [self.pref.getPackage()] ]
>+    def rhtsrequires(self): return [ self.opt.rhtsrequires, [] ]
>     def time(self):         return [ self.opt.time,         self.pref.getTime() ]
>     def priority(self):     return [ self.opt.priority,     self.pref.getPriority() ]
>     def confidential(self): return [ self.opt.confidential, self.pref.getConfidential() ]
>@@ -2045,6 +2051,24 @@ def validItem(self, item):
>         return RegExpPackage.match(item)
> 
> 
>+class RhtsRequires(MultipleChoice):
>+    """ List of libraries that the test uses, they must be installed on test system """
>+
>+    def init(self):
>+        self.name = "RhtsRequired packages"
>+        self.question = "RhtsRequires: libraries which test depends on"
>+        self.description = """Just write a list of test libraries
>+                which should be automatically installed on the test system."""
>+        self.list = []
>+        self.sort = True
>+        self.emptyListMeaning = "None"
>+        self.common = False
>+        self.default(self.options.rhtsrequires())
>+
>+    def validItem(self, item):
>+        return RegExpPackage.match(item)
>+
>+
> class Skeleton(SingleChoice):
>     """ Skeleton to be used for creating the runtest.sh """
> 
>@@ -2422,6 +2446,9 @@ def init(self):
>         self.list.extend(findNodeNames(self.options.pref.skeletons, "skeleton"))
>         self.common = False
>         self.default(self.options.skeleton())
>+        self.requires = None
>+        self.rhtsrequires = None
>+
> 
>     def replaceVariables(self, xml, test = None):
>         """ Replace all <variable> tags with their respective values """
>@@ -2469,6 +2496,20 @@ def getRuntest(self, test = None):
>         skeleton = re.sub("\n\s+$", "\n", skeleton)
>         return dedentText(skeleton)
> 
>+    def getRhtsRequires(self):
>+        """ Return packages/libraries listed in the arguments of a skeleton, if any """
>+        # get the template from predefined or user skeletons
>+        skeleton = findNode(self.skeletons, "skeleton", self.data) \
>+                or findNode(self.options.pref.skeletons, "skeleton", self.data)
>+        if not skeleton:
>+            return None
>+        try:
>+            rhtsrequires = skeleton.getAttribute("rhtsrequires")
>+            return rhtsrequires
>+        except:
>+            print "getRhtsRequires exception"
>+
>+
>     def getMakefile(self, type, testname, version, author, reproducers, meta):
>         """ Return Makefile skeleton """
> 
>@@ -2606,6 +2647,8 @@ def init(self):
>         self.author = Author(self.options)
>         self.email = Email(self.options)
> 
>+        self.rhtsrequires = RhtsRequires(self.options, suggest = self.skeleton.getRhtsRequires())
>+
>         # we escape review only in force mode
>         if not self.options.force(): self.confirm = True
>         if not self.confirm: self.format()
>@@ -2634,6 +2677,7 @@ def format(self):
>         print
>         self.runfor.format()
>         self.requires.format()
>+        self.rhtsrequires.format()
>         self.archs.format()
>         self.releases.format()
>         self.version.format()
>@@ -2672,7 +2716,7 @@ def edit(self, checkOnly = False):
> 
>         # check all fields for matching string (and edit if not checking only)
>         for field in self.testname, self.package, self.namespace, self.runfor, \
>-                self.requires, self.package, self.releases, self.version, \
>+                self.requires, self.rhtsrequires, self.package, self.releases, self.version, \
>                 self.time, self.desc, self.destructive, self.archs, \
>                 self.path, self.priority, self.confidential, self.license, \
>                 self.skeleton, self.author, self.email, self.testname.prefix, \
>@@ -2752,7 +2796,8 @@ def formatMakefile(self):
>                 self.type.formatMakefileLine(name = "Type") +
>                 self.time.formatMakefileLine(name = "TestTime") +
>                 self.runfor.formatMakefileLine(name = "RunFor") +
>-                self.requires.formatMakefileLine(name = "Requires") +
>+                self.requires.formatMakefileLine(name = "Requires") + 
>+                self.rhtsrequires.formatMakefileLine(name = "RhtsRequires") +
>                 provides +
>                 self.priority.formatMakefileLine() +
>                 self.license.formatMakefileLine() +

Flags:

mzember: review-

Actions: View | Diff

Attachments on bug 857090: 922910 | 924261 | 924520