diff --git a/tools/generate_package_config.dart b/tools/generate_package_config.dart index 7b9a07d9369..882c827b95b 100644 --- a/tools/generate_package_config.dart +++ b/tools/generate_package_config.dart @@ -92,22 +92,20 @@ void main(List args) { packages.sort((a, b) => a["name"].compareTo(b["name"])); var configFile = File(p.join(repoRoot, '.dart_tool', 'package_config.json')); + var json = + jsonDecode(configFile.readAsStringSync()) as Map; + var oldPackages = json['packages'] as List; // Validate the packages entry only, to avoid spurious failures from changes // in the dates embedded in the other entries. - if (checkOnly) { - var json = - jsonDecode(configFile.readAsStringSync()) as Map; - var oldPackages = json['packages'] as List; - if (jsonEncode(packages) == jsonEncode(oldPackages)) { - print("Package config up to date"); - exit(0); - } else { - print("Package config out of date"); - print("Run `gclient sync -D && dart tools/generate_package_config.dart` " - "to update."); - exit(1); - } + if (jsonEncode(packages) == jsonEncode(oldPackages)) { + print("Package config up to date"); + exit(0); + } else if (checkOnly) { + print("Package config out of date"); + print("Run `gclient sync -D && dart tools/generate_package_config.dart` " + "to update."); + exit(1); } var year = DateTime.now().year; @@ -130,8 +128,8 @@ void main(List args) { }; // TODO(rnystrom): Consider using package_config_v2 to generate this instead. - var json = JsonEncoder.withIndent(' ').convert(config); - configFile.writeAsStringSync('$json\n'); + var jsonString = JsonEncoder.withIndent(' ').convert(config); + configFile.writeAsStringSync('$jsonString\n'); print('Generated .dart_tool/package_config.dart containing ' '${packages.length} packages.'); }