-
-
Notifications
You must be signed in to change notification settings - Fork 636
refactor: add rule to do analysis time evaluation of environment markers #2832
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from 2 commits
Commits
Show all changes
28 commits
Select commit
Hold shift + click to select a range
799f9c3
wip: flag to evaluate dep specs
rickeylev a658e8f
add some todo, fake some more values
rickeylev e7ea75b
Merge branch 'main' into pep508.flag
rickeylev e65c973
source values from select and flags
rickeylev 6181509
sorta fix platform_python_implementation
rickeylev bf2173c
wip: analysis tests basic
rickeylev 1e23941
trying to get tests to work
rickeylev ef61724
got basic test passing
rickeylev d1b9c0d
Merge branch 'main' of https://github.com/bazel-contrib/rules_python …
rickeylev 77decbc
parameterize test
rickeylev bda07f7
expand testing a bit
rickeylev 5d37d7d
a few notes and links and continue with impl
aignas 38e6f5b
rename to env_marker_setting
rickeylev be7e007
Merge remote-tracking branch 'origin/pep508.flag' into pep508.flag
rickeylev 3f794c2
move select mappings to constants
rickeylev 7dae5d0
remove extras_flag; various code/comment cleanup
rickeylev 2f1b528
minor cleanup, doc
rickeylev 7ff220e
fixup: forgot to pass args
rickeylev b340de7
code share with pep508_env.bzl, add config_setting,
rickeylev 104c27b
Merge branch 'main' of https://github.com/bazel-contrib/rules_python …
rickeylev b2c73c0
more cleanup
rickeylev ba05d71
more cleanup
rickeylev 5316f2a
remove platform_version and platform_release flags for now
rickeylev 2b30b54
wire platform_release back to osx flag
rickeylev a6bc605
cleanup comment
rickeylev 764879d
Merge branch 'main' of https://github.com/bazel-contrib/rules_python …
rickeylev 72a8f1e
update dleted packages
rickeylev 8a210d2
restore bazelrc
rickeylev File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,97 @@ | ||
load("//python/private:toolchain_types.bzl", "TARGET_TOOLCHAIN_TYPE") | ||
load(":pep508_evaluate.bzl", "evaluate") | ||
|
||
def _impl(ctx): | ||
env = {} | ||
|
||
runtime = ctx.toolchains[TARGET_TOOLCHAIN_TYPE].py3_runtime | ||
if runtime.interpreter_version_info: | ||
rickeylev marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
version_info = runtime.interpreter_version_info | ||
env["python_version"] = "{v.major}.{v.minor}".format(v = version_info) | ||
full_version = format_full_version(version_info) | ||
env["python_full_version"] = full_version | ||
env["implementation_version"] = full_version | ||
else: | ||
env["python_version"] = get_flag(ctx.attr._python_version) | ||
full_version = get_flag(ctx.attr._python_full_version) | ||
env["python_full_version"] = full_version | ||
env["implementation_version"] = full_version | ||
|
||
# We assume cpython if the toolchain doesn't specify because it's most | ||
# likely to be true. | ||
env["implementation_name"] = runtime.implementation_name or "cpython" | ||
|
||
# todo: map from os constraint | ||
env["os_name"] = struct() | ||
rickeylev marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
|
||
# todo: map from os constraint | ||
env["sys_platform"] = struct() | ||
|
||
# todo: map from cpu flag (x86_64, etc) | ||
env["platform_machine"] = struct() | ||
|
||
# todo: add PyRuntimeInfo.platform_python_implementation | ||
# The values are slightly different to implementation_name | ||
env["platform_python_implementation"] = runtime.implementation_name | ||
|
||
# todo: add flag to carry this | ||
env["platform_release"] = struct() | ||
|
||
# todo: map from os constraint | ||
env["platform_system"] = struct() | ||
|
||
# todo: add flag to carry this | ||
env["platform_version"] = struct() | ||
rickeylev marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
|
||
if evalute(ctx.attr.expression, env): | ||
value = "yes" | ||
else: | ||
value = "no" | ||
return [config_common.FeatureFlagInfo(value = value)] | ||
|
||
# Adapted from spec code at: | ||
# https://packaging.python.org/en/latest/specifications/dependency-specifiers/#environment-markers | ||
def format_full_info(info): | ||
kind = info.releaselevel | ||
if kind == "final": | ||
kind = "" | ||
serial = "" | ||
else: | ||
kind = kind[0] if kind else "" | ||
serial = str(info.serial) if info.serial else "" | ||
|
||
return "{v.major}.{v.minor}.{v.micro}{kind}{serial}".format( | ||
v = version_info, | ||
kind = kind, | ||
serial = serial, | ||
) | ||
return version | ||
|
||
pypa_dependency_specification = rule( | ||
implementation = _impl, | ||
attrs = { | ||
"expression": attt.string(), | ||
"_os_name": attr.label(), | ||
"_sys_platform_flag": attr.label(), | ||
"_platform_release_flag": attr.label(), | ||
"_platform_system_flag": attr.label(), | ||
"_platform_version_flag": attr.label(), | ||
"_python_version_flag": attr.label( | ||
default = "//python/config_settings:_python_version_major_minor", | ||
), | ||
"_python_full_version_flag": attr.label( | ||
default = "//python/config_settings:python_version", | ||
), | ||
"_extra_flag": attr.label(), | ||
}, | ||
toolchains = [ | ||
TARGET_TOOLCHAIN_TYPE, | ||
], | ||
) | ||
|
||
def _get_flag(t): | ||
if config_common.FeatureFlagInfo in t: | ||
return t[config_common.FeatureFlagInfo].value | ||
if BuildSettingInfo in t: | ||
return t[BuildSettingInfo].value | ||
fail("Should not occur: {} does not have necessary providers") |
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.