Fix mlx-swift-lm dependency to use semantic versioning#91
Conversation
Replace `branch: "main"` with `from: "2.29.3"` for mlx-swift-lm dependency. This fixes SPM dependency resolution failures when consumers use AnyLanguageModel with semantic versioning (e.g., `from: "0.5.3"`). The branch-based dependency is incompatible with version-based transitive dependency resolution.
|
@alexey1312 Thank you for this PR. I believe the guidance from mlx-swift-lm to pin to a semantic tag is relatively new. Or at least is new since they migrated from mlx-swift-examples. I'm just as happy to do it this way to allow for semantic versioning. Taking a look at this now to see what the earliest version is that will work, to make the constraint as loose as possible (and therefore less likely to cause conflicts). |
|
Alright, so apparently mlx-swift-lm introduced some |
Summary
branch: "main"withfrom: "2.29.3"for mlx-swift-lm dependencyProblem
When consumers try to use AnyLanguageModel with semantic versioning:
SPM fails to resolve dependencies because
branch: "main"is incompatible with version-based transitive dependency resolution:Solution
Pin mlx-swift-lm to version
2.29.3(latest stable release) instead of tracking themainbranch.Test plan
swift package resolvesucceedsswift buildsucceeds