Added parent look up for dart-sdk license (#61)

Co-authored-by: Natalie Weizenbaum <[email protected]>

Author: leoafarias

CHANGELOG.md

@@ -1,5 +1,8 @@
 # 1.0.0-beta.11
 
+* Properly load the Dart SDK license when it's in the directory above the SDK,
+  as in a Homebrew installation.
+
 * Use the latest version of the `xml` package.
 
 # 1.0.0-beta.10

lib/src/utils.dart

@@ -86,14 +86,12 @@ Future<String> get license => _licenseMemo.runOnce(() async {
       // from Dart Team packages.
       var licenses = <String, List<String>>{};
       var thisPackageLicense = _readLicense(".");
+
       if (thisPackageLicense != null) {
         licenses[thisPackageLicense] = [humanName.value];
       }
 
-      licenses
-          .putIfAbsent(
-              File(p.join(sdkDir.path, 'LICENSE')).readAsStringSync(), () => [])
-          .add("Dart SDK");
+      licenses.putIfAbsent(_readSdkLicense(), () => []).add("Dart SDK");
 
       // Parse the package config rather than the pubspec so we include transitive
       // dependencies. This also includes dev dependencies, but it's possible those
@@ -132,6 +130,19 @@ final _licenseMemo = AsyncMemoizer<String>();
 final _licenseRegExp =
     RegExp(r"^(([a-zA-Z0-9]+[-_])?(LICENSE|COPYING)|UNLICENSE)(\..*)?$");
 
+/// Returns the contents of the `LICENSE` file in [sdkDir].
+String _readSdkLicense() {
+  final dartLicense = File(p.join(sdkDir.path, 'LICENSE'));
+
+  if (dartLicense.existsSync()) {
+    return dartLicense.readAsStringSync();
+  } else {
+    // Homebrew's Dart SDK installation puts the license in the directory above
+    // the SDK, so if it's not in the SDK itself check there.
+    return File(p.join(sdkDir.parent.path, 'LICENSE')).readAsStringSync();
+  }
+}
+
 /// Returns the contents of the `LICENSE` file in [dir], with various possible
 /// filenames and extensions, or `null`.
 String _readLicense(String dir) {