refactor: use frontend instead of plural
Signed-off-by: Henry Schreiner <henryschreineriii@gmail.com>
This commit is contained in:
@@ -241,7 +241,7 @@ def build_in_container(
|
|||||||
|
|
||||||
verbosity_flags = get_build_verbosity_extra_flags(build_options.build_verbosity)
|
verbosity_flags = get_build_verbosity_extra_flags(build_options.build_verbosity)
|
||||||
extra_flags = split_config_settings(
|
extra_flags = split_config_settings(
|
||||||
build_options.config_settings, plural=build_options.build_frontend == "pip"
|
build_options.config_settings, build_options.build_frontend
|
||||||
)
|
)
|
||||||
|
|
||||||
if build_options.build_frontend == "pip":
|
if build_options.build_frontend == "pip":
|
||||||
|
|||||||
@@ -375,7 +375,7 @@ def build(options: Options, tmp_path: Path) -> None:
|
|||||||
|
|
||||||
verbosity_flags = get_build_verbosity_extra_flags(build_options.build_verbosity)
|
verbosity_flags = get_build_verbosity_extra_flags(build_options.build_verbosity)
|
||||||
extra_flags = split_config_settings(
|
extra_flags = split_config_settings(
|
||||||
build_options.config_settings, plural=build_options.build_frontend == "pip"
|
build_options.config_settings, build_options.build_frontend
|
||||||
)
|
)
|
||||||
|
|
||||||
if build_options.build_frontend == "pip":
|
if build_options.build_frontend == "pip":
|
||||||
|
|||||||
@@ -208,9 +208,9 @@ def get_build_verbosity_extra_flags(level: int) -> list[str]:
|
|||||||
return []
|
return []
|
||||||
|
|
||||||
|
|
||||||
def split_config_settings(config_settings: str, *, plural: bool) -> list[str]:
|
def split_config_settings(config_settings: str, frontend: Literal["pip", "build"]) -> list[str]:
|
||||||
config_settings_list = shlex.split(config_settings)
|
config_settings_list = shlex.split(config_settings)
|
||||||
s = "s" if plural else ""
|
s = "s" if frontend == "pip" else ""
|
||||||
return [f"--config-setting{s}={setting}" for setting in config_settings_list]
|
return [f"--config-setting{s}={setting}" for setting in config_settings_list]
|
||||||
|
|
||||||
|
|
||||||
|
|||||||
@@ -412,7 +412,7 @@ def build(options: Options, tmp_path: Path) -> None:
|
|||||||
|
|
||||||
verbosity_flags = get_build_verbosity_extra_flags(build_options.build_verbosity)
|
verbosity_flags = get_build_verbosity_extra_flags(build_options.build_verbosity)
|
||||||
extra_flags = split_config_settings(
|
extra_flags = split_config_settings(
|
||||||
build_options.config_settings, plural=build_options.build_frontend == "pip"
|
build_options.config_settings, build_options.build_frontend
|
||||||
)
|
)
|
||||||
|
|
||||||
if build_options.build_frontend == "pip":
|
if build_options.build_frontend == "pip":
|
||||||
|
|||||||
@@ -283,13 +283,13 @@ def test_config_settings(platform_specific, platform, intercepted_build_args, mo
|
|||||||
|
|
||||||
assert build_options.config_settings == config_settings
|
assert build_options.config_settings == config_settings
|
||||||
|
|
||||||
assert split_config_settings(config_settings, plural=False) == [
|
assert split_config_settings(config_settings, "build") == [
|
||||||
"--config-setting=setting=value",
|
"--config-setting=setting=value",
|
||||||
"--config-setting=setting=value2",
|
"--config-setting=setting=value2",
|
||||||
"--config-setting=other=something else",
|
"--config-setting=other=something else",
|
||||||
]
|
]
|
||||||
|
|
||||||
assert split_config_settings(config_settings, plural=True) == [
|
assert split_config_settings(config_settings, "pip") == [
|
||||||
"--config-settings=setting=value",
|
"--config-settings=setting=value",
|
||||||
"--config-settings=setting=value2",
|
"--config-settings=setting=value2",
|
||||||
"--config-settings=other=something else",
|
"--config-settings=other=something else",
|
||||||
|
|||||||
Reference in New Issue
Block a user