<p>Joshua Colp <strong>merged</strong> this change.</p><p><a href="https://gerrit.asterisk.org/6123">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  Richard Mudgett: Looks good to me, but someone else must approve
  Joshua Colp: Looks good to me, approved; Approved for Submit

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">digium_commits.py:  Handle improperly formatted patch info<br><br>If the patches "submitted by" line was improperly formatted,<br>digium_commits would throw an exception and cause mkrelease.py<br>to crash.  It now handles the bad formatting by simply printing<br>the offending commit message and continuing.  The commit itself<br>is still processed, just without the solution attribution.<br><br>Change-Id: If740bf2f3e7b27cbd044ae853fb9aa5552953a93<br>---<br>M digium_commits.py<br>1 file changed, 8 insertions(+), 1 deletion(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/digium_commits.py b/digium_commits.py<br>index 5727c20..448e119 100644<br>--- a/digium_commits.py<br>+++ b/digium_commits.py<br>@@ -165,7 +165,14 @@<br>             if 'submitted by' in token.lower():<br>                 name = token[token.lower().index('submitted by') + 12:]<br>             if name:<br>-                user = self.extract_user(name)<br>+                user = None<br>+                try:<br>+                    user = self.extract_user(name)<br>+                    break<br>+                except:<br>+                    print("INFO: Unable to parse message for users: ")<br>+                    print(self.message)<br>+<br>                 if user and user not in coders:<br>                     coders.append(user)<br>         if len(coders) == 0:<br></pre><p>To view, visit <a href="https://gerrit.asterisk.org/6123">change 6123</a>. To unsubscribe, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/6123"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: repotools </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: merged </div>
<div style="display:none"> Gerrit-Change-Id: If740bf2f3e7b27cbd044ae853fb9aa5552953a93 </div>
<div style="display:none"> Gerrit-Change-Number: 6123 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Richard Mudgett <rmudgett@digium.com> </div>