lastGrep should ignore ANSI escape sequences

[szeiger]

sbt version: 1.2.8

lastGrep greps the raw output, including any ANSI color sequences, which leads to unexpected and unwanted effects, e.g.:

sbt:root> lastGrep error.* library/mimaReportBinaryIssues
[error]  * method foo()Int in class scala.collection.immutable.BitSet does not have a correspondent in other version
[error]    filter with: ProblemFilters.exclude[DirectMissingMethodProblem]("scala.collection.immutable.BitSet.foo")
sbt:root> lastGrep \[.*error.* library/mimaReportBinaryIssues
[0m[error]  * method foo()Int in class scala.collection.immutable.BitSet does not have a correspondent in other version
[0m[error]    filter with: ProblemFilters.exclude[DirectMissingMethodProblem]("scala.collection.immutable.BitSet.foo")
sbt:root> lastGrep \[error.* library/mimaReportBinaryIssues
sbt:root>

error is red in this case, eveything else has the default color, so there are at least two escape sequences per line, one to switch to red and the other one to switch back.

It would be more useful to filter out all escape sequences and grep the filtered output.

Bonus points for keeping track of all ANSI attributes, mapping the positions back to the attributed soutce, and synthesizing a new escaped version.

[eed3si9n]

@szeiger Thanks for the report.

It would be more useful to filter out all escape sequences and grep the filtered output.

I agree with your expectation.