Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(274)

Unified Diff: sdk/lib/_internal/pub/lib/src/barback/dart2js_transformer.dart

Issue 141113011: Support directories other than "web" in pub build. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 6 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: sdk/lib/_internal/pub/lib/src/barback/dart2js_transformer.dart
diff --git a/sdk/lib/_internal/pub/lib/src/barback/dart2js_transformer.dart b/sdk/lib/_internal/pub/lib/src/barback/dart2js_transformer.dart
index aa09244ed04baef220f855c6ed145c6783e178a1..136bfcd9fb03e7fd129387f4c3e1dfe027d72fbc 100644
--- a/sdk/lib/_internal/pub/lib/src/barback/dart2js_transformer.dart
+++ b/sdk/lib/_internal/pub/lib/src/barback/dart2js_transformer.dart
@@ -64,11 +64,15 @@ class Dart2JSTransformer extends Transformer {
Dart2JSTransformer(PackageGraph graph, BarbackMode mode)
: this.withSettings(graph, new BarbackSettings({}, mode));
- /// Only ".dart" files within "web/" are processed.
+ /// Only ".dart" files within a buildable directory are processed.
Future<bool> isPrimary(Asset asset) {
- return new Future.value(
- asset.id.extension == ".dart" &&
- asset.id.path.startsWith("web/"));
+ if (asset.id.extension != ".dart") return new Future.value(false);
+
+ for (var dir in ["benchmark", "example", "test", "web"]) {
+ if (asset.id.path.startsWith("$dir/")) return new Future.value(true);
+ }
+
+ return new Future.value(false);
}
Future apply(Transform transform) {

Powered by Google App Engine
This is Rietveld 408576698