dnf-json: disable pylint warnings
The _dnfrepo is full of branches, turn off warning. The keyfile doesn't use 'with' because the file needs to remain available when the function exits. Cleanup of persistdir will clean up the temporary file used for the key.
This commit is contained in:
parent
d5e666cc9c
commit
356f261bcd
1 changed files with 4 additions and 0 deletions
4
dnf-json
4
dnf-json
|
|
@ -56,6 +56,7 @@ class Solver():
|
|||
self.base.repos.add(self._dnfrepo(repo, self.base.conf))
|
||||
self.base.fill_sack(load_system_repo=False)
|
||||
|
||||
# pylint: disable=too-many-branches
|
||||
@staticmethod
|
||||
def _dnfrepo(desc, parent_conf=None):
|
||||
"""Makes a dnf.repo.Repo out of a JSON repository description"""
|
||||
|
|
@ -98,6 +99,9 @@ class Solver():
|
|||
|
||||
for key in desc["gpgkeys"]:
|
||||
if key.startswith("-----BEGIN PGP PUBLIC KEY BLOCK-----"):
|
||||
# Not using with because it needs to be a valid file for the duration. It
|
||||
# is inside the temporary persistdir so will be cleaned up on exit.
|
||||
# pylint: disable=consider-using-with
|
||||
keyfile = tempfile.NamedTemporaryFile(dir=keydir, delete=False)
|
||||
keyfile.write(key.encode("utf-8"))
|
||||
repo.gpgkey.append(f"file://{keyfile.name}")
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue