Skip to content

支持用路径列表覆盖 mxsml 动态库搜索路径#13

Open
ghangz wants to merge 2 commits into
MetaX-MACA:mainfrom
ghangz:mengz/mxsml-library-path-list-env
Open

支持用路径列表覆盖 mxsml 动态库搜索路径#13
ghangz wants to merge 2 commits into
MetaX-MACA:mainfrom
ghangz:mengz/mxsml-library-path-list-env

Conversation

@ghangz

@ghangz ghangz commented Jun 25, 2026

Copy link
Copy Markdown

单一路径环境变量在多版本 SDK 共存时不够用。这个改动让 MXSML_LIBRARY_PATH 支持系统路径分隔符列表,可以按优先级提供多个候选目录或文件,适合在 CI、容器和本地开发环境之间复用同一套配置。

这次改动保持默认行为兼容,只在明确需要的场景下引入新的诊断、配置入口或结果输出,避免把沐曦适配相关的问题拖到更晚的运行阶段才暴露出来。

我已经在沐曦 GPU 环境里完成了对应分支验证,并补充或执行了以下相关测试:

  • pkg/lib/search_paths_test.go

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request introduces support for configuring multiple SDK search paths via the MXSML_LIBRARY_PATH environment variable. It refactors the library loading logic to dynamically resolve and deduplicate library paths, expanding directory entries to include libmxsml.so while maintaining fallback defaults. Additionally, comprehensive unit tests have been added to verify path list expansion and deduplication. There are no review comments, and I have no feedback to provide.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

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.

1 participant