<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugzilla.yoctoproject.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugzilla.yoctoproject.org/"
          
          maintainer="it-coreprojects-helpdesk@linuxfoundation.org"
>

    <bug>
          <bug_id>6711</bug_id>
          
          <creation_ts>2014-09-11 17:28:41 +0000</creation_ts>
          <short_desc>SendErrorReport has codepath to a None command</short_desc>
          <delta_ts>2014-09-16 23:14:22 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>5</classification_id>
          <classification>Infrastructure</classification>
          <product>AutoBuilder</product>
          <component>autobuilder</component>
          <version>unspecified</version>
          <rep_platform>Other</rep_platform>
          <op_sys>other</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>Undecided</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Erik Antelman">eantelman</reporter>
          <assigned_to name="Beth Flanagan">elizabeth.flanagan</assigned_to>
          <cc>eantelman</cc>
    
    <cc>elizabeth.flanagan</cc>
    
    <cc>infras.ab.watcher</cc>
    
    <cc>Infras.watcher</cc>
          
          
          <cf_os>---</cf_os>
          <cf_regression_type>---</cf_regression_type>
          
          <cf_docchange>No (bug/feature does not impact docs)</cf_docchange>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>45545</commentid>
    <comment_count>0</comment_count>
    <who name="Erik Antelman">eantelman</who>
    <bug_when>2014-09-11 17:28:41 +0000</bug_when>
    <thetext>If ERROR_REPORT_COLLECT is not defined, it will abort the build by finding a none command. 

In master/HEAD at dc75c7eefba00dac

I think what you want is....

diff --git a/lib/python2.7/site-packages/autobuilder/buildsteps/SendErrorReport.py b/lib/python2.7/site-packages/autobuilder/buildsteps/SendErrorReport.py
index 159c33a..5441f31 100644
--- a/lib/python2.7/site-packages/autobuilder/buildsteps/SendErrorReport.py
+++ b/lib/python2.7/site-packages/autobuilder/buildsteps/SendErrorReport.py
@@ -50,6 +50,8 @@ class SendErrorReport(ShellCommand):
                     self.errorreportserver = &quot;&quot;
                 self.command = &quot;if [ ! -f ~/.oe-send-error ]; then host=`hostname`; echo \&quot;yocto-autobuilder-autogenerated-$host\&quot; &gt; ~/.oe-send-error; fi ; &quot;
                 self.command += &quot;. ./oe-init-build-env; if [ -d &quot; + self.errordir +&quot;/$x&quot; + &quot; ]; then for x in `ls &quot; + self.errordir +&quot; | grep error_report_`; do send-error-report &quot; + sel
+            else:
+                self.command = &quot;echo environment var ERROR_REPORT_COLLECT not set - Step is noop&quot;
             self.description = [&quot;Sending error reports&quot;]
         ShellCommand.start(self)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>45546</commentid>
    <comment_count>1</comment_count>
    <who name="Beth Flanagan">elizabeth.flanagan</who>
    <bug_when>2014-09-11 18:38:21 +0000</bug_when>
    <thetext>Yes, this is the exact fix. Can you send me a signed-off-by patch so I can apply your work? Thanks!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>45547</commentid>
    <comment_count>2</comment_count>
      <attachid>2123</attachid>
    <who name="Erik Antelman">eantelman</who>
    <bug_when>2014-09-11 19:43:57 +0000</bug_when>
    <thetext>Created attachment 2123
Signed-off-by formated patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>45639</commentid>
    <comment_count>3</comment_count>
    <who name="Beth Flanagan">elizabeth.flanagan</who>
    <bug_when>2014-09-16 23:14:22 +0000</bug_when>
    <thetext>http://git.yoctoproject.org/cgit/cgit.cgi/yocto-autobuilder/commit/?id=2bce1599a49725e154c0b092b7f41731827b9283

Thank you for your commit. Pulled into master and production.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>2123</attachid>
            <date>2014-09-11 19:43:57 +0000</date>
            <delta_ts>2014-09-11 19:43:57 +0000</delta_ts>
            <desc>Signed-off-by formated patch</desc>
            <filename>0001-If-ERROR_REPORT_COLLECT-is-not-defined-it-will-abort.patch</filename>
            <type>text/plain</type>
            <size>1510</size>
            <attacher name="Erik Antelman">eantelman</attacher>
            
              <data encoding="base64">RnJvbSA4NzQyMjc4YWFjZGVjMGI3YTI2YzkwMDM2MjE3ZjZlZDlmYjZkNDRjIE1vbiBTZXAgMTcg
MDA6MDA6MDAgMjAwMQpGcm9tOiBFcmlrIEFudGVsbWFuIDxldGE0MjY5QGRyYXBlci5jb20+CkRh
dGU6IFRodSwgMTEgU2VwIDIwMTQgMTU6Mzc6NTMgLTA0MDAKU3ViamVjdDogW1BBVENIXSBJZiBF
UlJPUl9SRVBPUlRfQ09MTEVDVCBpcyBub3QgZGVmaW5lZCwgaXQgd2lsbCBhYm9ydCB0aGUKIGJ1
aWxkIGJ5IHRyeWluZyB0byBydW4gYSBTaGVsbENvbWFuZCB3aXRoIGEgLmNvbW1hbmQgb2YKIE5v
bmUuCgpTaWduZWQtb2ZmLWJ5OiBFcmlrIEFudGVsbWFuIDxldGE0MjY5QGRyYXBlci5jb20+Ci0t
LQogLi4uL2F1dG9idWlsZGVyL2J1aWxkc3RlcHMvU2VuZEVycm9yUmVwb3J0LnB5ICAgICAgfCAg
ICAyICsrCiAxIGZpbGUgY2hhbmdlZCwgMiBpbnNlcnRpb25zKCspCgpkaWZmIC0tZ2l0IGEvbGli
L3B5dGhvbjIuNy9zaXRlLXBhY2thZ2VzL2F1dG9idWlsZGVyL2J1aWxkc3RlcHMvU2VuZEVycm9y
UmVwb3J0LnB5IGIvbGliL3B5dGhvbjIuNy9zaXRlLXBhY2thZ2VzL2F1dG9idWlsZGVyL2J1aWxk
c3RlcHMvU2VuZEVycm9yUmVwb3J0LnB5CmluZGV4IDE1OWMzM2EuLjU0NDFmMzEgMTAwNjQ0Ci0t
LSBhL2xpYi9weXRob24yLjcvc2l0ZS1wYWNrYWdlcy9hdXRvYnVpbGRlci9idWlsZHN0ZXBzL1Nl
bmRFcnJvclJlcG9ydC5weQorKysgYi9saWIvcHl0aG9uMi43L3NpdGUtcGFja2FnZXMvYXV0b2J1
aWxkZXIvYnVpbGRzdGVwcy9TZW5kRXJyb3JSZXBvcnQucHkKQEAgLTUwLDYgKzUwLDggQEAgY2xh
c3MgU2VuZEVycm9yUmVwb3J0KFNoZWxsQ29tbWFuZCk6CiAgICAgICAgICAgICAgICAgICAgIHNl
bGYuZXJyb3JyZXBvcnRzZXJ2ZXIgPSAiIgogICAgICAgICAgICAgICAgIHNlbGYuY29tbWFuZCA9
ICJpZiBbICEgLWYgfi8ub2Utc2VuZC1lcnJvciBdOyB0aGVuIGhvc3Q9YGhvc3RuYW1lYDsgZWNo
byBcInlvY3RvLWF1dG9idWlsZGVyLWF1dG9nZW5lcmF0ZWQtJGhvc3RcIiA+IH4vLm9lLXNlbmQt
ZXJyb3I7IGZpIDsgIgogICAgICAgICAgICAgICAgIHNlbGYuY29tbWFuZCArPSAiLiAuL29lLWlu
aXQtYnVpbGQtZW52OyBpZiBbIC1kICIgKyBzZWxmLmVycm9yZGlyICsiLyR4IiArICIgXTsgdGhl
biBmb3IgeCBpbiBgbHMgIiArIHNlbGYuZXJyb3JkaXIgKyIgfCBncmVwIGVycm9yX3JlcG9ydF9g
OyBkbyBzZW5kLWVycm9yLXJlcG9ydCAiICsgc2VsZi5lcnJvcmRpciArIi8keCIrIHNlbGYuZXJy
b3JyZXBvcnRzZXJ2ZXIgKyAiOyBkb25lOyBmaSIgCisgICAgICAgICAgICBlbHNlOgorICAgICAg
ICAgICAgICAgIHNlbGYuY29tbWFuZCA9ICJlY2hvIGVudmlyb25tZW50IHZhciBFUlJPUl9SRVBP
UlRfQ09MTEVDVCBub3Qgc2V0IC0gU3RlcCBpcyBub29wIgogICAgICAgICAgICAgc2VsZi5kZXNj
cmlwdGlvbiA9IFsiU2VuZGluZyBlcnJvciByZXBvcnRzIl0KICAgICAgICAgU2hlbGxDb21tYW5k
LnN0YXJ0KHNlbGYpCiAKLS0gCjEuNy45LjUKCg==
</data>

          </attachment>
      

    </bug>

</bugzilla>