Handle any null pointer exceptions that might occur when looking for …

…AppleSDKs
SeleniumHQ · Mar 25, 2019 · a1cafa8 · a1cafa8
1 parent b3ede28
commit a1cafa8
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 8 deletions.
diff --git a/src/com/facebook/buck/apple/toolchain/impl/AppleSdkDiscovery.java b/src/com/facebook/buck/apple/toolchain/impl/AppleSdkDiscovery.java
@@ -251,7 +251,7 @@ private static boolean buildSdkFromPath(
         sdkBuilder.addAllArchitectures(architectures);
         return true;
       }
-    } catch (NoSuchFileException e) {
+    } catch (NoSuchFileException | NullPointerException e) {
       LOG.warn(e, "Skipping SDK at path %s, no SDKSettings.plist found", sdkDir);
       return false;
     }

diff --git a/src/com/facebook/buck/jvm/java/BUCK b/src/com/facebook/buck/jvm/java/BUCK
@@ -331,11 +331,7 @@ java_library_with_plugins(
         "JavaLibraryDescription.java",
         "JavaLibraryRules.java",
         "JavaLibraryWithTests.java",
-<<<<<<< HEAD
-=======
         "JavaPluginDescription.java",
-        "JavaSourceJar.java",
->>>>>>> fb/master
         "JavaTest.java",
         "JavaTestDescription.java",
         "JavacFactory.java",
@@ -350,11 +346,8 @@ java_library_with_plugins(
         "MaybeRequiredForSourceOnlyAbiArg.java",
         "PrebuiltJar.java",
         "PrebuiltJarDescription.java",
-<<<<<<< HEAD
         "SourceJar.java",
-=======
         "StandardJavacPlugin.java",
->>>>>>> fb/master
         "UnusedDependenciesFinderFactory.java",
         "ZipArchiveDependencySupplier.java",
     ],