Skip to content

Commit 7ad2963

Browse files
jaracohoodmane
authored andcommitted
Extract is_valid filter on pip package entries with no URL.
1 parent cb676ca commit 7ad2963

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

micropip/_commands/freeze.py

+6-6
Original file line numberDiff line numberDiff line change
@@ -42,23 +42,23 @@ def freeze_data() -> dict[str, Any]:
4242
def load_pip_packages() -> Iterator[tuple[str, dict[str, Any]]]:
4343
return map(
4444
package_item,
45-
filter(None, map(load_pip_package, importlib.metadata.distributions())),
45+
filter(is_valid, map(load_pip_package, importlib.metadata.distributions())),
4646
)
4747

4848

4949
def package_item(entry: dict[str, Any]) -> tuple[str, dict[str, Any]]:
5050
return canonicalize_name(entry["name"]), entry
5151

5252

53-
def load_pip_package(dist: importlib.metadata.Distribution) -> dict[str, Any] | None:
53+
def is_valid(entry: dict[str, Any]) -> bool:
54+
return entry["file_name"] is not None
55+
56+
57+
def load_pip_package(dist: importlib.metadata.Distribution) -> dict[str, Any]:
5458
name = dist.name
5559
version = dist.version
5660
url = dist.read_text("PYODIDE_URL")
57-
if url is None:
58-
return
59-
6061
sha256 = dist.read_text("PYODIDE_SHA256")
61-
assert sha256
6262
imports = (dist.read_text("top_level.txt") or "").split()
6363
requires = dist.read_text("PYODIDE_REQUIRES")
6464
if not requires:

0 commit comments

Comments
 (0)