zhangyiss/spack
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

spack repo migrate: fix order of if-elif (#50579)

Browse Source
This commit is contained in:
Harmen Stoppels 2025-05-21 11:31:26 +02:00 committed by GitHub
parent 0107792a9e
commit bfc52d6f50
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 14 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
lib/spack/spack/cmd/repo.py
View File

@ -201,7 +201,19 @@ def repo_migrate(args: Any) -> int:
repo_v2 = None
exit_code = 0
if exit_code == 0 and isinstance(repo_v2, spack.repo.Repo):
if not args.fix:
tty.error(
f"No changes were made to the repository {repo.root} with namespace "
f"'{repo.namespace}'. Run with --fix to apply the above changes."
)
elif exit_code == 1:
tty.error(
f"Repository '{repo.namespace}' could not be migrated to the latest Package API. "
"Please check the error messages above."
)
elif isinstance(repo_v2, spack.repo.Repo):
tty.info(
f"Repository '{repo_v2.namespace}' was successfully migrated from "
f"package API {repo.package_api_str} to {repo_v2.package_api_str}."
@ -212,15 +224,9 @@ def repo_migrate(args: Any) -> int:
f" spack repo add {shlex.quote(repo_v2.root)}"
)
elif exit_code == 0:
else:
tty.info(f"Repository '{repo.namespace}' was successfully migrated")
elif not args.fix and exit_code == 1:
tty.error(
f"No changes were made to the repository {repo.root} with namespace "
f"'{repo.namespace}'. Run with --fix to apply the above changes."
)
return exit_code