Skip to content

fix(toolchain): delete 'share/terminfo' for recent linux python toolchains#1898

Merged
aignas merged 2 commits intobazel-contrib:mainfrom
aignas:fix/exclude-terminfo
May 14, 2024
Merged

fix(toolchain): delete 'share/terminfo' for recent linux python toolchains#1898
aignas merged 2 commits intobazel-contrib:mainfrom
aignas:fix/exclude-terminfo

Conversation

@aignas
Copy link
Collaborator

@aignas aignas commented May 14, 2024

This affects Linux toolchains that have the terminfo databases bundled
with the toolchain. Our solution to this is to remove the
share/terminfo altogether if we are downloading an affected linux
toolchain.

Tested with (on a Mac):

bazel build --platforms=//tests/support:linux_x86_64 @python_3_11//:files
bazel build --platforms=//tests/support:windows_x86_64 @python_3_11//:files

Workaround astral-sh/python-build-standalone#231
Fixes #1800

…hains

This affects Linux toolchains that have the `terminfo` databases bundled
with the toolchain. Our solution to this is to remove the
`share/terminfo` altogether if we are downloading an affected `linux`
toolchain.

Workaround astral-sh/python-build-standalone#231
Fixes bazel-contrib#1800
@aignas aignas requested review from f0rmiga and rickeylev as code owners May 14, 2024 01:47
@aignas aignas enabled auto-merge May 14, 2024 02:07
@aignas aignas added this pull request to the merge queue May 14, 2024
Merged via the queue into bazel-contrib:main with commit df55823 May 14, 2024
chasezheng added a commit to chasezheng/rules_python that referenced this pull request Sep 22, 2024
chasezheng added a commit to chasezheng/rules_python that referenced this pull request Sep 22, 2024
chasezheng added a commit to chasezheng/rules_python that referenced this pull request Sep 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Too many levels of symbolic links at share/terminfo/N/NCR260VT300WPP

2 participants