On Fri, Mar 26, 2010 at 2:43 PM, John Admanski <[email protected]> wrote:

> This should probably use %r instead of %s. In practice I think repr(args)
> and str(args) will end up being the same, but in principle repr is supposed
> to be the inverse of eval so it's the more correct choice of formatting.


Good point. Updated the patch. PTAL.


>
>
> On Fri, Mar 26, 2010 at 2:40 PM, Darin Petkov <[email protected]> wrote:
>
>> A simple patch to propagate user args (autoserv -a/--args) to the
>> client-side control file.
>>
>>
>
From 25bd8f0033135ac32bab1bc4b35579b1ff5e5541 Mon Sep 17 00:00:00 2001
From: Darin Petkov <[email protected]>
Date: Fri, 26 Mar 2010 14:37:16 -0700
Subject: [PATCH] Propagate user args to the client-side control file.

---
 server/autotest.py |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/server/autotest.py b/server/autotest.py
index c35f4a1..f19e350 100644
--- a/server/autotest.py
+++ b/server/autotest.py
@@ -357,6 +357,9 @@ class BaseAutotest(installable_object.InstallableObject):
         # build up the initialization prologue for the control file
         prologue_lines = []
 
+        # Add the additional user arguments
+        prologue_lines.append("args = %r\n" % self.job._args)
+
         # If the packaging system is being used, add the repository list.
         repos = None
         try:
-- 
1.7.0.1

_______________________________________________
Autotest mailing list
[email protected]
http://test.kernel.org/cgi-bin/mailman/listinfo/autotest

Reply via email to