Skip to content

Introduce vcpkg-workspace - #54

Open
SineStriker wants to merge 8 commits into
mainfrom
vcpkg-workspace
Open

SineStriker wants to merge 8 commits into
mainfrom
vcpkg-workspace

Conversation

@SineStriker

Copy link
Copy Markdown
Contributor

No description provided.

@CrSjimo

CrSjimo commented Oct 25, 2026

Copy link
Copy Markdown
Member

我审了一下,感觉有这些问题:

  1. 现在这个拉取vcpkg-workspace.cmake的机制有点别扭,我希望能弄成类似gradle wrapper那样,就是你提供一个专门用来下载的脚本,而且不是cmake脚本,是平台的shell脚本,纳入仓库版本管理。然后在再仓库里弄一个配置文件,或者直接用现成的配置文件,指定vcpkg-workspace.cmake本身从哪里获取。
  2. 现在的pr里vcpkg仓库,overlay仓库,workspace脚本似乎都没有固定,用的都是main,我希望固定一下。
  3. 如果你是想做成类似npm那样的专门管理工具,我其实建议你也提供一个GitHub Actions的action,让这个action去把初始化,依赖安装,缓存机制都管理好,在构建的workflow里可以直接调用,你研究一下

另外我建议你把图标的这个拆成另一个pr

@SineStriker

Copy link
Copy Markdown
Contributor Author
  1. 这个跟 cc 讨论过,结论是不适合,再搞一份下载脚本就是纯套娃,而且多两份要维护的,直接在readme里写好下载脚本就完事了
  2. 这个pr完了你自己设一个可以
  3. 可以我拆分一下

@SineStriker SineStriker changed the title Introduce vcpkg-workspace and qiconmaker Introduce vcpkg-workspace Oct 25, 2026
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