Skip to content

Commit

Permalink
Make revs sync
Browse files Browse the repository at this point in the history
There were some unclosed socket warnings on 3.12 that I can't
immediately track down.  I'm slowly moving away from using async for
this project as it is troublesome to parallelize in the `deps`
subcommand, troublesome to test, and doesn't add a lot of value for
commands that just check one project.
  • Loading branch information
thatch committed Feb 6, 2024
1 parent 2e04611 commit b4f899e
Showing 1 changed file with 4 additions and 5 deletions.
9 changes: 4 additions & 5 deletions honesty/cmdline.py
Original file line number Diff line number Diff line change
Expand Up @@ -566,19 +566,18 @@ def current_versions_callback(p: str) -> Optional[str]:
@click.option("--try-order", default="likely_tags,tags,branches", show_default=True)
@click.option("--fresh", "-f", is_flag=True)
@click.argument("package_names", nargs=-1)
@wrap_async
async def revs(
def revs(
verbose: bool, url_only: bool, fresh: bool, try_order: str, package_names: List[str]
) -> None:
async with Cache(fresh_index=fresh) as cache:
with Cache(fresh_index=fresh) as cache:
for package_name in package_names:
url = None
if "@" in package_name:
package_name, url = package_name.split("@")

package_name, operator, version = package_name.partition("==")
try:
package = await async_parse_index(package_name, cache, use_json=True)
package = parse_index(package_name, cache, use_json=True)
except Exception as e:
print(f"{package_name}: error {e}")
continue
Expand Down Expand Up @@ -610,7 +609,7 @@ async def revs(
]
type_suffix = "wheel"

lp = await cache.async_fetch(pkg=package_name, url=sdists[0].url)
lp = cache.fetch(pkg=package_name, url=sdists[0].url)

# TODO: More than just *.py...
archive_root, names = extract_and_get_names(
Expand Down

0 comments on commit b4f899e

Please sign in to comment.