Fix jcommander artifact parsing so that it supports version ranges.

bazelbuild / migration-tooling

Migration tools for Bazel

Apache License 2.0

45 stars 30 forks source link

Closed petroseskinder closed 7 years ago

petroseskinder commented 7 years ago

Previously, jcommander would parse --a=group:artifact:[1.0,2.0] as {"group:artifact:[1.0", "2.0]"} rather than {"group:artifact:[1.0,2.0]"}.

jcommander defaults to splitting by comma, which we don't want.

reviewer:@kchodorow cc:@hhclam, @cgrushko, @jin

bazel-io commented 7 years ago

Can one of the admins verify this patch?