-
Notifications
You must be signed in to change notification settings - Fork 6k
Add more logging for UnsatisfiedLinkError #51534
Conversation
| // cpu architecture. | ||
|
|
||
| String cpuArch = System.getProperty("os.arch"); | ||
| String[] nativeLibs = new File(flutterApplicationInfo.nativeLibraryDir).list(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should you filter this list down to just files that end with .so or change the message below on line 208?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll change the message to the following files (I suppose I was assuming only .so files end up there because thats all I saw when I tested it, but that is probably a bad assumption)
|
auto label is removed for flutter/engine/51534, due to - The status or check suite Mac mac_unopt has failed. Please fix the issues identified (or deflake) before re-applying this label. |
| + " for more detail.\n" | ||
| + "App is using cpu architecture: " | ||
| + cpuArch | ||
| + ", and the native libraries directory contains the following files: " |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you include the native library directory path in this message? Or if the files in nativeLibs are absolute paths that no need to do that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Didn't see this before the merge, sorry. The files aren't shown as absolute paths, I'll put up a quick follow up to print the path too
follow up to #51534 (comment) [C++, Objective-C, Java style guides]: https://github.com/flutter/engine/blob/main/CONTRIBUTING.md#style
…145479) flutter/engine@f803f2a...1b52f57 2024-03-20 [email protected] Add more logging for UnsatisfiedLinkError (flutter/engine#51534) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll Please CC [email protected],[email protected],[email protected] on the revert to ensure that a human is aware of the problem. To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose To report a problem with the AutoRoller itself, please file a bug: https://issues.skia.org/issues/new?component=1389291&template=1850622 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
Add additional logging for this error, because there are still no local reproduction cases.
related to flutter/flutter#144291
follow up to #50247
Pre-launch Checklist
///).If you need help, consider asking for advice on the #hackers-new channel on Discord.