Skip to content

Comments

[runner] Drop buildLDLibraryPathEnv()#3593

Merged
un-def merged 1 commit intomasterfrom
pr_runner_drop_dead_ld_code
Feb 20, 2026
Merged

[runner] Drop buildLDLibraryPathEnv()#3593
un-def merged 1 commit intomasterfrom
pr_runner_drop_dead_ld_code

Conversation

@un-def
Copy link
Collaborator

@un-def un-def commented Feb 19, 2026

This code is:

  • conda-specific
  • dead

Basically, the function (ab)uses python3-config 1 utility (which, BTW, is not present in dstack base images since conda -> uv migration) to calculate the path to conda-installed shared objects and export it via LD_LIBRARY_PATH (the proper way to add conda-installed libs would be to use ld.so's configuration files, that is, ld.so.conf.d/*).

Although this code was added in
#1354, it is not related to TPU support at all.

This code:

- is conda-specific
- is dead
- overwrites user/image-defined LD_LIBRARY_PATH in rare cases
  when it works

Basically, the function (ab)uses python3-config [1] utility (which,
BTW, is not present in dstack base images since conda -> uv migration)
to calculate the path to conda-installed shared objects and export it
via LD_LIBRARY_PATH (the proper way to add conda-installed libs
would be to use ld.so's configuration files, that is, ld.so.conf.d/*).

Although this code was added in
#1354, it is not related to TPU
support at all.

[1]: https://manpages.debian.org/testing/python3-dev/python3-config.1.en.html
@un-def un-def requested a review from r4victor February 19, 2026 13:18
@un-def un-def merged commit c9cf54e into master Feb 20, 2026
28 checks passed
@un-def un-def deleted the pr_runner_drop_dead_ld_code branch February 20, 2026 08:02
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.

2 participants