firka/flutter
firka/flutter
5
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

move where the mega_gallery benchmark code is generated by default (#20157)

Browse Source 
* move where the mega_gallery benchmark code is generated by default

* generate into a temporary diectory
This commit is contained in:
Devon Carew 2018-08-06 09:26:32 -07:00 committed by GitHub
parent 81458c7925
commit 53bd0e20da
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 29 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

29
dev/bots/test.dart
View File

@ -189,14 +189,27 @@ Future<Null> _analyzeRepo() async {
await _checkForTrailingSpaces();
// Try an analysis against a big version of the gallery.
await _runCommand(dart,
<String>['--preview-dart-2', path.join(flutterRoot, 'dev', 'tools', 'mega_gallery.dart')],
workingDirectory: flutterRoot,
);
await _runFlutterAnalyze(path.join(flutterRoot, 'dev', 'benchmarks', 'mega_gallery'),
options: <String>['--watch', '--benchmark'],
);
// Try analysis against a big version of the gallery; generate into a temporary directory.
final String outDir = Directory.systemTemp.createTempSync('mega_gallery').path;
try {
await _runCommand(dart,
<String>[
'--preview-dart-2',
path.join(flutterRoot, 'dev', 'tools', 'mega_gallery.dart'),
'--out',
outDir,
],
workingDirectory: flutterRoot,
);
await _runFlutterAnalyze(outDir, options: <String>['--watch', '--benchmark']);
} finally {
try {
new Directory(outDir).deleteSync(recursive: true);
} catch (e) {
// ignore
}
}
print('${bold}DONE: Analysis successful.$reset');
}

11
dev/tools/mega_gallery.dart
View File

@ -18,8 +18,7 @@ void main(List<String> args) {
Directory.current = Directory.current.parent.parent;
final ArgParser argParser = new ArgParser();
// ../mega_gallery? dev/benchmarks/mega_gallery?
argParser.addOption('out', defaultsTo: _normalize('dev/benchmarks/mega_gallery'));
argParser.addOption('out');
argParser.addOption('copies');
argParser.addFlag('delete', negatable: false);
argParser.addFlag('help', abbr: 'h', negatable: false);
@ -45,6 +44,12 @@ void main(List<String> args) {
exit(0);
}
if (!results.wasParsed('out')) {
print('The --out parameter is required.');
print(argParser.usage);
exit(1);
}
int copies;
if (!results.wasParsed('copies')) {
final SourceStats stats = getStatsFor(_dir(source, 'lib'));
@ -128,7 +133,7 @@ void _copyGallery(Directory galleryDir, int index) {
void _copy(Directory source, Directory target) {
if (!target.existsSync())
target.createSync();
target.createSync(recursive: true);
for (FileSystemEntity entity in source.listSync(followLinks: false)) {
final String name = path.basename(entity.path);