search

builddoctor / maven-antsy-color

Maven ANSI colour wrapper to give you shiny, readable output from Maven
68 stars 14 forks source link

Update mvn #7

Closed claussni closed 10 years ago

claussni commented 10 years ago

Works fine! Just bash keeps complaining about '-e: command not found' at the end of Mavens output. I figured out that appending '\' to the last sed command fixes this. Honestly, I've no clue, but it seems to work. ^_^

My system: Debian 3.11.8-1 (Testing)

simpsonjulian commented 10 years ago

Hey @claussni

What version of bash are you running?

It's all good with

GNU bash, version 3.2.48(1)-release (x86_64-apple-darwin10.0)

claussni commented 10 years ago

Hi @simpsonjulian

its GNU bash, version 4.2.45(1)-release (x86_64-pc-linux-gnu)

Might just be a problem of Bash: http://stackoverflow.com/questions/18599711/how-can-i-split-a-command-over-multiple-lines-when-using-an-if-statement. The problem occurs when there are spaces after the backslashes. It's an escaped space then, instead of an escaped newline! O_o

Bash-Fu...

claussni commented 10 years ago

I checked the file, there is no newline. So everything should be fine. I installed it again and now the output fine. Maybe it was just a glitch with my bash. I'll keep an eye on it.

Looks like the Pull Request can be put to the X-Files...

simpsonjulian commented 10 years ago

Cool - thanks for looking into it, @claussni !