Skip to content

Commit 8024b33

Browse files
committed
main: ensure config_settings are passed to get_requires_for_build
1 parent 0ab7b05 commit 8024b33

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

src/build/__main__.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -113,7 +113,7 @@ def _build_in_isolated_env(
113113
# first install the build dependencies
114114
env.install(builder.build_system_requires)
115115
# then get the extra required dependencies from the backend (which was installed in the call above :P)
116-
env.install(builder.get_requires_for_build(distribution))
116+
env.install(builder.get_requires_for_build(distribution, config_settings or {}))
117117
return builder.build(distribution, outdir, config_settings or {})
118118

119119

@@ -127,7 +127,7 @@ def _build_in_current_env(
127127
builder = _ProjectBuilder(srcdir)
128128

129129
if not skip_dependency_check:
130-
missing = builder.check_dependencies(distribution)
130+
missing = builder.check_dependencies(distribution, config_settings or {})
131131
if missing:
132132
dependencies = ''.join('\n\t' + dep for deps in missing for dep in (deps[0], _format_dep_chain(deps[1:])) if dep)
133133
_cprint()

tests/test_main.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -135,7 +135,7 @@ def test_build_isolated(mocker, package_test_flit):
135135

136136
install.assert_any_call({'flit_core >=2,<3'})
137137

138-
required_cmd.assert_called_with('sdist')
138+
required_cmd.assert_called_with('sdist', {})
139139
install.assert_any_call(['dep1', 'dep2'])
140140

141141
build_cmd.assert_called_with('sdist', '.', {})

0 commit comments

Comments
 (0)