Code review comment for lp:~parthm/bzr/503670-vila-grep-as-builtin

As a windows user, I have no problem with the idea of a `bzr grep` that depends on an external grep tool, I'd just want you to lose the xargs dependency.

The -- thing is ugly, why not just pull out the bazaar-specific arguments from the list given and pass the rest onto the external tool?