-
-
Notifications
You must be signed in to change notification settings - Fork 429
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
update abseil to 20230802.1 #2884
Conversation
20230802.1 这个版本需要打patch |
And it’s really annoying to have this git autocrlf, you have to cut the patch into lf, calculate the hash, and then cut it back. . . |
人家的 patch 不一定格式兼容,这边是用 git apply 的,那应该用 git diff 去生成 patch。。conan 可能用的 patch 命令生成的,格式有差异。 你可以先本地合入补丁后,走 git diff 再输出个新补丁 |
Other people's patches may not necessarily have compatible formats. Here we use git apply, so we should use git diff to generate patches. . conan may be generated using the patch command, and the format is different. You can merge the patch locally first, run git diff and then output a new patch. |
I haven’t used patches much, I thought they were all in the same format. |
Still not working, the operation process |
没问题了 |
No problem |
I don’t know why I specified add_links before. |
There is a question, if |
abseil这个链接顺序好坑,这里调整之后,字符串示例没问题,估计后面实际使用其他模块的时候还会有问题,需要使用者自己设置链接顺序 |
The link order of abseil is a problem. After adjusting here, the string example is fine. It is estimated that there will be problems when other modules are actually used later. Users need to set the link order themselves. |
提一嘴,为什么删掉原来的add_links列表,因为abseil迭代很快,到20230802.1这个版本之后,库名变化很多,特别是internal的。而且还新增了三十几个库 |
Just to mention, why did you delete the original add_links list? Because abseil iterates very quickly. After version 20230802.1, the library name has changed a lot, especially internal. And more than thirty new libraries have been added |
No description provided.