Skip to content
This repository was archived by the owner on Apr 6, 2026. It is now read-only.

Darwin: rewrite Nix store paths#167

Merged
EricLBuehler merged 1 commit into
mainfrom
macos-rewrite-nix-store-paths
Jun 24, 2025
Merged

Darwin: rewrite Nix store paths#167
EricLBuehler merged 1 commit into
mainfrom
macos-rewrite-nix-store-paths

Conversation

@danieldk

Copy link
Copy Markdown
Member

Mirror the rpath stripping that we do on Linux. Without this change, we potentially up end loading two different Torch versions into one Python process on a Nix system.

@danieldk danieldk force-pushed the macos-rewrite-nix-store-paths branch from 9e971ae to cfd4ad1 Compare June 24, 2025 08:47
Mirror the rpath stripping that we do on Linux. Without this change,
we potentially up end loading two different Torch versions into one
Python process on a Nix system.
@danieldk danieldk force-pushed the macos-rewrite-nix-store-paths branch from cfd4ad1 to ca403a4 Compare June 24, 2025 08:49

@EricLBuehler EricLBuehler left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Tested here.

@EricLBuehler EricLBuehler merged commit bfcde3a into main Jun 24, 2025
8 checks passed
@EricLBuehler EricLBuehler deleted the macos-rewrite-nix-store-paths branch June 24, 2025 15:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants