-
Notifications
You must be signed in to change notification settings - Fork 283
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 Google Breakpad #801
Conversation
I was getting unresolved reference error:
It was just matter of inculding new pe_file.cc in qmake - 588101d: --- a/src/MEGASync/google_breakpad/google_breakpad.pri
+++ b/src/MEGASync/google_breakpad/google_breakpad.pri
@@ -31,6 +31,7 @@ unix:!macx {
SOURCES += $$PWD/client/linux/handler/exception_handler.cc
SOURCES += $$PWD/client/linux/handler/minidump_descriptor.cc
SOURCES += $$PWD/client/linux/minidump_writer/minidump_writer.cc
+ SOURCES += $$PWD/client/linux/minidump_writer/pe_file.cc
SOURCES += $$PWD/client/linux/minidump_writer/linux_dumper.cc
SOURCES += $$PWD/client/linux/minidump_writer/linux_ptrace_dumper.cc
SOURCES += $$PWD/client/linux/log/log.cc Note, I also had some issues with freeimage library and its neon optimizations. I found out they should be disabled for ARM, however it seems to be already done in mega's SDK:
With that I was able to build everything:
|
It works on Asahi Linux with M2 ❤️ I am sharing compiled binary with you: megasync_v4.9.3_aarch64, virus total scan here. |
Hi I'd appreciate it if you could take a look and help.
|
Despite following the provided steps I was not able to build on a up to date install of asahi linux due to the breakpad issue. |
@koxu1996 Any updates on this? |
I will try to build it again with the latest version today - and submit relevant PR - but this is the last time I am dedicating time to MEGAsync. If maintainers are not interested in supporting ARM (or just keeping dependencies up to date) then I don't want to use mega. |
Well, |
@farid220 @archisman-panigrahi Compiling the new dependency |
@koxu1996 Thank you very much for persistently trying to revive the megasync for ARM. I am sorry that your efforts were not properly appreciated. |
This PR updates Google Breakpad to the latest version.
It should close ARM related issues: #303, #310, #456.
You can reproduce all changes with the following steps:
1. Prepare breakpad
2. Prepare lss
3. Clone MEGAsync repository
4. Update breakpad files - 2751244
5. Apply workaround for
inttypes.h has already been included before this header file, but without __STDC_FORMAT_MACROS defined
error - fbbe423I am not sure about this step, maybe
__STDC_FORMAT_MACROS
should be defined somewhere in megasync SDK 🤔