Skip to content

Commit

Permalink
Skip non-existing libflutters for symbol check (flutter#6374)
Browse files Browse the repository at this point in the history
  • Loading branch information
goderbauer authored Sep 28, 2018
1 parent 0cd9ee3 commit 7dc6b3a
Showing 1 changed file with 8 additions and 12 deletions.
20 changes: 8 additions & 12 deletions testing/symbols/verify_exported.dart
Original file line number Diff line number Diff line change
Expand Up @@ -49,13 +49,11 @@ int _checkIos(String outPath, String nmPath, Iterable<String> builds) {
int failures = 0;
for (String build in builds) {
final String libFlutter = p.join(outPath, build, 'Flutter.framework', 'Flutter');
if (!new File(libFlutter).existsSync()) {
print('SKIPPING: $libFlutter does not exist.');
continue;
}
final ProcessResult nmResult = Process.runSync(nmPath, <String>['-gUm', libFlutter]);
print('+++ DEBUG: stdout of nm +++');
print(nmResult.stdout);
print('+++ END: stdout of nm +++');
print('+++ DEBUG: stderr of nm +++');
print(nmResult.stderr);
print('+++ END: stdout of nm +++');
if (nmResult.exitCode != 0) {
print('ERROR: failed to execute "nm -gUm $libFlutter":\n${nmResult.stderr}');
failures++;
Expand Down Expand Up @@ -83,13 +81,11 @@ int _checkAndroid(String outPath, String nmPath, Iterable<String> builds) {
int failures = 0;
for (String build in builds) {
final String libFlutter = p.join(outPath, build, 'libflutter.so');
if (!new File(libFlutter).existsSync()) {
print('SKIPPING: $libFlutter does not exist.');
continue;
}
final ProcessResult nmResult = Process.runSync(nmPath, <String>['-gU', libFlutter]);
print('+++ DEBUG: stdout of nm +++');
print(nmResult.stdout);
print('+++ END: stdout of nm +++');
print('+++ DEBUG: stderr of nm +++');
print(nmResult.stderr);
print('+++ END: stdout of nm +++');
if (nmResult.exitCode != 0) {
print('ERROR: failed to execute "nm -gU $libFlutter":\n${nmResult.stderr}');
failures++;
Expand Down

0 comments on commit 7dc6b3a

Please sign in to comment.