Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add libtorch #348

Closed
wants to merge 5 commits into from
Closed

add libtorch #348

wants to merge 5 commits into from

Conversation

xq114
Copy link
Contributor

@xq114 xq114 commented Mar 30, 2021

No description provided.

@xq114
Copy link
Contributor Author

xq114 commented Mar 30, 2021

对optional的处理出了点问题

@waruqi
Copy link
Member

waruqi commented Mar 31, 2021

对optional的处理出了点问题

更新过了,再试试

packages/n/nvtx/xmake.lua Outdated Show resolved Hide resolved
packages/m/mkl/xmake.lua Outdated Show resolved Hide resolved
on_load("windows|x64", "macosx", "linux", function (package)

-- ensure that git core.longpaths is enabled
os.vrun("git config --global core.longpaths true")
Copy link
Member

Choose a reason for hiding this comment

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

这个会影响用户的全局配置,不要这么做。。另外也不要放到 on_load 里面做。。

回头我看看在 xmake 内部对 local git repo 开启 git config core.longpaths true 试试,全局影响 不好。

Copy link
Member

Choose a reason for hiding this comment

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

这个删了吧,我 dev 上改进过了,git clone 的时候,会去设置 longpaths

git clone -c core.longpaths=true

Copy link
Contributor Author

Choose a reason for hiding this comment

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

这里光在clone的时候设置恐怕不行,报错是在git submodule update --recursive的时候发生的,主要是submodule嵌套才导致路径过长

Copy link
Member

Choose a reason for hiding this comment

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

这里光在clone的时候设置恐怕不行,报错是在git submodule update --recursive的时候发生的,主要是submodule嵌套才导致路径过长

只有 git clone 才支持 core.longpaths=true 参数设置。。

不过我看到 git clone 传递 longpaths 后,clone 下来的repo 的 本地 .git/config 配置里面已经默认生效 longpaths了。。

也就是说 后续的 一切操作 git pull/git submodule 按理应该默认都是 生效 longpaths 的。。

[core]
	longpaths = true

你可以 clone 一个确认下,我觉得现在应该就已经可以了,只需要 clone 时候 设置 longpaths,就能对这个 repo 永久生效。。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

发现不行,并不生效

Copy link
Contributor Author

Choose a reason for hiding this comment

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

另一个思路是安装前获取longpath的设置,装完给他改回去

Copy link
Member

Choose a reason for hiding this comment

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

先放着吧,我晚上再研究下。。按理本地设置肯定能生效才对

Copy link
Member

Choose a reason for hiding this comment

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

另一个思路是安装前获取longpath的设置,装完给他改回去

只能试试这个了,今天尝试了下 git submodule foreach --recurisve git config core.langpaths true 去对每个 submodule 更新本地 longpaths 设置,也无效,蛋疼~

我后两天会提供个 set_policy("platform.longpaths", true) 的包接口设置,专门用来解决 longpaths 问题。。

@waruqi
Copy link
Member

waruqi commented Apr 2, 2021

我dev上改进过了,等 这个 https://github.com/xmake-io/xmake-repo/actions/runs/712143551 跑完看下效果

之后跟进这个 patch #357

这里我先关了

@waruqi waruqi closed this Apr 2, 2021
@waruqi
Copy link
Member

waruqi commented Apr 3, 2021

ci上过了,我merge了

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