#136 honor smtp_host option in koji-gc
Merged 7 years ago by mikem. Opened 7 years ago by mikem.
https://github.com/mikem23/koji-playground.git koji-gc-smtp-host  into  master

honor smtp_host option in koji-gc (jflorian)
Mike McLean • 7 years ago  
util/koji-gc
file modified
+1 -2
@@ -433,9 +433,8 @@

          if options.debug:

              print "Sending warning notice to %s" % msg['To']

          s = smtplib.SMTP(options.smtp_host)

-         s.connect()

          s.sendmail(msg['From'], msg['To'], msg.as_string())

-         s.close()

+         s.quit()

  

  

  def main(args):

no initial comment

This looks good to me. It brings the gc smtp code in line with the smtp code elsewhere in koji.

:thumbsup:

Looks good to me, too. Nice and simple! :thumbsup:

Commit e3043f2 fixes this pull-request

Pull-Request has been merged by mikem@redhat.com

7 years ago
Metadata