diff options
author | Joshua M. Boniface <joshua@boniface.me> | 2024-04-07 16:41:26 -0400 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-04-07 16:41:26 -0400 |
commit | b89a12d7d7c8c54ffa872ca52c58b9a01bd38ca1 (patch) | |
tree | b4f8fb4d7a21c15f1377e5b8e01a568d21be24b3 | |
parent | Use buildx build explicitly instead of alias (diff) | |
parent | Refactor arch checking into a function (diff) | |
download | jellyfin-packaging-b89a12d7d7c8c54ffa872ca52c58b9a01bd38ca1.tar.gz jellyfin-packaging-b89a12d7d7c8c54ffa872ca52c58b9a01bd38ca1.tar.bz2 jellyfin-packaging-b89a12d7d7c8c54ffa872ca52c58b9a01bd38ca1.zip |
Merge pull request #13 from s-t-e-v-e-n-k/dedupe-architecture
Refactor arch checking into a function
-rwxr-xr-x | build.py | 55 |
1 files changed, 19 insertions, 36 deletions
@@ -36,6 +36,21 @@ except Exception as e: exit(1) +# Shared functions +def _determine_arch(build_type, build_arch, build_version): + PACKAGE_ARCH = ( + configurations[build_type]["archmaps"][build_arch]["PACKAGE_ARCH"] + if build_arch in configurations[build_type]["archmaps"].keys() + else None + ) + if PACKAGE_ARCH is None: + raise ValueError( + f"{build_arch} is not a valid {build_type} {build_version} architecture in {configurations[build_type]['archmaps'].keys()}" + ) + else: + return PACKAGE_ARCH + + def build_package_deb( jellyfin_version, build_type, build_arch, build_version, local=False ): @@ -60,15 +75,7 @@ def build_package_deb( raise ValueError( f"{build_version} is not a valid {build_type} version in {configurations[build_type]['releases'].keys()}" ) - PACKAGE_ARCH = ( - configurations[build_type]["archmaps"][build_arch]["PACKAGE_ARCH"] - if build_arch in configurations[build_type]["archmaps"].keys() - else None - ) - if PACKAGE_ARCH is None: - raise ValueError( - f"{build_arch} is not a valid {build_type} {build_version} architecture in {configurations[build_type]['archmaps'].keys()}" - ) + PACKAGE_ARCH = _determine_arch(build_type, build_arch, build_version) except Exception as e: log(f"Invalid/unsupported arguments: {e}") exit(1) @@ -124,15 +131,7 @@ def build_linux( log("") try: - PACKAGE_ARCH = ( - configurations[build_type]["archmaps"][build_arch]["PACKAGE_ARCH"] - if build_arch in configurations[build_type]["archmaps"].keys() - else None - ) - if PACKAGE_ARCH is None: - raise ValueError( - f"{build_arch} is not a valid {build_type} {build_version} architecture in {configurations[build_type]['archmaps'].keys()}" - ) + PACKAGE_ARCH = _determine_arch(build_type, build_arch, _build_version) DOTNET_ARCH = configurations[build_type]["archmaps"][build_arch]["DOTNET_ARCH"] except Exception as e: log(f"Invalid/unsupported arguments: {e}") @@ -168,15 +167,7 @@ def build_windows( log("") try: - PACKAGE_ARCH = ( - configurations[build_type]["archmaps"][build_arch]["PACKAGE_ARCH"] - if build_arch in configurations[build_type]["archmaps"].keys() - else None - ) - if PACKAGE_ARCH is None: - raise ValueError( - f"{build_arch} is not a valid {build_type} {build_version} architecture in {configurations[build_type]['archmaps'].keys()}" - ) + PACKAGE_ARCH = _determine_arch(build_type, build_arch, _build_version) DOTNET_ARCH = configurations[build_type]["archmaps"][build_arch]["DOTNET_ARCH"] except Exception as e: log(f"Invalid/unsupported arguments: {e}") @@ -212,15 +203,7 @@ def build_macos( log("") try: - PACKAGE_ARCH = ( - configurations[build_type]["archmaps"][build_arch]["PACKAGE_ARCH"] - if build_arch in configurations[build_type]["archmaps"].keys() - else None - ) - if PACKAGE_ARCH is None: - raise ValueError( - f"{build_arch} is not a valid {build_type} {build_version} architecture in {configurations[build_type]['archmaps'].keys()}" - ) + PACKAGE_ARCH = _determine_arch(build_type, build_arch, _build_version) DOTNET_ARCH = configurations[build_type]["archmaps"][build_arch]["DOTNET_ARCH"] except Exception as e: log(f"Invalid/unsupported arguments: {e}") |