diff options
author | Mark Schaller <mschaller@google.com> | 2017-03-10 23:01:45 +0000 |
---|---|---|
committer | Vladimir Moskva <vladmos@google.com> | 2017-03-12 01:44:59 +0000 |
commit | fb2d38b34219b06cdbb280ad2ab5132b658dce8f (patch) | |
tree | 84a71463d08397adfccf4df780131f271603105a /src/main/java/com/google/devtools/build/lib/skyframe/RecursivePackageProviderBackedTargetPatternResolver.java | |
parent | 56d0348dc53599fc44313a685aab98b4cf6a739c (diff) |
Improve query error msg when a package has a broken Skylark load
The error message logged during query (and build) when a package has a
broken Skylark load statement was not specific. Previously, it said
"package contains errors:" and then the package name.
Also, this error message was not emitted when using SkyQueryEnvironment
and evaluating a query containing a "TargetsBelowDirectory" pattern
(such as //foo/...) when a package below the specified directory had
such an error.
The approach taken by this CL is to include any package loading error
message in the SkyValue produced by
CollectPackagesUnderDirectoryFunction, and report them during
evaluation of a TargetsBelowDirectory pattern.
RELNOTES: Evaluation of commands on TargetsBelowDirectory patterns
(e.g. //foo/...) matching packages that fail to load now report more
detailed error messages in keep_going mode.
--
PiperOrigin-RevId: 149802362
MOS_MIGRATED_REVID=149802362
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/skyframe/RecursivePackageProviderBackedTargetPatternResolver.java')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/skyframe/RecursivePackageProviderBackedTargetPatternResolver.java | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/skyframe/RecursivePackageProviderBackedTargetPatternResolver.java b/src/main/java/com/google/devtools/build/lib/skyframe/RecursivePackageProviderBackedTargetPatternResolver.java index dc84f5b6a1..b68ef437f8 100644 --- a/src/main/java/com/google/devtools/build/lib/skyframe/RecursivePackageProviderBackedTargetPatternResolver.java +++ b/src/main/java/com/google/devtools/build/lib/skyframe/RecursivePackageProviderBackedTargetPatternResolver.java @@ -97,7 +97,7 @@ public class RecursivePackageProviderBackedTargetPatternResolver private Map<PackageIdentifier, Package> bulkGetPackages(Iterable<PackageIdentifier> pkgIds) throws NoSuchPackageException, InterruptedException { - return recursivePackageProvider.bulkGetPackages(eventHandler, pkgIds); + return recursivePackageProvider.bulkGetPackages(pkgIds); } @Override @@ -244,8 +244,9 @@ public class RecursivePackageProviderBackedTargetPatternResolver Iterable<PathFragment> packagesUnderDirectory; try { pathFragment = TargetPatternResolverUtil.getPathFragment(directory); - packagesUnderDirectory = recursivePackageProvider.getPackagesUnderDirectory( - repository, pathFragment, excludedSubdirectories); + packagesUnderDirectory = + recursivePackageProvider.getPackagesUnderDirectory( + eventHandler, repository, pathFragment, excludedSubdirectories); } catch (TargetParsingException e) { return Futures.immediateFailedFuture(e); } catch (InterruptedException e) { |