Move allowTrailingOptions into ArgParser.

sdk/lib/_internal/pub/lib/src/command.dart

 // Copyright (c) 2013, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 library pub.command;
 
 import 'dart:async';
 import 'dart:math' as math;
 
 import 'package:args/args.dart';
@@ skipping 111 matching lines @@
   /// If false, pub won't look for a pubspec and [entrypoint] will be null when
   /// the command runs. This only needs to be set in leaf commands.
   bool get requiresEntrypoint => true;
 
   /// Whether or not this command takes arguments in addition to options.
   ///
   /// If false, pub will exit with an error if arguments are provided. This
   /// only needs to be set in leaf commands.
   bool get takesArguments => false;
 
+  /// Override this and return `false` to disallow trailing options from being
+  /// parsed after a non-option argument is parsed.
+  bool get allowTrailingOptions => true;
+
   /// Alternate names for this command. These names won't be used in the
   /// documentation, but they will work when invoked on the command line.
   final aliases = const <String>[];
 
   /// The [ArgParser] for this command.
-  final commandParser = new ArgParser();
+  ArgParser get commandParser => _commandParser;
+  ArgParser _commandParser;
 
   /// Subcommands exposed by this command.
   ///
   /// If empty, then this command has no subcommands. Otherwise, a subcommand
   /// must be specified by the user. In that case, this command's [onRun] will
   /// not be called and the subcommand's will.
   final subcommands = <String, PubCommand>{};
 
   /// Override this to use offline-only sources instead of hitting the network.
   ///
   /// This will only be called before the [SystemCache] is created. After that,
   /// it has no effect. This only needs to be set in leaf commands.
   bool get isOffline => false;
 
   PubCommand() {
+    _commandParser = new ArgParser(allowTrailingOptions: allowTrailingOptions);
+
     // Allow "--help" after a command to get command help.
     commandParser.addFlag('help', abbr: 'h', negatable: false,
         help: 'Print usage information for this command.');
   }
 
   /// Runs this command using a system cache at [cacheDir] with [options].
   Future run(String cacheDir, ArgResults options) {
     _commandOptions = options;
 
     cache = new SystemCache.withSources(cacheDir, isOffline: isOffline);
@@ skipping 93 matching lines @@
     for (var alias in command.aliases) {
       commands[alias] = command;
     }
   }
 
   return commands;
 }
 
 /// Creates the top-level [ArgParser] used to parse the pub command line.
 ArgParser _initArgParser() {
-  var argParser = new ArgParser();
+  var argParser = new ArgParser(allowTrailingOptions: true);
 
   // Add the global options.
   argParser.addFlag('help', abbr: 'h', negatable: false,
       help: 'Print this usage information.');
   argParser.addFlag('version', negatable: false,
       help: 'Print pub version.');
   argParser.addFlag('trace',
        help: 'Print debugging information when an error occurs.');
   argParser.addOption('verbosity',
       help: 'Control output verbosity.',
@@ skipping 17 matching lines @@
 
 /// Registers a [command] with [name] on [parser].
 void _registerCommand(String name, PubCommand command, ArgParser parser) {
   parser.addCommand(name, command.commandParser);
 
   // Recursively wire up any subcommands.
   command.subcommands.forEach((name, subcommand) {
     _registerCommand(name, subcommand, command.commandParser);
   });
 }