D12226: balooshow: Colorize only when attached to terminal

2018-04-15 Thread Michael Heidelbach
michaelh created this revision.
michaelh added a reviewer: Baloo.
Restricted Application added projects: Frameworks, Baloo.
Restricted Application added a subscriber: Frameworks.
michaelh requested review of this revision.

TEST PLAN
$ balooshow otto.txt
$ balooshow otto.txt | head -n 12

REPOSITORY
  R293 Baloo

BRANCH
  no-color (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D12226

AFFECTED FILES
  src/tools/balooshow/main.cpp

To: michaelh, #baloo
Cc: #frameworks, ashaposhnikov, michaelh, astippich, spoorun, ngraham, bruns, 
alexeymin


D12226: balooshow: Colorize only when attached to terminal

2018-04-15 Thread Stefan Brüns
bruns added reviewers: bruns, FreeBSD.

REPOSITORY
  R293 Baloo

REVISION DETAIL
  https://phabricator.kde.org/D12226

To: michaelh, #baloo, bruns, #freebsd
Cc: #frameworks, ashaposhnikov, michaelh, astippich, spoorun, ngraham, bruns, 
alexeymin


D12226: balooshow: Colorize only when attached to terminal

2018-04-15 Thread Stefan Brüns
bruns accepted this revision.
bruns added a comment.


  LGTM
  Can somebody FreeBSD folk confirm this works also on FreeBSD?

REPOSITORY
  R293 Baloo

REVISION DETAIL
  https://phabricator.kde.org/D12226

To: michaelh, #baloo, bruns, #freebsd
Cc: #frameworks, ashaposhnikov, michaelh, astippich, spoorun, ngraham, bruns, 
alexeymin


D12226: balooshow: Colorize only when attached to terminal

2018-04-15 Thread Tobias C . Berner
tcberner accepted this revision.
tcberner added a comment.
This revision is now accepted and ready to land.


  Works as intended on FreeBSD.

REPOSITORY
  R293 Baloo

BRANCH
  no-color (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D12226

To: michaelh, #baloo, bruns, #freebsd, tcberner
Cc: tcberner, #frameworks, ashaposhnikov, michaelh, astippich, spoorun, 
ngraham, bruns


D12226: balooshow: Colorize only when attached to terminal

2018-04-16 Thread Michael Heidelbach
michaelh updated this revision to Diff 32256.
michaelh added a comment.


  - Rebase

REPOSITORY
  R293 Baloo

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D12226?vs=32193&id=32256

BRANCH
  no-color (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D12226

AFFECTED FILES
  src/tools/balooshow/main.cpp

To: michaelh, #baloo, bruns, #freebsd, tcberner
Cc: tcberner, #frameworks, ashaposhnikov, michaelh, astippich, spoorun, 
ngraham, bruns


D12226: balooshow: Colorize only when attached to terminal

2018-04-16 Thread Michael Heidelbach
This revision was automatically updated to reflect the committed changes.
Closed by commit R293:094d6e90d4c0: balooshow: Colorize only when attached to 
terminal (authored by michaelh).

REPOSITORY
  R293 Baloo

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D12226?vs=32256&id=32257

REVISION DETAIL
  https://phabricator.kde.org/D12226

AFFECTED FILES
  src/tools/balooshow/main.cpp

To: michaelh, #baloo, bruns, #freebsd, tcberner
Cc: tcberner, #frameworks, ashaposhnikov, michaelh, astippich, spoorun, 
ngraham, bruns