#1767 check ConfigParser object rather than config path list
Merged 2 years ago by mikem. Opened 2 years ago by julian8628.
julian8628/koji issue/1763  into  master

file modified
+1 -1
@@ -1776,7 +1776,7 @@ 

                      result[name] = value

  

      # Check if the specified profile had a config specified

-     if configs and not got_conf:

+     if config.sections() and not got_conf:

          raise ConfigurationError("no configuration for profile name: %s" % profile_name)

  

      # special handling for cert defaults

Looking at this code now, it seems like we're probably doing the wrong thing with this check when config is missing entirely, but I'll file a separate follow up issue for that.

Commit c146423 fixes this pull-request

Pull-Request has been merged by mikem

2 years ago

Filed #1786 for the follow up

Metadata Update from @julian8628:
- Pull-request tagged with: no_qe

2 years ago