Skip to content

Commit

Permalink
Fix dynlib loading issue references
Browse files Browse the repository at this point in the history
  • Loading branch information
ncoghlan committed Oct 23, 2024
1 parent 68eba3f commit 3ab8ffb
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@ Caveats and Limitations
* while the `venvstacks` CLI is broadly stable, there are still some specific areas
where changes may occur (such as in the handling of relative paths).
* dynamic library dependencies across layers currently only work on Windows.
There is a [proposal](https://github.com/lmstudio-ai/venvstacks/issues/1) in
There is a [proposal](https://github.com/lmstudio-ai/venvstacks/issues38) in
place for resolving that limitation, but it has not yet been implemented.
* local exports to filesystems which do not support symlinks (such as `VFAT` and
`FAT32`) are nominally supported (with symlinks being replaced by the files
Expand Down
2 changes: 1 addition & 1 deletion src/venvstacks/stacks.py
Original file line number Diff line number Diff line change
Expand Up @@ -1031,7 +1031,7 @@ def __post_init__(self) -> None:
self.dynlib_path = self.env_path / "Library" / "bin"
else:
# No extra library search paths on non-Windows systems
# (but see https://github.com/lmstudio-ai/venvstacks/issues/1)
# (but see https://github.com/lmstudio-ai/venvstacks/issues/38)
self.dynlib_path = None
self.env_lock = EnvironmentLock(
self.requirements_path,
Expand Down

0 comments on commit 3ab8ffb

Please sign in to comment.