Changeset View
Changeset View
Standalone View
Standalone View
swh/loader/package/opam/loader.py
Show First 20 Lines • Show All 124 Lines • ▼ Show 20 Lines | def _compute_versions(self) -> List[str]: | ||||
Returns: | Returns: | ||||
The list of versions for the package | The list of versions for the package | ||||
""" | """ | ||||
# TODO: use `opam show` instead of this workaround when it support the `--repo` | # TODO: use `opam show` instead of this workaround when it support the `--repo` | ||||
# flag | # flag | ||||
package_dir = self.get_package_dir() | package_dir = self.get_package_dir() | ||||
if not os.path.exists(package_dir): | if not os.path.exists(package_dir): | ||||
raise ValueError( | raise ValueError( | ||||
f"can't get versions for package {self.opam_package} " | f"can't get versions for package {self.opam_package} " | ||||
f"(at url {self.url})." | f"(at url {self.url})." | ||||
) | ) | ||||
versions = [ | versions = [ | ||||
".".join(version.split(".")[1:]) for version in os.listdir(package_dir) | ".".join(version.split(".")[1:]) for version in os.listdir(package_dir) | ||||
] | ] | ||||
if not versions: | if not versions: | ||||
raise ValueError( | raise ValueError( | ||||
f"can't get versions for package {self.opam_package} " | f"can't get versions for package {self.opam_package} " | ||||
f"(at url {self.url})" | f"(at url {self.url})" | ||||
) | ) | ||||
Show All 14 Lines | def get_versions(self) -> List[str]: | ||||
# folder | # folder | ||||
call( | call( | ||||
[ | [ | ||||
"opam", | "opam", | ||||
"init", | "init", | ||||
"--reinit", | "--reinit", | ||||
"--bare", | "--bare", | ||||
"--no-setup", | "--no-setup", | ||||
"--yes", | |||||
ardumont: Revert previous commit D7248 (which did not work) | |||||
"--root", | "--root", | ||||
self.opam_root, | self.opam_root, | ||||
self.opam_instance, | self.opam_instance, | ||||
self.opam_url, | self.opam_url, | ||||
] | ] | ||||
) | ) | ||||
else: | else: | ||||
# for standard/production loaders, no need to initialize the opam root | # for standard/production loaders, no need to initialize the opam root | ||||
▲ Show 20 Lines • Show All 84 Lines • Show Last 20 Lines |
Revert previous commit D7248 (which did not work)