[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Xen-devel] [OSSTEST PATCH] cs-bisection-step: Limit size of revision log included in reports



There is a limit in cr-daily-branch, but none in cs-bisection-step.

adhoc-revtuple-generator could usefully have this built in but that's
not so simple, so do it again here.  We already slurp the whole thing
into core so from a resource usage point of view we might as well do
the length check here too.

Reported-by: David Vrabel <david.vrabel@xxxxxxxxxx>
Signed-off-by: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
---
 cs-bisection-step |    5 +++++
 1 file changed, 5 insertions(+)

diff --git a/cs-bisection-step b/cs-bisection-step
index e51babd..928a147 100755
--- a/cs-bisection-step
+++ b/cs-bisection-step
@@ -898,6 +898,11 @@ END
                    "./adhoc-revtuple-generator -S @revtuplegenargs $rts";
                 my $revinfo= `$revrune`;
                 if (!$?) {
+                   if (length($revinfo) > 24000) {
+                       $revinfo = <<END;
+(Revision log too long, ommitted.)
+END
+                   }
                     summary_report("",$revinfo,-1);
                 }
             }
-- 
1.7.10.4


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.