Bug #1191

+ The 'execute' command should not be considered a 'write' command.

ChangeLog

@@ -93,6 +93,8 @@ Bugs
    builds (thanks to Jakub Wilk).
  + Fixed bug #1189, which caused wide Asian UTF8 characters to be measured as
    narrow characters (thanks to Roy Zuo).
+ + Fixed bug #1191, which kept file locks active for longer than necessary,
+   and caused the 'execute' command to be considered a 'write' command.
  + Improved hyphenation by splitting on commas (even if no whitespace after).
    Leads to better output of, for example, 'task show', where comma-separated
    lists are common.

src/TDB2.cpp

@@ -312,7 +312,7 @@ void TF2::load_tasks ()
         _U2I[task.get ("uuid")] = task.id;
       }
     }
- 
+
     if (_auto_dep_scan)
       dependency_scan ();
 

src/commands/CmdExec.cpp

@@ -40,8 +40,8 @@ CmdExec::CmdExec ()
   _keyword     = "execute";
   _usage       = "task          execute <external command>";
   _description = STRING_CMD_EXEC_USAGE;
-  _read_only   = false;
-  _displays_id = true;
+  _read_only   = true;
+  _displays_id = false;
 }
 
 ////////////////////////////////////////////////////////////////////////////////