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

feat: add symbols for release #455

Merged
merged 5 commits into from
Dec 10, 2024
Merged

feat: add symbols for release #455

merged 5 commits into from
Dec 10, 2024

Conversation

neko-para
Copy link
Contributor

@neko-para neko-para commented Dec 9, 2024

@MaaXYZ/binding-developers macos下, bin目录会多若干个.dSYM目录, 含有库的调试信息, 如果不需要可以移除

已经移动到symbol目录

@neko-para neko-para requested a review from MistEO December 9, 2024 07:19
@MistEO
Copy link
Member

MistEO commented Dec 9, 2024

直接改 Release 了啊,那我在想 Windows 要不要也这么改(

@MistEO
Copy link
Member

MistEO commented Dec 9, 2024

这个 install 目录是哪里控制的,要不像 Windows 一样放到 symbol 文件夹里,这样 binding 就不用都跟着改了(

64d7d14#diff-8c5c5ffd3813b068605bdad312131c1c6736494293abeecd79360b81c03c2bf0

@neko-para
Copy link
Contributor Author

这个 install 目录是哪里控制的,要不像 Windows 一样放到 symbol 文件夹里,这样 binding 就不用都跟着改了(

脚本手动做的, 但是如果不放在边上的话调试的时候lldb就不能自动找到了, 必须手动载入

@MistEO
Copy link
Member

MistEO commented Dec 9, 2024

这个 install 目录是哪里控制的,要不像 Windows 一样放到 symbol 文件夹里,这样 binding 就不用都跟着改了(

脚本手动做的, 但是如果不放在边上的话调试的时候lldb就不能自动找到了, 必须手动载入

调试的时候自己挪一下,举例来说 share 里面那些文件其实也没法用也得自己挪一下(
win 的 pdb 也得挪

@MistEO
Copy link
Member

MistEO commented Dec 9, 2024

linux 和 安卓的 symbol 是啥啊(

@MistEO MistEO changed the title feat: add macos symbols feat: add symbols for release Dec 10, 2024
@MistEO
Copy link
Member

MistEO commented Dec 10, 2024

linux 和 安卓的我研究了下都没搞懂,算了反正现在也没人用,之后有需要再加吧

@MistEO MistEO merged commit 75057d5 into main Dec 10, 2024
18 checks passed
@MistEO MistEO deleted the feat/macos_symbols branch December 10, 2024 01:53
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