fatrace

Merge lp://staging/~bryce2/fatrace/filter_option into lp://staging/fatrace

filter_option
Merge into obsolete-moved-to-git

Proposed by Bryce Nesbitt on 2015-03-04

Status:	Merged
Merged at revision:	62
Proposed branch:	lp://staging/~bryce2/fatrace/filter_option
Merge into:	lp://staging/fatrace
Diff against target:	94 lines (+36/-1) 1 file modified fatrace.c (+36/-1)
To merge this branch:	bzr merge lp://staging/~bryce2/fatrace/filter_option
Related bugs:	Link a bug report

Reviewer	Date Requested	Status
Martin Pitt	2015-03-04	Approve on 2015-04-29
Bryce Nesbitt (community)		Needs Resubmitting on 2015-03-06
Review via email: mp+251697@code.staging.launchpad.net

Description of the change

New option:

-f MASK, --filter Show only the given event types (choose from C,R,O or W)

I use "-f W" to monitor writes, of which Google Chrome seems to be by far the worst culprit, especially if any tab has ever opened Youtube!

lp://staging/~bryce2/fatrace/filter_option updated on 2015-03-04

64. By Bryce Nesbitt <email address hidden> on 2015-03-04: Fix no flag case

Revision history for this message

Martin Pitt (pitti) wrote on 2015-03-04:

Very nice, thanks for this! I just see some minor issues below. I'll merge this once these get fixed.

review: Needs Fixing

lp://staging/~bryce2/fatrace/filter_option updated on 2015-03-06

65. By Bryce Nesbitt <email address hidden> on 2015-03-06: fix a few style issues
make --filter case insensitive

Revision history for this message

Bryce Nesbitt (bryce2) wrote on 2015-03-06:

bzr commit
modified fatrace.c
Committed revision 65.

bzr push
Pushed up to revision 65.

review: Needs Resubmitting

Revision history for this message

Martin Pitt (pitti) wrote on 2015-04-29:

Thanks! I added an error message for unknown options, documented the new version in the manpage, and fixed some style issues. I did a new upstream release https://launchpad.net/fatrace/trunk/0.10 with this, and will package this for Debian unstable now (from where it will flow into the next Ubuntu development series).

review: Approve