Skip to content
This repository has been archived by the owner on May 12, 2018. It is now read-only.

Fixed version of #395 #522

Merged
merged 2 commits into from Jul 3, 2015
Merged

Fixed version of #395 #522

merged 2 commits into from Jul 3, 2015

Conversation

ghost
Copy link

@ghost ghost commented Jul 3, 2015

  • do not use pdict
  • do not enable color support by default. once we have the new
    getopt version, we can add a new type of command line flag
    for that.
  • fix Dialyzer warnings
  • use atom instead of boolean
  • use better name for internal function
  • do not try (and fail) to access rebar's app env in retest test

Fix verbose level in global rebar config

Replace application:set_env on erlang:put
@ghost ghost mentioned this pull request Jul 3, 2015
@ferd
Copy link
Contributor

ferd commented Jul 3, 2015

Tests fail.

@ghost
Copy link
Author

ghost commented Jul 3, 2015

@ferd do you know why the relevant log label regex to match color codes doesn't make a difference (not matched)? If we cannot reliably match that, we might as well simplify the patterns and restore the original version. Tried with colored and uncolored rebar.

 * do not use pdict
 * do not enable color support by default. once we have the new
   getopt version, we can add a new type of command line flag
   for that.
 * fix Dialyzer warnings
 * use atom instead of boolean
 * use better name for internal function
 * do not try (and fail) to access rebar's app env in retest test
@ferd
Copy link
Contributor

ferd commented Jul 3, 2015

No idea. I remember trying the first time around when this branch was submitted and couldn't figure it out in the time I had to spend on the issue.

@ghost
Copy link
Author

ghost commented Jul 3, 2015

Okay, let's merge it, then.

@ghost
Copy link
Author

ghost commented Jul 3, 2015

Unless you want to simplify the patterns first.

ferd added a commit that referenced this pull request Jul 3, 2015
@ferd ferd merged commit e83a214 into rebar:master Jul 3, 2015
@ghost ghost deleted the platinumthinker-colored-logs branch July 3, 2015 17:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants