Skip to content

Commit ac76fda

Browse files
committed
wip
1 parent aad3290 commit ac76fda

2 files changed

Lines changed: 6 additions & 2 deletions

File tree

python/private/pypi/requirements_files_by_platform.bzl

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -140,7 +140,7 @@ def requirements_files_by_platform(
140140

141141
platforms_from_args = _platforms_from_args(extra_pip_args)
142142
if logger:
143-
logger.debug(lambda: "Platforms from pip args: {}".format(platforms_from_args))
143+
logger.debug(lambda: "Platforms from pip args: {} (from {})".format(platforms_from_args, extra_pip_args))
144144

145145
input_platforms = platforms
146146
default_platforms = platforms
@@ -175,7 +175,7 @@ def requirements_files_by_platform(
175175
platform
176176
for filter_or_platform in specifier.split(",")
177177
for platform in (_default_platforms(filter = filter_or_platform, platforms = platforms) if filter_or_platform.endswith("*") else [filter_or_platform])
178-
if platform in input_platforms
178+
if _platform(platform, python_version) in input_platforms
179179
]
180180
for file, specifier in requirements_by_platform.items()
181181
}.items()

tests/pypi/hub_builder/hub_builder_tests.bzl

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -723,6 +723,10 @@ simple==0.0.3 \
723723
"requirements.linux_x86_64.txt": "linux_x86_64",
724724
"requirements.osx_aarch64.txt": "osx_aarch64",
725725
},
726+
target_platforms = [
727+
"linux_x86_64",
728+
"osx_aarch64",
729+
],
726730
),
727731
)
728732
pypi = builder.build()

0 commit comments

Comments
 (0)