X-Git-Url: http://git.scottworley.com/pinch/blobdiff_plain/b3ea39b882d1f48781edff1863181274753b32b5..55ae4ff693dee4323804eae4232141fb2e524c7e:/pinch.py?ds=sidebyside diff --git a/pinch.py b/pinch.py index 58f7a32..4c2341f 100644 --- a/pinch.py +++ b/pinch.py @@ -22,8 +22,12 @@ from typing import ( Dict, Iterable, List, + Mapping, + NamedTuple, NewType, Tuple, + Type, + Union, ) # Use xdg module when it's less painful to have as a dependency @@ -86,19 +90,38 @@ class ChannelTableEntry(types.SimpleNamespace): url: str -class SearchPath(types.SimpleNamespace, ABC): +class AliasPin(NamedTuple): + pass + + +class GitPin(NamedTuple): + git_revision: str release_name: str + +class ChannelPin(NamedTuple): + git_revision: str + release_name: str + tarball_url: str + tarball_sha256: str + + +Pin = Union[AliasPin, GitPin, ChannelPin] + + +class SearchPath(types.SimpleNamespace, ABC): + @abstractmethod - def pin(self, v: Verification, conf: configparser.SectionProxy) -> None: + def pin(self, v: Verification) -> Pin: pass class AliasSearchPath(SearchPath): alias_of: str - def pin(self, v: Verification, conf: configparser.SectionProxy) -> None: + def pin(self, v: Verification) -> AliasPin: assert not hasattr(self, 'git_repo') + return AliasPin() # (This lint-disable is for pylint bug https://github.com/PyCQA/pylint/issues/179 @@ -109,24 +132,23 @@ class TarrableSearchPath(SearchPath, ABC): # pylint: disable=abstract-method forwarded_url: str git_ref: str git_repo: str - git_revision: str old_git_revision: str table: Dict[str, ChannelTableEntry] class GitSearchPath(TarrableSearchPath): - def pin(self, v: Verification, conf: configparser.SectionProxy) -> None: + def pin(self, v: Verification) -> GitPin: if hasattr(self, 'git_revision'): self.old_git_revision = self.git_revision del self.git_revision git_fetch(v, self) - conf['release_name'] = git_revision_name(v, self) - conf['git_revision'] = self.git_revision + return GitPin(release_name=git_revision_name(v, self), + git_revision=self.git_revision) def fetch(self, v: Verification, section: str, conf: configparser.SectionProxy) -> str: - if 'git_repo' not in conf or 'release_name' not in conf: + if 'git_revision' not in conf or 'release_name' not in conf: raise Exception( 'Cannot update unpinned channel "%s" (Run "pin" before "update")' % section) @@ -136,16 +158,21 @@ class GitSearchPath(TarrableSearchPath): class ChannelSearchPath(TarrableSearchPath): - def pin(self, v: Verification, conf: configparser.SectionProxy) -> None: + def pin(self, v: Verification) -> ChannelPin: if hasattr(self, 'git_revision'): self.old_git_revision = self.git_revision del self.git_revision - pin_channel(v, self) - conf['release_name'] = self.release_name - conf['tarball_url'] = self.table['nixexprs.tar.xz'].absolute_url - conf['tarball_sha256'] = self.table['nixexprs.tar.xz'].digest - conf['git_revision'] = self.git_revision + fetch(v, self) + parse_channel(v, self) + fetch_resources(v, self) + ensure_git_rev_available(v, self) + check_channel_contents(v, self) + return ChannelPin( + release_name=self.release_name, + tarball_url=self.table['nixexprs.tar.xz'].absolute_url, + tarball_sha256=self.table['nixexprs.tar.xz'].digest, + git_revision=self.git_revision) # Lint TODO: Put tarball_url and tarball_sha256 in ChannelSearchPath # pylint: disable=no-self-use @@ -278,7 +305,7 @@ def fetch_with_nix_prefetch_url( return path # type: ignore # (for old mypy) -def fetch_resources(v: Verification, channel: TarrableSearchPath) -> None: +def fetch_resources(v: Verification, channel: ChannelSearchPath) -> None: for resource in ['git-revision', 'nixexprs.tar.xz']: fields = channel.table[resource] fields.absolute_url = urllib.parse.urljoin( @@ -538,14 +565,6 @@ def check_channel_contents( v.ok() -def pin_channel(v: Verification, channel: TarrableSearchPath) -> None: - fetch(v, channel) - parse_channel(v, channel) - fetch_resources(v, channel) - ensure_git_rev_available(v, channel) - check_channel_contents(v, channel) - - def git_revision_name(v: Verification, channel: TarrableSearchPath) -> str: v.status('Getting commit date') process = subprocess.run(['git', @@ -564,11 +583,12 @@ def git_revision_name(v: Verification, channel: TarrableSearchPath) -> str: def read_search_path(conf: configparser.SectionProxy) -> SearchPath: - if 'alias_of' in conf: - return AliasSearchPath(**dict(conf.items())) - if 'channel_url' in conf: - return ChannelSearchPath(**dict(conf.items())) - return GitSearchPath(**dict(conf.items())) + mapping: Mapping[str, Type[SearchPath]] = { + 'alias': AliasSearchPath, + 'channel': ChannelSearchPath, + 'git': GitSearchPath, + } + return mapping[conf['type']](**dict(conf.items())) def read_config(filename: str) -> configparser.ConfigParser: @@ -589,7 +609,7 @@ def read_config_files( return merged_config -def pin(args: argparse.Namespace) -> None: +def pinCommand(args: argparse.Namespace) -> None: v = Verification() config = read_config(args.channels_file) for section in config.sections(): @@ -598,13 +618,13 @@ def pin(args: argparse.Namespace) -> None: sp = read_search_path(config[section]) - sp.pin(v, config[section]) + config[section].update(sp.pin(v)._asdict()) with open(args.channels_file, 'w') as configfile: config.write(configfile) -def update(args: argparse.Namespace) -> None: +def updateCommand(args: argparse.Namespace) -> None: v = Verification() exprs: Dict[str, str] = {} config = read_config_files(args.channels_file) @@ -646,11 +666,11 @@ def main() -> None: parser_pin = subparsers.add_parser('pin') parser_pin.add_argument('channels_file', type=str) parser_pin.add_argument('channels', type=str, nargs='*') - parser_pin.set_defaults(func=pin) + parser_pin.set_defaults(func=pinCommand) parser_update = subparsers.add_parser('update') parser_update.add_argument('--dry-run', action='store_true') parser_update.add_argument('channels_file', type=str, nargs='+') - parser_update.set_defaults(func=update) + parser_update.set_defaults(func=updateCommand) args = parser.parse_args() args.func(args)