Prevent the CLI to canonicalize the binary

For one, it is not necessary: the first argument on the command line is not used any
further when executing the command line. Also, if there is an extension that starts
with the same string as the binary, the binary gets also marked as an extension.
This in turn prohibits the detection of any other Timewarrior command such that e.g.
the command 'summary' in 'timew summary' is treated as an external report, which is
not present and thus makes the call fail.

Closes #677

Signed-off-by: Thomas Lauf <thomas.lauf@tngtech.com>

AUTHORS

@@ -129,3 +129,4 @@ Thanks to the following, who submitted detailed bug reports and excellent sugges
   Eugene Morozov
   Stefan Herold
   Sebastian Carlos
+  ftambara

ChangeLog

@@ -1,3 +1,5 @@
+- #677    Extension names starting with 'timew' cause problems
+          (thanks to ftambara)
 - #661    Make display of ids and annotations the default in summary report for new users
 - #669    id filtering for charts and reports
 - #660    Fix man page section numbers and reference formatting

src/CLI.cpp

@@ -460,6 +460,12 @@ void CLI::canonicalizeNames ()
 
   for (auto& a : _args)
   {
+    // Do not canonicalize the BINARY
+    if (a.hasTag ("BINARY"))
+    {
+      continue;
+    }
+
     auto raw = a.attribute ("raw");
     std::string canonical = raw;