From 10ddbaff0197f0ef93a910d7ea3f9f7d0ebac3a4 Mon Sep 17 00:00:00 2001 From: Scott Worley Date: Thu, 28 May 2020 23:52:01 -0700 Subject: [PATCH] Factor out fetch_channel Appeases pylint's "Too many branches" --- pinch.py | 36 ++++++++++++++++++++---------------- 1 file changed, 20 insertions(+), 16 deletions(-) diff --git a/pinch.py b/pinch.py index f84440c..10481e4 100644 --- a/pinch.py +++ b/pinch.py @@ -497,6 +497,25 @@ def pin(args: argparse.Namespace) -> None: config.write(configfile) +def fetch_channel( + v: Verification, + section: str, + conf: configparser.SectionProxy) -> str: + if 'git_repo' not in conf or 'release_name' not in conf: + raise Exception( + 'Cannot update unpinned channel "%s" (Run "pin" before "update")' % + section) + + if 'channel_url' in conf: + return fetch_with_nix_prefetch_url( + v, conf['tarball_url'], Digest16( + conf['tarball_sha256'])) + + channel = Channel(**dict(conf.items())) + ensure_git_rev_available(v, channel) + return git_get_tarball(v, channel) + + def update(args: argparse.Namespace) -> None: v = Verification() config = configparser.ConfigParser() @@ -504,25 +523,10 @@ def update(args: argparse.Namespace) -> None: configs = [read_config(filename) for filename in args.channels_file] for config in configs: for section in config.sections(): - if 'alias_of' in config[section]: assert 'git_repo' not in config[section] continue - - if 'git_repo' not in config[section] or 'release_name' not in config[section]: - raise Exception( - 'Cannot update unpinned channel "%s" (Run "pin" before "update")' % - section) - - if 'channel_url' in config[section]: - tarball = fetch_with_nix_prefetch_url( - v, config[section]['tarball_url'], Digest16( - config[section]['tarball_sha256'])) - else: - channel = Channel(**dict(config[section].items())) - ensure_git_rev_available(v, channel) - tarball = git_get_tarball(v, channel) - + tarball = fetch_channel(v, section, config[section]) if section in exprs: raise Exception('Duplicate channel "%s"' % section) exprs[section] = ( -- 2.44.1