[Collab-qa-commits] r2703 - collab-qa-tools/lib/collab-qa

[Lucas Nussbaum]

Author: lucas
Date: 2013-05-28 19:08:24 +0000 (Tue, 28 May 2013)
New Revision: 2703

Modified:
   collab-qa-tools/lib/collab-qa/log-parser.rb
Log:
make PERSON a local variable to avoid double-assignments errors

Modified: collab-qa-tools/lib/collab-qa/log-parser.rb
===================================================================
--- collab-qa-tools/lib/collab-qa/log-parser.rb	2013-05-28 07:12:03 UTC (rev 2702)
+++ collab-qa-tools/lib/collab-qa/log-parser.rb	2013-05-28 19:08:24 UTC (rev 2703)
@@ -166,7 +166,7 @@
     end
 
     def to_mail(date, fullname, email, bugtype, filename = "")
-      PERSON = "lucas"
+      person = "lucas"
       sdate = date.gsub(/\//, '')
       bfooter = <<-EOF
 A list of current common problems and possible solutions is available at 
@@ -196,7 +196,7 @@
 > #{@extract.map { |l| l.gsub(/^\s+/,'') }.join("\n> ")}
 #{@comment != nil ? "\n" + @comment.rstrip + "\n": ''}
 The full build log is available from:
- http://people.debian.org/~#{PERSON}/logs/#{date}/#{@package}.log
+ http://people.debian.org/~#{person}/logs/#{date}/#{@package}.log
 
 It is reproducible by installing your package in a clean chroot, using
 the debconf Noninteractive frontend, and priority: critical.
@@ -251,7 +251,7 @@
 > #{@extract.join("\n> ")}
 
 The full build log is available from:
-   http://people.debian.org/~#{PERSON}/logs/#{date}/#{filename}
+   http://people.debian.org/~#{person}/logs/#{date}/#{filename}
 
 This bug is part of the "dash" release goal (see
 http://release.debian.org/lenny/goals.txt). If you intend to fix this
@@ -289,7 +289,7 @@
 > #{@extract.join("\n> ")}
 
 The full build log is available from:
-   http://people.debian.org/~#{PERSON}/logs/#{date}/#{filename}
+   http://people.debian.org/~#{person}/logs/#{date}/#{filename}
 
 It is possible that your package builds fine in sid, but you should make sure
 that your package also builds fine in squeeze before the release.
@@ -324,7 +324,7 @@
 > #{@extract.join("\n> ")}
 
 The full build log is available from:
-   http://people.debian.org/~#{PERSON}/logs/#{date}/#{filename}
+   http://people.debian.org/~#{person}/logs/#{date}/#{filename}
 
 #{bfooter}
 EOF
@@ -351,7 +351,7 @@
 > #{@extract.join("\n> ")}
 
 The full build log is available from:
-   http://people.debian.org/~#{PERSON}/logs/#{date}/#{filename}
+   http://people.debian.org/~#{person}/logs/#{date}/#{filename}
 
 #{bfooter}
 EOF
@@ -378,7 +378,7 @@
 > #{@extract.join("\n> ")}
 
 The full build log is available from:
-   http://people.debian.org/~#{PERSON}/logs/#{date}/#{filename}
+   http://people.debian.org/~#{person}/logs/#{date}/#{filename}
 
 #{bfooter}
 EOF