revert

Author: aignas

python/private/pypi/hub_builder.bzl

@@ -536,9 +536,8 @@ def _whl_repo(
             target_platforms = src.target_platforms if is_multiple_versions else []
             return struct(
                 repo_name = pypi_repo_name(
-                    whl_name = normalize_name(src.distribution),
-                    target_platforms = target_platforms,
-                    extras = src.extras,
+                    normalize_name(src.distribution),
+                    *target_platforms
                 ),
                 args = args,
                 config_setting = whl_config_setting(
@@ -568,7 +567,7 @@ def _whl_repo(
         ]
 
     return struct(
-        repo_name = whl_repo_name(src.filename, src.sha256, extras = src.extras),
+        repo_name = whl_repo_name(src.filename, src.sha256),
         args = args,
         config_setting = whl_config_setting(
             version = python_version,

python/private/pypi/parse_requirements.bzl

@@ -274,7 +274,6 @@ def _package_srcs(
                     sha256 = dist.sha256,
                     url = dist.url,
                     yanked = dist.yanked,
-                    extras = requirement(r.requirement_line).extras,
                 ),
             )
 

python/private/pypi/whl_repo_name.bzl

@@ -18,7 +18,7 @@
 load("//python/private:normalize_name.bzl", "normalize_name")
 load(":parse_whl_name.bzl", "parse_whl_name")
 
-def whl_repo_name(filename, sha256, extras = []):
+def whl_repo_name(filename, sha256):
     """Return a valid whl_library repo name given a distribution filename.
 
     Args:
@@ -59,12 +59,9 @@ def whl_repo_name(filename, sha256, extras = []):
     elif version:
         parts.insert(1, version)
 
-    if extras:
-        parts.extend(sorted([e for e in extras if e]))
-
     return "_".join(parts)
 
-def pypi_repo_name(whl_name, target_platforms = [], extras = []):
+def pypi_repo_name(whl_name, *target_platforms):
     """Return a valid whl_library given a requirement line.
 
     Args:
@@ -79,7 +76,4 @@ def pypi_repo_name(whl_name, target_platforms = [], extras = []):
     ]
     parts.extend([p.partition("_")[-1] for p in target_platforms])
 
-    if extras:
-        parts.extend(sorted([e for e in extras if e]))
-
     return "_".join(parts)