koji

#1296 extract read_config_files util for config parsing
Merged by tkopecek. Opened by julian8628.
julian8628/koji issue/865  into  master

Download 1296.patch

fixes: #865
based on PR#866 and PR#1150

  • extend read_config_files with strict option and directory support
  • use ConfigParser for all invokings except kojixmlrpc

:thumbsup:

rebased onto 7ea77b549e969fdb6f2178c9319cac6f543c0c94

1 new commit added

  • fix test_runroot_builder tests

change ConfigParser to RawConfigParser for kojid, related to PR #1543

cc @tkopecek

@julian8628 Can you rebase this one?

rebased onto 8d5525e47b9dc7108d2c7aa6c77473694a09c90a

rebased

Metadata Update from @tkopecek:
- Pull-request tagged with: testing-ready

rebased onto 8120e03d40cadf06dd3e1ad3eda6fd97627b98fa

add some tweaks
- remove default argument. The first config in config_files is actually the default one
- drop strict option for all config files and dirs
- change config_files argument, it could be a list of string and tuple, the second item in the tuple is actualyl strict option for this config file/dir(the 1st item).

In previous code, "global" strict option is too strict.

:thumbsup:

:thumbsup:

Commit c05d8ea9 fixes this pull-request

Pull-Request has been merged by tkopecek
Metadata
None

testing-ready

Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.