[PATCH] tracedump: Add tri-state --color option (auto, always, or never)

[Carl Worth]

On Tue, 25 Oct 2011 19:26:37 +0100, José Fonseca <jose.r.fonseca at gmail.com> wrote:
> Looks good to me. I agree that tackling the low hanging fruit first is
> best.

Great. I think that as soon as we're past these first few hurdles, the
rest of the new command-line interface will fall into place quickly.

> Just one request, please put each command on its separate .cpp file
> (e.g., apitrace_dump), leaving apitrace.{cpp,hpp} for stuff shared by
> all of them.
> 
> Furthermore, let's put all the command line commands in a "cli"
> subdirectory, and put piece of functionality which can be shared with
> the gui in a library -- maybe "common" for the time being.

These two are done now in a new "cli" branch in my repository, (I also
fixed "command_t" to now be "Command").

I see your os-path branch and will now build on top of that to implement
"apitrace trace" as well as doing external invocation for other commands
that already exist as scripts.

-Carl

-- 
carl.d.worth at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/apitrace/attachments/20111028/1ec02f7d/attachment.pgp>