Release date for 1.8.4

HACK - case-insensitive file system problem again.
2009-11-17 12:00:36 +01:00 · 2009-11-16 23:42:32 -05:00 · 2009-11-16 23:42:00 -05:00 · 2009-11-16 23:24:47 -05:00 · 2009-11-16 22:10:47 -05:00 · 2009-11-13 23:32:40 +01:00
57 changed files with 1118 additions and 447 deletions
--- a/.gitignore
+++ b/.gitignore
@@ -15,5 +15,4 @@ www.xls
 src/tests/all.log
 src/tests/*.data
 *~
-movie*
 .*.swp
--- a/6
+++ b/6
@@ -17,6 +17,8 @@ The following submitted code, packages or analysis, and deserve special thanks:
  P.C. Shyamshankar
  Johan Friis
  Steven de Brouwer
+  Pietro Cerutti
+  Cory Donnelly

 Thanks to the following, who submitted detailed bug reports and excellent suggestions:
  Eugene Kramer
@@ -34,6 +36,8 @@ Thanks to the following, who submitted detailed bug reports and excellent sugges
  Eric Farris
  Bruce Dillahunty
  Askme Too
+  Mike Adonay
  Thomas@BIC
  Ian Mortimer
-
+  Zach Frazier
+  Joe Pulliam
--- a/68
+++ b/68
@@ -1,7 +1,69 @@

 ------ current release ---------------------------

-1.8.0 (7/21/2009)
+1.8.4 (11/17/2009)
+ + Fixed bug that caused wait: dates to not be properly rendered in a
+   readable and preferred format with the "edit" command.
+ + Fixed bug that caused a hang on cygwin, when a task with multiple
+   annotations was edited (thanks to Joe Pulliam).
+ + Fixed bug #314 where the edit command fails when data.location includes
+   directories containing spaces (thanks to Cory Donnelly).
+ + Added a warning (issue #312) when modifying recurring tasks, that all
+   instances of that task may be modified.  When task confirms a bulk edit
+   the recurrence is again indicated (thanks to Cory Donnelly).
+
+------ old releases ------------------------------
+
+1.8.3 (10/21/2009) bcdcbeeea0d92f21c3565aebfaf6332b959f4025
+ + Added support for Haiku R1/alpha1
+
+1.8.2 (9/7/2009) f243f0ed443ecd7dde779de8a6525222591024db
+ + Added feature #282 that returns useful exit codes to the shell.  Now a
+   script can detect whether no tasks were returned by a report (thanks to
+   Pietro Cerutti).
+ + Fixed bug #287 that causes color control codes to be written to shadow
+   files (thanks to Richard Querin).
+ + Fixed bug #289 which imported task from todo.sh without valid uuids
+   (thanks to Ben Jackson).
+ + Fixed bug #291 which generated a false warning about an unrecognized
+   variable when enabling default.projects in .taskrc (thanks to Thomas@BIC).
+ + Fixed bug #288 which failed to propagate rc file overrides on the command
+   line to the default command (thanks to Zach Frazier).
+
+1.8.1 (8/20/2009) 35792e7874d2bb664abb1a0a67960b7fe7e0fccf
+ + Fixed bug #231 that broke the build on OpenBSD 32-bit due to a time_t
+   and int collision (thanks to Pietro Cerutti).
+ + Fixed bug #241 that prevented bash's tab-completion of projects in Fedora 
+   11 and likely anything using bash-4 (thanks to John Florian).
+ + Fixed bug #242 that sometimes causes the ID echoed after a task is added
+   to be incorrect (thanks to John Florian).
+ + Fixed bug #245 that quoted date fields on export, that were subsequently
+   improperly parsed on import (thanks to John Florian).
+ + Fixed bug #248 where single and double quotes are both stored as
+   ampersand-quot-semi (thanks to John Florian).
+ + Fixed bug #249 that caused annotations with the same date to be lost after
+   a "task edit" command (thanks to Federico Hernandez).
+ + Fixed bug #250 whereby rc.dateformat was not observed when parsing the
+   creation date of an annotation (thanks to Federico Hernandez).
+ + Fixed bug #251 whereby the presence of annotations cause the .hasnt attribute
+   modifier to not work (thanks to John Florian).
+ + Fixed bug #252 that prevented use of attribute modifiers on dates to effect
+   a range, such as "task ls due.after:eom due.before:eoy" (thanks to John
+   Florian).
+ + Fixed bug #256 that allowed a recurring task with no due date.
+ + Fixed bug #257 where an extant ~/.taskrc file prevented the override and
+   automatic creation of an alternate rc file (thanks to Zach Frazier).
+ + Fixed bug #259 that cause a build failure on Snow Leopard 10a432.
+ + Fixed bug #260 whereby the start, stop and delete commands did not complain
+   when filter arguments were specified, even though they were ignored
+   (thanks to T. Charles Yun).
+ + Fixed bug that allowed a recurring task to be added without a due date.
+ + Fixed bug that displays the wrong .taskrc file name on override (thanks to
+   Federico Hernandez).
+ + Fixed bug that failed to suppress color control code in the header and footnote
+   when redirecting output to a file (thanks to John Florian).
+
+1.8.0 (7/21/2009) 14977ef317bd004dae2f2c313e806af9f2a2140c
 + Added zsh tab completion script (thanks to P.C. Shyamshankar).
 + Fixed bug that cause the _forcecolor configuration variable to be
   considered obsolete (thank to Bruce Dillahunty).
@@ -70,9 +132,7 @@
   and completed tasks.
 + Now over 1,600 unit tests, helping to maintain code quality.

------ old releases ------------------------------
-
-1.7.1 (6/8/2009)
+1.7.1 (6/8/2009) 1422a15cbc470cff590bf06daad20d01fe1b05ef
 + Fixed build failure on OpenBSD (thanks to Mike Adonay).
 + Took the opportunity of a patch release to update the various email
   addresses and URLs in the various documents.
--- a/20
+++ b/20
@@ -1,5 +1,5 @@

-New Features in task 1.8.0
+New Features in task 1.8

  - Attribute modifiers, for precise queries
  - Improved calendar feature
@@ -9,26 +9,24 @@ New Features in task 1.8.0
  - In addition to being a standard part of Fedora 10 and 11 (yum install task),
    task is now also a standard part of Cygwin 1.5
  - There are new demo movies on taskwarrior.org
+  - Shell-friendly exit codes

  Please refer to the ChangeLog file for full details.  There are too many to
  list here.

 Task has been built and tested on the following configurations:

-  - OS X 10.5 Leopard
-  - OS X 10.4 Tiger
-  - Fedora Core 11 Leonidas
-  - Fedora Core 10 Cambridge
-  - Ubuntu 9.04 Jaunty Jackalope
-  - Ubuntu 8.10 Intrepid Ibex
-  - Ubuntu 8.04 Hardy Heron
+  - OS X 10.6 Snow Leopard and 10.5 Leopard
+  - Fedora 11 Leonidas and 10 Cambridge
+  - Ubuntu 9.04 Jaunty Jackalope and 8.10 Intrepid Ibex
  - Slackware 12.2
  - Arch Linux
-  - Solaris 10
-  - Solaris 8
+  - Gentoo Linux
+  - Solaris 10 and 8
  - OpenBSD 4.5
  - FreeBSD
-  - Cygwin 1.5.25-14
+  - Cygwin 1.5
+  - Haiku R1/alpha1

 While Task has undergone testing, bugs are sure to remain.  If you encounter a
 bug, please enter a new issue at:
--- a/configure.ac
+++ b/configure.ac
@@ -2,7 +2,7 @@
 # Process this file with autoconf to produce a configure script.

 AC_PREREQ(2.61)
-AC_INIT(task, 1.8.0, support@taskwarrior.org)
+AC_INIT(task, 1.8.4, support@taskwarrior.org)

 CFLAGS="${CFLAGS=}"
 CXXFLAGS="${CXXFLAGS=}"
--- a/doc/man/task-tutorial.5
+++ b/doc/man/task-tutorial.5
@@ -1,4 +1,4 @@
-.TH task-tutorial 5 2009-07-14 "Task 1.8.0" "User Manuals"
+.TH task-tutorial 5 2009-10-21 "task 1.8.4" "User Manuals"

 .SH NAME
 task-tutorial \- A tutorial for the task(1) command line todo manager.
--- a/doc/man/task.1
+++ b/doc/man/task.1
@@ -1,4 +1,4 @@
-.TH task 1 2009-07-14 "Task 1.8.0" "User Manuals"
+.TH task 1 2009-10-21 "task 1.8.4" "User Manuals"

 .SH NAME
 task \- A command line todo manager.
--- a/doc/man/taskrc.5
+++ b/doc/man/taskrc.5
@@ -1,4 +1,4 @@
-.TH taskrc 5 2009-07-14 "Task 1.8.0" "User Manuals"
+.TH taskrc 5 2009-10-21 "task 1.8.4" "User Manuals"

 .SH NAME
 taskrc \- Configuration file for the task(1) command
--- a/package-config/cygwin/CYGWIN-PATCHES/setup.hint
+++ b/package-config/cygwin/CYGWIN-PATCHES/setup.hint
@@ -1,4 +1,4 @@
-# setup.hint for task 1.7.1-1
+# setup.hint for task 1.8.1-1
 category: Utils
 requires: libncurses9 cygwin 
 sdesc: A command-line to do list manager
--- a/package-config/cygwin/CYGWIN-PATCHES/task-1.8.1-1.README
+++ b/package-config/cygwin/CYGWIN-PATCHES/task-1.8.1-1.README
@@ -1,4 +1,4 @@
-task 1.7.1-1
+task 1.8.1-1
 ------------------------------------------

 A command-line to do list manager that can be used
@@ -28,10 +28,10 @@ Language:
 ------------------------------------

 Build instructions:
-  unpack task-1.7.1-1-src.tar.bz2
+  unpack task-1.8.1-1-src.tar.bz2
    if you use setup to install this src package,
    it will be unpacked under /usr/src automatically
-  cd /usr/src/task-1.7.1-1
+  cd /usr/src/task-1.8.1-1
  ./configure --prefix=/usr
  make
  make install
@@ -42,4 +42,4 @@ task was packaged for cygwin by
 Federico Hernandez <ultrafredde@gmail.com> and
 is licensed under the GPL

---- task-1.7.1-1 -- 2009-06-17 ----
+---- task-1.8.1-1 -- 2009-08-20 ----
--- a/package-config/cygwin/task-1.8.1-1.patch
+++ b/package-config/cygwin/task-1.8.1-1.patch
@@ -1,8 +1,8 @@
-diff -Nrup task-1.7.1-1/CYGWIN-PATCHES/setup.hint task-1.7.1-1.cygwin/CYGWIN-PATCHES/setup.hint
--- task-1.7.1-1/CYGWIN-PATCHES/setup.hint	1970-01-01 01:00:00.000000000 +0100
-+++ task-1.7.1-1.cygwin/CYGWIN-PATCHES/setup.hint	2009-06-17 15:47:56.304750000 +0200
+diff -Nrup task-1.8.1-1/CYGWIN-PATCHES/setup.hint task-1.8.1-1.cygwin/CYGWIN-PATCHES/setup.hint
+--- task-1.8.1-1/CYGWIN-PATCHES/setup.hint	1970-01-01 01:00:00.000000000 +0100
+++ task-1.8.1-1.cygwin/CYGWIN-PATCHES/setup.hint	2009-08-20 15:47:56.304750000 +0200
@@ -0,0 +1,10 @@
-+# setup.hint for task 1.7.1-1
+# setup.hint for task 1.8.1-1
 +category: Utils
 +requires: libncurses9 cygwin 
 +sdesc: A command-line to do list manager
@@ -12,11 +12,11 @@ diff -Nrup task-1.7.1-1/CYGWIN-PATCHES/setup.hint task-1.7.1-1.cygwin/CYGWIN-PAT
 +reports and graphs, lots of manipulation commands,
 +low-level API, abbreviations for all commands and
 +options, multiuser file locking, recurring tasks."
-diff -Nrup task-1.7.1-1/CYGWIN-PATCHES/task-1.7.1-1.README task-1.7.1-1.cygwin/CYGWIN-PATCHES/task-1.7.1-1.README
--- task-1.7.1-1/CYGWIN-PATCHES/task-1.7.1-1.README	1970-01-01 01:00:00.000000000 +0100
-+++ task-1.7.1-1.cygwin/CYGWIN-PATCHES/task-1.7.1-1.README	2009-06-17 15:47:44.320375000 +0200
+diff -Nrup task-1.8.1-1/CYGWIN-PATCHES/task-1.8.1-1.README task-1.8.1-1.cygwin/CYGWIN-PATCHES/task-1.8.1-1.README
+--- task-1.8.1-1/CYGWIN-PATCHES/task-1.8.1-1.README	1970-01-01 01:00:00.000000000 +0100
+++ task-1.8.1-1.cygwin/CYGWIN-PATCHES/task-1.8.1-1.README	2009-08-20 15:47:44.320375000 +0200
@@ -0,0 +1,45 @@
-+task 1.7.1-1
+task 1.8.1-1
 +------------------------------------------
 +
 +A command-line to do list manager that can be used
@@ -46,10 +46,10 @@ diff -Nrup task-1.7.1-1/CYGWIN-PATCHES/task-1.7.1-1.README task-1.7.1-1.cygwin/C
 +------------------------------------
 +
 +Build instructions:
-+  unpack task-1.7.1-1-src.tar.bz2
+  unpack task-1.8.1-1-src.tar.bz2
 +    if you use setup to install this src package,
 +    it will be unpacked under /usr/src automatically
-+  cd /usr/src/task-1.7.1-1
+  cd /usr/src/task-1.8.1-1
 +  ./configure --prefix=/usr
 +  make
 +  make install
@@ -60,4 +60,4 @@ diff -Nrup task-1.7.1-1/CYGWIN-PATCHES/task-1.7.1-1.README task-1.7.1-1.cygwin/C
 +Federico Hernandez <ultrafredde@gmail.com> and
 +is licensed under the GPL
 +
-+---- task-1.7.1-1 -- 2009-06-17 ----
+---- task-1.8.1-1 -- 2009-08-20 ----
--- a/package-config/cygwin/usr/share/doc/Cygwin/task-1.8.1-1.README
+++ b/package-config/cygwin/usr/share/doc/Cygwin/task-1.8.1-1.README
@@ -1,4 +1,4 @@
-task 1.7.1-1
+task 1.8.1-1
 ------------------------------------------

 A command-line to do list manager that can be used
@@ -28,10 +28,10 @@ Language:
 ------------------------------------

 Build instructions:
-  unpack task-1.7.1-1-src.tar.bz2
+  unpack task-1.8.1-1-src.tar.bz2
    if you use setup to install this src package,
    it will be unpacked under /usr/src automatically
-  cd /usr/src/task-1.7.1-1
+  cd /usr/src/task-1.8.1-1
  ./configure --prefix=/usr
  make
  make install
@@ -42,4 +42,4 @@ task was packaged for cygwin by
 Federico Hernandez <ultrafredde@gmail.com> and
 is licensed under the GPL

---- task-1.7.1-1 -- 2009-06-17 ----
+---- task-1.8.1-1 -- 2009-08-20 ----
--- a/package-config/fedora/task.spec
+++ b/package-config/fedora/task.spec
@@ -1,6 +1,6 @@
 Name:           task
-Version:        1.8.0
-Release:        0%{?dist}
+Version:        1.8.1
+Release:        1%{?dist}
 Summary:        A command-line to do list manager

 Group:          Applications/Productivity
@@ -40,18 +40,25 @@ rm -rf $RPM_BUILD_ROOT

 %files
 %defattr(-,root,root,-)
-%doc AUTHORS ChangeLog COPYING NEWS README scripts
+%doc AUTHORS ChangeLog COPYING NEWS README scripts i18n
 %{_bindir}/task
 %{_mandir}/man1/task.1.gz
 %{_mandir}/man5/taskrc.5.gz
 %{_mandir}/man5/task-tutorial.5.gz
-%{_sysconfdir}/bash_completion.d
+%config(noreplace) %{_sysconfdir}/bash_completion.d


 %changelog
-* Mon Jun 15 2009 Federico Hernandez <ultrafredde@gmail.com> - 1.8.0-0
+* Thu Aug 20 2009 Federico Hernandez <ultrafredde@gmail.com> - 1.8.1-1
+  Intial RPM for task bugfix release 1.8.1
+* Tue Jul 21 2009 Federico Hernandez <ultrafredde@gmail.com> - 1.8.0-1
  Intial RPM for task release 1.8.0
-  Installs now bash_completion file
+* Mon Jul 13 2009 Federico Hernandez <ultrafredde@gmail.com> - 1.8.0.beta3-1
+  Intial RPM for task beta release 1.8.0.beta3
+* Wed Jul 08 2009 Federico Hernandez <ultrafredde@gmail.com> - 1.8.0.beta2-1
+  Intial RPM for task beta release 1.8.0.beta2
+* Tue Jul 07 2009 Federico Hernandez <ultrafredde@gmail.com> - 1.8.0.beta1-1
+  Intial RPM for task beta release 1.8.0.beta1
 * Tue Jun 08 2009 Federico Hernandez <ultrafredde@gmail.com> - 1.7.1-2
 - Fixed inclusion of manpages.
 * Tue Jun 08 2009 Federico Hernandez <ultrafredde@gmail.com> - 1.7.1-1
--- a/package-config/osx/task.pmdoc/01task-contents.xml
+++ b/package-config/osx/task.pmdoc/01task-contents.xml
@@ -1 +1 @@
-<pkg-contents spec="1.12"><f n="task" o="paul" g="staff" p="33261" pt="/Users/paul/task.git/package-config/osx/binary/task" m="false" t="file"/></pkg-contents>
+<pkg-contents spec="1.12"><f n="task" o="root" g="wheel" p="33261" pt="/Users/paul/task.git/package-config/osx/binary/task" m="false" t="file"><mod>group</mod><mod>owner</mod></f></pkg-contents>
--- a/package-config/osx/task.pmdoc/01task.xml
+++ b/package-config/osx/task.pmdoc/01task.xml
@@ -1 +1 @@
-<pkgref spec="1.12" uuid="C71026FD-E252-42CD-89C3-2F6F087AAF17"><config><identifier>com.beckingham.task180.task.pkg</identifier><version>1.8.0</version><description></description><post-install type="none"/><requireAuthorization/><installFrom mod="true">/Users/paul/task.git/package-config/osx/binary/task</installFrom><installTo mod="true" relocatable="true">/usr/local/bin</installTo><flags><followSymbolicLinks/></flags><packageStore type="internal"></packageStore><mod>parent</mod><mod>locationType</mod><mod>relocatable</mod><mod>version</mod><mod>installTo.path</mod><mod>installTo</mod></config><contents><file-list>01task-contents.xml</file-list><filter>/CVS$</filter><filter>/\.svn$</filter><filter>/\.cvsignore$</filter><filter>/\.cvspass$</filter><filter>/\.DS_Store$</filter></contents></pkgref>
+<pkgref spec="1.12" uuid="C71026FD-E252-42CD-89C3-2F6F087AAF17"><config><identifier>com.beckingham.task182.task.pkg</identifier><version>1.8.2</version><description></description><post-install type="none"/><requireAuthorization/><installFrom mod="true">/Users/paul/task.git/package-config/osx/binary/task</installFrom><installTo mod="true" relocatable="true">/usr/local/bin</installTo><flags><followSymbolicLinks/></flags><packageStore type="internal"></packageStore><mod>installTo</mod><mod>locationType</mod><mod>relocatable</mod><mod>identifier</mod><mod>parent</mod><mod>version</mod><mod>installTo.path</mod></config><contents><file-list>01task-contents.xml</file-list><filter>/CVS$</filter><filter>/\.svn$</filter><filter>/\.cvsignore$</filter><filter>/\.cvspass$</filter><filter>/\.DS_Store$</filter></contents></pkgref>
--- a/package-config/osx/task.pmdoc/index.xml
+++ b/package-config/osx/task.pmdoc/index.xml
@@ -1 +1 @@
-<pkmkdoc spec="1.12"><properties><title>Task 1.8.0</title><build>/Users/paul/Desktop/task-1.8.0.pkg</build><organization>com.beckingham</organization><userSees ui="easy"/><min-target os="3"/><domain anywhere="true" system="true"/></properties><distribution><versions min-spec="1.000000"/><scripts></scripts></distribution><contents><choice title="task" id="choice0" starts_selected="true" starts_enabled="true" starts_hidden="false"><pkgref id="com.beckingham.task180.task.pkg"/><choice-reqs><requirement id="tosv" operator="ge" value="'10.5.0'" selected="no" enabled="no" hidden="unchanged" startSelected="unchanged" startEnabled="unchanged" startHidden="unchanged"/></choice-reqs></choice></contents><resources bg-scale="proportional" bg-align="center"><locale lang="en"><resource mod="true" type="license">/Users/paul/task.git/package-config/osx/binary/COPYING.txt</resource><resource mod="true" type="readme">/Users/paul/task.git/package-config/osx/binary/README.txt</resource></locale></resources><flags/><item type="file">01task.xml</item><mod>properties.title</mod><mod>properties.anywhereDomain</mod><mod>properties.systemDomain</mod></pkmkdoc>
+<pkmkdoc spec="1.12"><properties><title>Task 1.8.2</title><build>/Users/paul/Desktop/task-1.8.2.pkg</build><organization>com.beckingham</organization><userSees ui="both"/><min-target os="3"/><domain anywhere="true" system="true"/></properties><distribution><versions min-spec="1.000000"/><scripts></scripts></distribution><contents><choice title="task" id="choice0" starts_selected="true" starts_enabled="true" starts_hidden="false"><pkgref id="com.beckingham.task182.task.pkg"/><choice-reqs><requirement id="tosv" operator="ge" value="'10.6.0'" selected="no" enabled="no" hidden="unchanged" startSelected="unchanged" startEnabled="unchanged" startHidden="unchanged"/></choice-reqs></choice></contents><resources bg-scale="proportional" bg-align="center"><locale lang="en"><resource mod="true" type="license">/Users/paul/task.git/package-config/osx/binary/COPYING.txt</resource><resource mod="true" type="readme">/Users/paul/task.git/package-config/osx/binary/README.txt</resource></locale></resources><flags/><item type="file">01task.xml</item><mod>properties.title</mod><mod>properties.customizeOption</mod><mod>properties.anywhereDomain</mod><mod>properties.systemDomain</mod></pkmkdoc>
--- a/package-config/ubuntu/debian/changelog
+++ b/package-config/ubuntu/debian/changelog
@@ -1,8 +1,32 @@
+task (1.8.1-0ubuntu1) jaunty; urgency=low
+
+  * Initial deb package for task bugfix release 1.8.1 
+
+ -- Federico Hernandez <ultrafredde@gmail.com>  Thu, 20 Aug 2009 20:26:33 +0200
+
 task (1.8.0-0ubuntu1) jaunty; urgency=low

  * Initial deb package for task release 1.8.0 

- -- Federico Hernandez <ultrafredde@gmail.com>  Thu, 11 Jun 2009 23:02:28 +0200
+ -- Federico Hernandez <ultrafredde@gmail.com>  Tue, 21 Jul 2009 15:26:33 +0200
+
+task (1.8.0-0ubuntu1~beta3) jaunty; urgency=low
+
+  * Initial deb package for task beta release 1.8.0.beta3 
+
+ -- Federico Hernandez <ultrafredde@gmail.com>  Mon, 13 Jul 2009 15:09:38 +0200
+
+task (1.8.0-0ubuntu1~beta2) jaunty; urgency=low
+
+  * Initial deb package for task beta release 1.8.0.beta2 
+
+ -- Federico Hernandez <ultrafredde@gmail.com>  Wed, 08 Jul 2009 15:40:50 +0200
+
+task (1.8.0-0ubuntu1~beta1) jaunty; urgency=low
+
+  * Initial deb package for task beta release 1.8.0.beta1 
+
+ -- Federico Hernandez <ultrafredde@gmail.com>  Tue, 07 Jul 2009 01:35:28 +0200

 task (1.7.1-0ubuntu1) jaunty; urgency=low

--- a/package-config/ubuntu/debian/copyright
+++ b/package-config/ubuntu/debian/copyright
@@ -1,6 +1,6 @@
 This package was debianized by:

-    Federico Hernandez <ultrafredde@gmail.com> on Thu, 11 Jun 2009 23:02:28 +0200
+    Federico Hernandez <ultrafredde@gmail.com> on  Thu, 20 Aug 2009 20:26:33 +0200

 It was downloaded from:

@@ -9,6 +9,9 @@ It was downloaded from:
 Upstream Authors:

    Paul Beckingham <paul@beckingham.net>
+    Federico Hernandez
+    David J Patrick
+    John Florian
    Damian Glenny
    Andy Lester
    H. İbrahim Güngör
@@ -17,18 +20,18 @@ Upstream Authors:
    Benjamin Tegarden
    Chris Pride
    Richard Querin
-    Federico Hernandez
    T. Charles Yun
-    David J Patrick
    P.C. Shyamshankar
    Johan Friis
    Steven de Brouwer
+    Pietro Cerutti

 Copyright:

    Copyright 2006 - 2009, Paul Beckingham
    Copyright 2009 Federico Hernandez
    Copyright 2009 P.C. Shyamshankar
+    Copyright 2009 John Florian

 License:

--- a/package-config/ubuntu/debian/examples
+++ b/package-config/ubuntu/debian/examples
@@ -0,0 +1,2 @@
+scripts
+i18n
--- a/package-config/ubuntu/debian/rules
+++ b/package-config/ubuntu/debian/rules
@@ -31,8 +31,9 @@ binary-arch: install
 	dh_installchangelogs
 	dh_installdocs 
 	dh_installman
-	install -D -m644 task_completion.sh $(CURDIR)/debian/task/etc/bash_completion.d/task
-	rm -rf $(CURDIR)/debian/task/usr/share/doc/task-1.7.1 
+	dh_installexamples
+	install -D -m644 scripts/bash/task_completion.sh $(CURDIR)/debian/task/etc/bash_completion.d/task
+	rm -rf $(CURDIR)/debian/task/usr/share/doc/task-1.8.1
 	dh_strip
 	dh_compress
 	dh_fixperms
--- a/scripts/bash/task_completion.sh
+++ b/scripts/bash/task_completion.sh
@@ -48,10 +48,6 @@
 #       http://taskwarrior.org
 #

-_task_get_projects() {
-    task _projects
-}
-
 _task_get_tags() {
    task _tags
 }
@@ -60,6 +56,10 @@ _task_get_config() {
    task _config
 }

+_task_offer_projects() {
+    COMPREPLY=( $(compgen -W "$(task _projects)" -- ${cur/*:/}) )
+}
+
 _task() 
 {
    local cur prev opts base
@@ -67,34 +67,61 @@ _task()
    COMPREPLY=()
    cur="${COMP_WORDS[COMP_CWORD]}"
    prev="${COMP_WORDS[COMP_CWORD-1]}"
+    if [ ${#COMP_WORDS[*]} -gt 2 ]
+    then
+        prev2="${COMP_WORDS[COMP_CWORD-2]}"
+    else
+        prev2=""
+    fi
+#   echo -e "\ncur='$cur'"
+#   echo "prev='$prev'"
+#   echo "prev2='$prev2'"

    opts="$(task _commands) $(task _ids)"

-    case "${cur}" in
-        pro*:*)
-            local projects=$(_task_get_projects)
-            local partial_project="${cur/*:/}"
-            COMPREPLY=( $(compgen -W "${projects}" -- ${partial_project}) )
-            return 0
+    case "${prev}" in
+        :)
+            case "${prev2}" in
+                pro*)
+                    _task_offer_projects
+                    return 0
+                    ;;
+            esac
            ;;
-        +*)
-            local tags=$(_task_get_tags | sed 's/^/+/')
-            COMPREPLY=( $(compgen -W "${tags}" -- ${cur}) )
-            return 0
-            ;;
-        -*)
-            local tags=$(_task_get_tags | sed 's/^/-/')
-            COMPREPLY=( $(compgen -W "${tags}" -- ${cur}) )
-            return 0
-            ;;
-        rc.*)
-            local config=$(_task_get_config | sed 's/^/rc\./')
-            COMPREPLY=( $(compgen -W "${config}" -- ${cur}) )
-            return 0
+        *)
+            case "${cur}" in
+                pro*:*)
+                    _task_offer_projects
+                    return 0
+                    ;;
+                :)
+                    case "${prev}" in
+                        pro*)
+                            _task_offer_projects
+                            return 0
+                            ;;
+                    esac
+                    ;;
+                +*)
+                    local tags=$(_task_get_tags | sed 's/^/+/')
+                    COMPREPLY=( $(compgen -W "${tags}" -- ${cur}) )
+                    return 0
+                    ;;
+                -*)
+                    local tags=$(_task_get_tags | sed 's/^/-/')
+                    COMPREPLY=( $(compgen -W "${tags}" -- ${cur}) )
+                    return 0
+                    ;;
+                rc.*)
+                    local config=$(_task_get_config | sed -e 's/^/rc\./' -e 's/$/:/')
+                    COMPREPLY=( $(compgen -W "${config}" -- ${cur}) )
+                    return 0
+                    ;;
+            esac
            ;;
    esac
      
    COMPREPLY=( $(compgen -W "${opts}" -- ${cur}) )
    return 0
 }
-complete -F _task task
+complete -o nospace -F _task task
--- a/src/Att.cpp
+++ b/src/Att.cpp
@@ -407,6 +407,17 @@ std::string Att::type (const std::string& name) const
    return "text";
 }

+////////////////////////////////////////////////////////////////////////////////
+// The type of an attribute is useful for modifier evaluation.
+std::string Att::modType (const std::string& name) const
+{
+  if (name == "hasnt" ||
+      name == "isnt")
+    return "negative";
+
+  return "positive";
+}
+
 ////////////////////////////////////////////////////////////////////////////////
 //
 // start --> name --> . --> mod --> : --> " --> value --> " --> end
@@ -461,7 +472,7 @@ void Att::parse (Nibbler& n)
  else
    throw std::string ("Missing : after attribute name"); // TODO i18n

-/* TODO This might be too slow to include.  Test.
+/* TODO This might be too slow to include.  Test this assumption.
  validNameValue (mName, mMod, mValue);
 */
 }
@@ -711,8 +722,11 @@ void Att::encode (std::string& value) const
  while ((i = value.find ('\t')) != std::string::npos)
    value.replace (i, 1, "&tab;"); // no i18n

+  while ((i = value.find ('\'')) != std::string::npos)
+    value.replace (i, 1, "&squot;"); // no i18n
+
  while ((i = value.find ('"')) != std::string::npos)
-    value.replace (i, 1, "&quot;"); // no i18n
+    value.replace (i, 1, "&dquot;"); // no i18n

  while ((i = value.find (',')) != std::string::npos)
    value.replace (i, 1, "&comma;"); // no i18n
@@ -730,7 +744,8 @@ void Att::encode (std::string& value) const
 ////////////////////////////////////////////////////////////////////////////////
 // Decode values after parse.
 //   \t <- &tab;
-//   "  <- &quot;
+//   "  <- &quot; or &dquot;
+//   '  <- &squot;
 //   ,  <- &comma;
 //   [  <- &open;
 //   ]  <- &close;
@@ -742,6 +757,12 @@ void Att::decode (std::string& value) const
  while ((i = value.find ("&tab;")) != std::string::npos) // no i18n
    value.replace (i, 5, "\t");

+  while ((i = value.find ("&dquot;")) != std::string::npos) // no i18n
+    value.replace (i, 7, "\"");
+
+  while ((i = value.find ("&squot;")) != std::string::npos) // no i18n
+    value.replace (i, 7, "'");
+
  while ((i = value.find ("&quot;")) != std::string::npos) // no i18n
    value.replace (i, 6, "\"");

--- a/src/Att.h
+++ b/src/Att.h
@@ -50,6 +50,7 @@ public:
  static bool validNameValue (std::string&, std::string&, std::string&);
  static bool validMod (const std::string&);
  std::string type (const std::string&) const;
+  std::string modType (const std::string&) const;
  void parse (const std::string&);
  void parse (Nibbler&);
  bool match (const Att&) const;
--- a/src/Context.cpp
+++ b/src/Context.cpp
@@ -30,6 +30,7 @@
 #include <pwd.h>
 #include <stdlib.h>
 #include <string.h>
+#include <unistd.h>
 #include "Context.h"
 #include "Timer.h"
 #include "text.h"
@@ -49,6 +50,7 @@ Context::Context ()
 , tdb ()
 , stringtable ()
 , program ("")
+, overrides ("")
 , cmd ()
 , inShadow (false)
 {
@@ -126,98 +128,110 @@ void Context::initialize ()
 ////////////////////////////////////////////////////////////////////////////////
 int Context::run ()
 {
+  int rc;
  Timer t ("Context::run");

  std::string output;
  try
  {
-    parse ();             // Parse command line.
-    output = dispatch (); // Dispatch to command handlers.
+    parse ();               // Parse command line.
+    rc = dispatch (output); // Dispatch to command handlers.
  }

  catch (const std::string& error)
  {
    footnote (error);
+    rc = 2;
  }

  catch (...)
  {
    footnote (stringtable.get (100, "Unknown error."));
+    rc = 3;
  }

  // Dump all debug messages.
  if (config.get (std::string ("debug"), false))
    foreach (d, debugMessages)
-      std::cout << colorizeDebug (*d) << std::endl;
+      if (config.get ("color", true) || config.get (std::string ("_forcecolor"), false))
+        std::cout << colorizeDebug (*d) << std::endl;
+      else
+        std::cout << *d << std::endl;

  // Dump all headers.
  foreach (h, headers)
-    std::cout << colorizeHeader (*h) << std::endl;
+    if (config.get ("color", true) || config.get (std::string ("_forcecolor"), false))
+      std::cout << colorizeHeader (*h) << std::endl;
+    else
+      std::cout << *h << std::endl;

  // Dump the report output.
  std::cout << output;

  // Dump all footnotes.
  foreach (f, footnotes)
-    std::cout << colorizeFootnote (*f) << std::endl;
+    if (config.get ("color", true) || config.get (std::string ("_forcecolor"), false))
+      std::cout << colorizeFootnote (*f) << std::endl;
+    else
+      std::cout << *f << std::endl;

-  return 0;
+  return rc;
 }

 ////////////////////////////////////////////////////////////////////////////////
-std::string Context::dispatch ()
+int Context::dispatch (std::string &out)
 {
+  int rc = 0;
  Timer t ("Context::dispatch");

  // TODO Just look at this thing.  It cries out for a dispatch table.
-  std::string out;
-       if (cmd.command == "projects")      { out = handleProjects           (); }
-  else if (cmd.command == "tags")          { out = handleTags               (); }
-  else if (cmd.command == "colors")        { out = handleColor              (); }
-  else if (cmd.command == "version")       { out = handleVersion            (); }
-  else if (cmd.command == "help")          { out = longUsage                (); }
-  else if (cmd.command == "stats")         { out = handleReportStats        (); }
-  else if (cmd.command == "info")          { out = handleInfo               (); }
-  else if (cmd.command == "history")       { out = handleReportHistory      (); }
-  else if (cmd.command == "ghistory")      { out = handleReportGHistory     (); }
-  else if (cmd.command == "summary")       { out = handleReportSummary      (); }
-  else if (cmd.command == "calendar")      { out = handleReportCalendar     (); }
-  else if (cmd.command == "timesheet")     { out = handleReportTimesheet    (); }
-  else if (cmd.command == "add")           { out = handleAdd                (); }
-  else if (cmd.command == "append")        { out = handleAppend             (); }
-  else if (cmd.command == "annotate")      { out = handleAnnotate           (); }
-  else if (cmd.command == "done")          { out = handleDone               (); }
-  else if (cmd.command == "delete")        { out = handleDelete             (); }
-  else if (cmd.command == "start")         { out = handleStart              (); }
-  else if (cmd.command == "stop")          { out = handleStop               (); }
-  else if (cmd.command == "export")        { out = handleExport             (); }
-  else if (cmd.command == "import")        { out = handleImport             (); }
-  else if (cmd.command == "duplicate")     { out = handleDuplicate          (); }
-  else if (cmd.command == "edit")          { out = handleEdit               (); }
+       if (cmd.command == "projects")      { rc = handleProjects           (out); }
+  else if (cmd.command == "tags")          { rc = handleTags               (out); }
+  else if (cmd.command == "colors")        { rc = handleColor              (out); }
+  else if (cmd.command == "version")       { rc = handleVersion            (out); }
+  else if (cmd.command == "help")          { rc = longUsage                (out); }
+  else if (cmd.command == "stats")         { rc = handleReportStats        (out); }
+  else if (cmd.command == "info")          { rc = handleInfo               (out); }
+  else if (cmd.command == "history")       { rc = handleReportHistory      (out); }
+  else if (cmd.command == "ghistory")      { rc = handleReportGHistory     (out); }
+  else if (cmd.command == "summary")       { rc = handleReportSummary      (out); }
+  else if (cmd.command == "calendar")      { rc = handleReportCalendar     (out); }
+  else if (cmd.command == "timesheet")     { rc = handleReportTimesheet    (out); }
+  else if (cmd.command == "add")           { rc = handleAdd                (out); }
+  else if (cmd.command == "append")        { rc = handleAppend             (out); }
+  else if (cmd.command == "annotate")      { rc = handleAnnotate           (out); }
+  else if (cmd.command == "done")          { rc = handleDone               (out); }
+  else if (cmd.command == "delete")        { rc = handleDelete             (out); }
+  else if (cmd.command == "start")         { rc = handleStart              (out); }
+  else if (cmd.command == "stop")          { rc = handleStop               (out); }
+  else if (cmd.command == "export")        { rc = handleExport             (out); }
+  else if (cmd.command == "import")        { rc = handleImport             (out); }
+  else if (cmd.command == "duplicate")     { rc = handleDuplicate          (out); }
+  else if (cmd.command == "edit")          { rc = handleEdit               (out); }
 #ifdef FEATURE_SHELL
-  else if (cmd.command == "shell")         {       handleShell              (); }
+  else if (cmd.command == "shell")         {      handleShell              (   ); }
 #endif
-  else if (cmd.command == "undo")          {       handleUndo               (); }
-  else if (cmd.command == "_projects")     { out = handleCompletionProjects (); }
-  else if (cmd.command == "_tags")         { out = handleCompletionTags     (); }
-  else if (cmd.command == "_commands")     { out = handleCompletionCommands (); }
-  else if (cmd.command == "_ids")          { out = handleCompletionIDs      (); }
-  else if (cmd.command == "_config")       { out = handleCompletionConfig   (); }
+  else if (cmd.command == "undo")          {      handleUndo               (   ); }
+  else if (cmd.command == "_projects")     { rc = handleCompletionProjects (out); }
+  else if (cmd.command == "_tags")         { rc = handleCompletionTags     (out); }
+  else if (cmd.command == "_commands")     { rc = handleCompletionCommands (out); }
+  else if (cmd.command == "_ids")          { rc = handleCompletionIDs      (out); }
+  else if (cmd.command == "_config")       { rc = handleCompletionConfig   (out); }
  else if (cmd.command == "" &&
-           sequence.size ())               { out = handleModify             (); }
+           sequence.size ())               { rc = handleModify             (out); }

  // Command that display IDs and therefore need TDB::gc first.
-  else if (cmd.command == "next")          { if (!inShadow) tdb.gc (); out = handleReportNext    (); }
-  else if (cmd.validCustom (cmd.command))  { if (!inShadow) tdb.gc (); out = handleCustomReport  (cmd.command); }
+  else if (cmd.command == "next")          { if (!inShadow) tdb.gc (); rc = handleReportNext (out); }
+  else if (cmd.validCustom (cmd.command))  { if (!inShadow) tdb.gc (); rc = handleCustomReport  (cmd.command, out); }

  // If the command is not recognized, display usage.
-  else                                     { out = shortUsage (); }
+  else                                     { rc = shortUsage (out); }

  // Only update the shadow file if such an update was not suppressed (shadow),
  if (cmd.isWriteCommand () && !inShadow)
    shadow ();

-  return out;
+  return rc;
 }

 ////////////////////////////////////////////////////////////////////////////////
@@ -243,8 +257,6 @@ void Context::shadow ()

    std::string oldCurses = config.get ("curses");
    std::string oldColor  = config.get ("color");
-    config.set ("curses", "off");
-    config.set ("color",  "off");

    clear ();

@@ -255,8 +267,12 @@ void Context::shadow ()
    split (args, command, ' ');

    initialize ();
+    config.set ("curses", "off");
+    config.set ("color",  "off");
+
    parse ();
-    std::string result = dispatch ();
+    std::string result;
+    (void)dispatch (result);
    std::ofstream out (shadowFile.c_str ());
    if (out.good ())
    {
@@ -307,6 +323,18 @@ std::string Context::canonicalize (const std::string& input) const
  return canonical;
 }

+////////////////////////////////////////////////////////////////////////////////
+void Context::disallowModification () const
+{
+  if (task.size ()         ||
+      subst.mFrom != ""    ||
+      tagAdditions.size () ||
+      tagRemovals.size ())
+    throw std::string ("The '")
+        + cmd.command
+        + "' command does not allow further modification of a task.";
+}
+
 ////////////////////////////////////////////////////////////////////////////////
 void Context::loadCorrectConfigFile ()
 {
@@ -330,6 +358,14 @@ void Context::loadCorrectConfigFile ()
    else if (arg->substr (0, 3) == "rc:")
    {
      rc = arg->substr (3, std::string::npos);
+
+      home = rc;
+      std::string::size_type last_slash = rc.rfind ("/");
+      if (last_slash != std::string::npos)
+        home = rc.substr (0, last_slash);
+      else
+        home = ".";
+
      args.erase (arg);
      header ("Using alternate .taskrc file " + rc); // TODO i18n
      break;
@@ -358,13 +394,19 @@ void Context::loadCorrectConfigFile ()
  }

  // Do we need to create a default rc?
-  if (access (rc.c_str (), F_OK) &&
-      confirm ("A configuration file could not be found in " // TODO i18n
-             + home
-             + "\n\n"
-             + "Would you like a sample .taskrc created, so task can proceed?"))
+  if (access (rc.c_str (), F_OK))
  {
-    config.createDefaultRC (rc, data);
+    if (confirm ("A configuration file could not be found in " // TODO i18n
+               + home
+               + "\n\n"
+               + "Would you like a sample "
+               + rc
+               + " created, so task can proceed?"))
+    {
+      config.createDefaultRC (rc, data);
+    }
+    else
+      throw std::string ("Cannot proceed without rc file.");
  }

  // Create data location, if necessary.
@@ -398,6 +440,7 @@ void Context::loadCorrectConfigFile ()
          n.getUntilEOS (value))
      {
        config.set (name, value);
+        overrides += " " + *arg;
        footnote (std::string ("Configuration override ") +  // TODO i18n
                  arg->substr (3, std::string::npos));
      }
@@ -528,7 +571,12 @@ void Context::parse (
          attribute.mod (mod);
          attribute.value (value);

-          parseTask[attribute.name ()] = attribute;
+          // Preserve modifier in the key, to allow multiple modifiers on the
+          // same attribute.  Bug #252.
+          if (name != "" && mod != "")
+            parseTask[name + "." + mod] = attribute;
+          else
+            parseTask[name] = attribute;
        }

        // *arg has the appearance of an attribute (foo:bar), but isn't
@@ -609,12 +657,13 @@ void Context::parse (
  // then invoke the default command.
  if (parseCmd.command == "" && parseArgs.size () == 0)
  {
-    std::string defaultCommand = config.get ("default.command");
+    // Apply overrides, if any.
+    std::string defaultCommand = config.get ("default.command") + overrides;
    if (defaultCommand != "")
    {
      // Stuff the command line.
-      parseArgs.clear ();
-      split (parseArgs, defaultCommand, ' ');
+      args.clear ();
+      split (args, defaultCommand, ' ');
      header ("[task " + defaultCommand + "]");

      // Reinitialize the context and recurse.
@@ -657,46 +706,46 @@ void Context::autoFilter (Task& t, Filter& f)
  foreach (att, t)
  {
    // Words are found in the description using the .has modifier.
-    if (att->first == "description" && att->second.mod () == "")
+    if (att->second.name () == "description" && att->second.mod () == "")
    {
      std::vector <std::string> words;
      split (words, att->second.value (), ' ');
      foreach (word, words)
      {
        f.push_back (Att ("description", "has", *word));
-        debug ("auto filter: " + att->first + ".has:" + *word);
+        debug ("auto filter: " + att->second.name () + ".has:" + *word);
      }
    }

    // Projects are matched left-most.
-    else if (att->first == "project" && att->second.mod () == "")
+    else if (att->second.name () == "project" && att->second.mod () == "")
    {
      if (att->second.value () != "")
      {
        f.push_back (Att ("project", "startswith", att->second.value ()));
-        debug ("auto filter: " + att->first + ".startswith:" + att->second.value ());
+        debug ("auto filter: " + att->second.name () + ".startswith:" + att->second.value ());
      }
      else
      {
        f.push_back (Att ("project", "is", att->second.value ()));
-        debug ("auto filter: " + att->first + ".is:" + att->second.value ());
+        debug ("auto filter: " + att->second.name () + ".is:" + att->second.value ());
      }
    }

    // The limit attribute does not participate in filtering, and needs to be
    // specifically handled in handleCustomReport.
-    else if (att->first == "limit")
+    else if (att->second.name () == "limit")
    {
    }

    // Every task has a unique uuid by default, and it shouldn't be included,
    // because it is guaranteed to not match.
-    else if (att->first == "uuid")
+    else if (att->second.name () == "uuid")
    {
    }

    // The mechanism for filtering on tags is +/-<tag>.
-    else if (att->first == "tags")
+    else if (att->second.name () == "tags")
    {
    }

@@ -705,7 +754,7 @@ void Context::autoFilter (Task& t, Filter& f)
    {
      f.push_back (att->second);
      debug ("auto filter: " +
-             att->first +
+             att->second.name () +
             (att->second.mod () != "" ?
               ("." + att->second.mod () + ":") :
               ":") +
--- a/src/Context.h
+++ b/src/Context.h
@@ -50,7 +50,7 @@ public:
  void initialize ();                  // for reinitializing
  int run ();                          // task classic
  int interactive ();                  // task interactive (not implemented)
-  std::string dispatch ();             // command handler dispatch
+  int dispatch (std::string&);         // command handler dispatch
  void shadow ();                      // shadow file update

  int getWidth ();                     // determine terminal width
@@ -65,6 +65,7 @@ public:
  void clear ();

  std::string canonicalize (const std::string&) const;
+  void disallowModification () const;

 private:
  void loadCorrectConfigFile ();
@@ -82,6 +83,7 @@ public:
  StringTable               stringtable;
  std::string               program;
  std::vector <std::string> args;
+  std::string               overrides;
  Cmd                       cmd;
  std::map <std::string, std::string> aliases;
  std::vector <std::string> tagAdditions;
--- a/src/Date.cpp
+++ b/src/Date.cpp
@@ -29,6 +29,7 @@
 #include <time.h>
 #include <assert.h>
 #include <stdlib.h>
+#include <ctype.h>
 #include "Date.h"
 #include "text.h"
 #include "util.h"
@@ -82,14 +83,14 @@ Date::Date (const std::string& mdy, const std::string& format /* = "m/d/Y" */)
    // Single or double digit.
    case 'm':
      if (i >= mdy.length () ||
-          ! ::isdigit (mdy[i]))
+          ! isdigit (mdy[i]))
      {
        throw std::string ("\"") + mdy + "\" is not a valid date.";
      }

      if (i + 1 < mdy.length ()                    &&
          (mdy[i + 0] == '0' || mdy[i + 0] == '1') &&
-          ::isdigit (mdy[i + 1]))
+          isdigit (mdy[i + 1]))
      {
        month = ::atoi (mdy.substr (i, 2).c_str ());
        i += 2;
@@ -103,14 +104,14 @@ Date::Date (const std::string& mdy, const std::string& format /* = "m/d/Y" */)

    case 'd':
      if (i >= mdy.length () ||
-          ! ::isdigit (mdy[i]))
+          ! isdigit (mdy[i]))
      {
        throw std::string ("\"") + mdy + "\" is not a valid date.";
      }

      if (i + 1 < mdy.length ()                                                              &&
          (mdy[i + 0] == '0' || mdy[i + 0] == '1' || mdy[i + 0] == '2' || mdy[i + 0] == '3') &&
-          ::isdigit (mdy[i + 1]))
+          isdigit (mdy[i + 1]))
      {
        day = ::atoi (mdy.substr (i, 2).c_str ());
        i += 2;
@@ -125,8 +126,8 @@ Date::Date (const std::string& mdy, const std::string& format /* = "m/d/Y" */)
    // Double digit.
    case 'y':
      if (i + 1 >= mdy.length ()   ||
-          ! ::isdigit (mdy[i + 0]) ||
-          ! ::isdigit (mdy[i + 1]))
+          ! isdigit (mdy[i + 0]) ||
+          ! isdigit (mdy[i + 1]))
      {
        throw std::string ("\"") + mdy + "\" is not a valid date.";
      }
@@ -137,8 +138,8 @@ Date::Date (const std::string& mdy, const std::string& format /* = "m/d/Y" */)

    case 'M':
      if (i + 1 >= mdy.length () ||
-          ! ::isdigit (mdy[i + 0]) ||
-          ! ::isdigit (mdy[i + 1]))
+          ! isdigit (mdy[i + 0]) ||
+          ! isdigit (mdy[i + 1]))
      {
        throw std::string ("\"") + mdy + "\" is not a valid date.";
      }
@@ -149,8 +150,8 @@ Date::Date (const std::string& mdy, const std::string& format /* = "m/d/Y" */)

    case 'D':
      if (i + 1 >= mdy.length () ||
-          ! ::isdigit (mdy[i + 0]) ||
-          ! ::isdigit (mdy[i + 1]))
+          ! isdigit (mdy[i + 0]) ||
+          ! isdigit (mdy[i + 1]))
      {
        throw std::string ("\"") + mdy + "\" is not a valid date.";
      }
@@ -162,10 +163,10 @@ Date::Date (const std::string& mdy, const std::string& format /* = "m/d/Y" */)
    // Quadruple digit.
    case 'Y':
      if (i + 3 >= mdy.length () ||
-          ! ::isdigit (mdy[i + 0]) ||
-          ! ::isdigit (mdy[i + 1]) ||
-          ! ::isdigit (mdy[i + 2]) ||
-          ! ::isdigit (mdy[i + 3]))
+          ! isdigit (mdy[i + 0]) ||
+          ! isdigit (mdy[i + 1]) ||
+          ! isdigit (mdy[i + 2]) ||
+          ! isdigit (mdy[i + 3]))
      {
        throw std::string ("\"") + mdy + "\" is not a valid date.";
      }
--- a/src/Duration.cpp
+++ b/src/Duration.cpp
@@ -51,12 +51,6 @@ Duration::Duration (const std::string& input)
  parse (input);
 }

-////////////////////////////////////////////////////////////////////////////////
-Duration::operator int ()
-{
-  return (int) mDays;
-}
-
 ////////////////////////////////////////////////////////////////////////////////
 Duration::operator time_t ()
 {
--- a/src/Duration.h
+++ b/src/Duration.h
@@ -40,7 +40,6 @@ public:
  bool operator> (const Duration&);
  ~Duration ();                          // Destructor

-  operator int ();
  operator time_t ();
  operator std::string ();

--- a/src/Filter.cpp
+++ b/src/Filter.cpp
@@ -40,37 +40,51 @@ bool Filter::pass (const Record& record) const
  // First do description/annotation matches.
  foreach (att, (*this))
  {
-    // Descriptions have special handling.
+    // Descriptions have special handling such that they are linked to
+    // annotations, and filtering on description implies identical filtering
+    // on annotations, and that both filter matches must succeed for the filter
+    // to succeed overall.
    if (att->name () == "description")
    {
+      bool description_result = true;
+      int annotation_pass_count = 0;
+      int annotation_fail_count = 0;
+
      if ((r = record.find (att->name ())) != record.end ())
      {
-        // A description match failure can be salvaged by an annotation match.
-        if (! att->match (r->second))
-        {
-          bool annoMatch = false;
-          foreach (ra, record)
-          {
-            if (ra->first.length () > 11 &&
-                ra->first.substr (0, 11) == "annotation_")
-            {
-              if (att->match (ra->second))
-              {
-                annoMatch = true;
-                break;
-              }
-            }
-          }
+        description_result = att->match (r->second);

-          if (!annoMatch)
-            return false;
+        foreach (ra, record)
+        {
+          if (ra->first.length () > 11 &&
+              ra->first.substr (0, 11) == "annotation_")
+          {
+            if (att->match (ra->second))
+              ++annotation_pass_count;
+            else
+              ++annotation_fail_count;
+          }
        }
      }
      else if (! att->match (Att ()))
        return false;
+
+      // This innocuous little if-else took significantly more thinking and
+      // debugging than anything else in task.  Only change this code if you
+      // are willing to invest a week understanding and testing it.
+      if (att->modType (att->mod ()) == "positive")
+      {
+        if (! (description_result || annotation_pass_count > 0))
+          return false;
+      }
+      else
+      {
+        if (!description_result || annotation_fail_count > 0)
+          return false;
+      }
    }

-    // Annotations are skipped.
+    // Annotations are skipped, because they are handled above.
    else if (att->name ().length () > 11 &&
             att->name ().substr (0, 11) == "annotation_")
    {
--- a/src/Nibbler.cpp
+++ b/src/Nibbler.cpp
@@ -243,7 +243,7 @@ bool Nibbler::getInt (int& result)
      ++i;
  }

-  while (i < mInput.length () && ::isdigit (mInput[i]))
+  while (i < mInput.length () && isdigit (mInput[i]))
    ++i;

  if (i > mCursor)
@@ -260,7 +260,7 @@ bool Nibbler::getInt (int& result)
 bool Nibbler::getUnsignedInt (int& result)
 {
  std::string::size_type i = mCursor;
-  while (i < mInput.length () && ::isdigit (mInput[i]))
+  while (i < mInput.length () && isdigit (mInput[i]))
    ++i;

  if (i > mCursor)
--- a/src/Permission.cpp
+++ b/src/Permission.cpp
@@ -57,8 +57,15 @@ bool Permission::confirmed (const Task& task, const std::string& question)
            << task.id
            << " \""
            << task.get ("description")
-            << "\""
-            << std::endl;
+            << "\"";
+
+  if (task.getStatus () == Task::recurring ||
+      task.has ("parent"))
+  {
+    std::cout << " (Recurring)";
+  }
+
+  std::cout << std::endl;

  int answer = confirm3 (question);
  if (answer == 2)
--- a/src/Sequence.cpp
+++ b/src/Sequence.cpp
@@ -153,7 +153,7 @@ bool Sequence::validId (const std::string& input) const
    return false;

  for (size_t i = 0; i < input.length (); ++i)
-    if (!::isdigit (input[i]))
+    if (!isdigit (input[i]))
      return false;

  return true;
--- a/src/Table.cpp
+++ b/src/Table.cpp
@@ -876,7 +876,7 @@ void Table::sort (std::vector <int>& order)
    if (gap > 1)
    {
      gap = (int) ((float)gap / 1.3);
-      if (gap == 10 or gap == 9)
+      if (gap == 10 || gap == 9)
        gap = 11;
    }

--- a/src/Task.cpp
+++ b/src/Task.cpp
@@ -67,8 +67,8 @@ bool Task::operator== (const Task& other)
    return false;

  foreach (att, *this)
-    if (att->first != "uuid")
-      if (att->second.value () != other.get (att->first))
+    if (att->second.name () != "uuid")
+      if (att->second.value () != other.get (att->second.name ()))
        return false;

  return true;
@@ -134,7 +134,7 @@ void Task::setEntry ()
 }

 ////////////////////////////////////////////////////////////////////////////////
-Task::status Task::getStatus ()
+Task::status Task::getStatus () const
 {
  return textToStatus (get ("status")); // No i18n
 }
@@ -363,12 +363,12 @@ std::string Task::composeCSV () const
  out <<        get ("start")    <<  ","; // No i18n

  if (has ("due"))
-    out << "'" << get ("due")      << "',"; // No i18n
+    out <<      get ("due")      <<  ","; // No i18n
  else
    out << ","; // No i18n

  if (has ("recur"))
-    out << "'" << get ("recur")    << "',"; // No i18n
+    out <<      get ("recur")    <<  ","; // No i18n
  else
    out << ","; // No i18n

@@ -441,10 +441,14 @@ void Task::addAnnotation (const std::string& description)
 void Task::removeAnnotations ()
 {
  // Erase old annotations.
-  Record::iterator i;
-  for (i = this->begin (); i != this->end (); ++i)
+  Record::iterator i = this->begin ();
+  while (i != this->end ())
+  {
    if (i->first.substr (0, 11) == "annotation_") // No i18n
-      this->erase (i);
+      this->erase (i++);
+    else
+      i++;
+  }
 }

 ////////////////////////////////////////////////////////////////////////////////
--- a/src/Task.h
+++ b/src/Task.h
@@ -57,7 +57,7 @@ public:

  void setEntry ();

-  status getStatus ();
+  status getStatus () const;
  void setStatus (status);

  int getTagCount ();
--- a/src/Timer.cpp
+++ b/src/Timer.cpp
@@ -52,7 +52,7 @@ Timer::~Timer ()
    << mDescription
    << " "
    << std::setprecision (6)
-    << std::fixed
+//    << std::fixed
    << ((end.tv_sec - mStart.tv_sec) + ((end.tv_usec - mStart.tv_usec )
       / 1000000.0))
    << " sec";
--- a/src/command.cpp
+++ b/src/command.cpp
@@ -49,7 +49,7 @@
 extern Context context;

 ////////////////////////////////////////////////////////////////////////////////
-std::string handleAdd ()
+int handleAdd (std::string &outs)
 {
  std::stringstream out;

@@ -84,6 +84,15 @@ std::string handleAdd ()
  foreach (tag, context.tagAdditions)
    context.task.addTag (*tag);

+  // Perform some logical consistency checks.
+  if (context.task.has ("recur") &&
+      !context.task.has ("due"))
+    throw std::string ("You cannot specify a recurring task without a due date.");
+
+  if (context.task.has ("until")  &&
+      !context.task.has ("recur"))
+    throw std::string ("You cannot specify an until date for a non-recurring task.");
+
  // Only valid tasks can be added.
  context.task.validate ();

@@ -101,19 +110,20 @@ std::string handleAdd ()
  context.tdb.commit ();
  context.tdb.unlock ();

-  return out.str ();
+  outs = out.str ();
+  return 0;
 }

 ////////////////////////////////////////////////////////////////////////////////
-std::string handleProjects ()
+int handleProjects (std::string &outs)
 {
+  int rc = 0;
  std::stringstream out;

  context.filter.push_back (Att ("status", "pending"));

  std::vector <Task> tasks;
  context.tdb.lock (context.config.get ("locking", true));
-  handleRecurrence ();
  int quantity = context.tdb.loadPending (tasks, context.filter);
  context.tdb.commit ();
  context.tdb.unlock ();
@@ -187,19 +197,21 @@ std::string handleProjects ()
        << " (" << quantity << (quantity == 1 ? " task" : " tasks") << ")"
        << std::endl;
  }
-  else
+  else {
    out << "No projects."
        << std::endl;
+    rc = 1;
+  }

-  return out.str ();
+  outs = out.str ();
+  return rc;
 }

 ////////////////////////////////////////////////////////////////////////////////
-std::string handleCompletionProjects ()
+int handleCompletionProjects (std::string &outs)
 {
  std::vector <Task> tasks;
  context.tdb.lock (context.config.get ("locking", true));
-  handleRecurrence ();

  Filter filter;
  if (context.config.get (std::string ("complete.all.projects"), false))
@@ -221,19 +233,20 @@ std::string handleCompletionProjects ()
    if (project->first.length ())
      out << project->first << std::endl;

-  return out.str ();
+  outs = out.str ();
+  return 0;
 }

 ////////////////////////////////////////////////////////////////////////////////
-std::string handleTags ()
+int handleTags (std::string &outs)
 {
+  int rc = 0;
  std::stringstream out;

  context.filter.push_back (Att ("status", "pending"));

  std::vector <Task> tasks;
  context.tdb.lock (context.config.get ("locking", true));
-  handleRecurrence ();
  int quantity = context.tdb.loadPending (tasks, context.filter);
  context.tdb.commit ();
  context.tdb.unlock ();
@@ -284,19 +297,22 @@ std::string handleTags ()
        << " (" << quantity << (quantity == 1 ? " task" : " tasks") << ")"
        << std::endl;
  }
-  else
+  else {
    out << "No tags."
        << std::endl;
+    rc = 1;
+  }

-  return out.str ();
+  outs = out.str ();
+
+  return rc;
 }

 ////////////////////////////////////////////////////////////////////////////////
-std::string handleCompletionTags ()
+int handleCompletionTags (std::string &outs)
 {
  std::vector <Task> tasks;
  context.tdb.lock (context.config.get ("locking", true));
-  handleRecurrence ();

  Filter filter;
  if (context.config.get (std::string ("complete.all.tags"), false))
@@ -323,11 +339,12 @@ std::string handleCompletionTags ()
  foreach (tag, unique)
    out << tag->first << std::endl;

-  return out.str ();
+  outs = out.str ();
+  return 0;
 }

 ////////////////////////////////////////////////////////////////////////////////
-std::string handleCompletionCommands ()
+int handleCompletionCommands (std::string &outs)
 {
  std::vector <std::string> commands;
  context.cmd.allCommands (commands);
@@ -337,11 +354,12 @@ std::string handleCompletionCommands ()
  foreach (command, commands)
    out << *command << std::endl;

-  return out.str ();
+  outs = out.str ();
+  return 0;
 }

 ////////////////////////////////////////////////////////////////////////////////
-std::string handleCompletionConfig ()
+int handleCompletionConfig (std::string &outs)
 {
  std::vector <std::string> configs;
  context.config.all (configs);
@@ -351,15 +369,15 @@ std::string handleCompletionConfig ()
  foreach (config, configs)
    out << *config << std::endl;

-  return out.str ();
+  outs = out.str ();
+  return 0;
 }

 ////////////////////////////////////////////////////////////////////////////////
-std::string handleCompletionIDs ()
+int handleCompletionIDs (std::string &outs)
 {
  std::vector <Task> tasks;
  context.tdb.lock (context.config.get ("locking", true));
-  handleRecurrence ();
  Filter filter;
  context.tdb.loadPending (tasks, filter);
  context.tdb.commit ();
@@ -377,20 +395,24 @@ std::string handleCompletionIDs ()
  foreach (id, ids)
    out << *id << std::endl;

-  return out.str ();
+  outs = out.str ();
+  return 0;
 }

 ////////////////////////////////////////////////////////////////////////////////
 void handleUndo ()
 {
+  context.disallowModification ();
+
  context.tdb.lock (context.config.get ("locking", true));
  context.tdb.undo ();
  context.tdb.unlock ();
 }

 ////////////////////////////////////////////////////////////////////////////////
-std::string handleVersion ()
+int handleVersion (std::string &outs)
 {
+  int rc = 0;
  std::stringstream out;

  // Create a table for the disclaimer.
@@ -477,8 +499,8 @@ std::string handleVersion ()
    " blanklines bulk color color.active color.due color.overdue color.pri.H "
    "color.pri.L color.pri.M color.pri.none color.recurring color.tagged "
    "color.footnote color.header color.debug confirmation curses data.location "
-    "dateformat debug default.command default.priority defaultwidth due locale "
-    "displayweeknumber echo.command locking monthsperline nag next project "
+    "dateformat debug default.command default.priority default.project defaultwidth "
+    "due locale displayweeknumber echo.command locking monthsperline nag next project "
    "shadow.command shadow.file shadow.notify weekstart editor import.synonym.id "
    "import.synonym.uuid longversion complete.all.projects complete.all.tags "
 #ifdef FEATURE_SHELL
@@ -518,7 +540,7 @@ std::string handleVersion ()
  if (unrecognized.size ())
  {
    out << "Your .taskrc file contains these unrecognized variables:"
-        << std::endl;
+         << std::endl;

    foreach (i, unrecognized)
      out << "  " << *i << std::endl;
@@ -529,9 +551,11 @@ std::string handleVersion ()
  // Verify installation.  This is mentioned in the documentation as the way to
  // ensure everything is properly installed.

-  if (all.size () == 0)
+  if (all.size () == 0) {
    out << "Configuration error: .taskrc contains no entries"
        << std::endl;
+    rc = 1;
+  }
  else
  {
    if (context.config.get ("data.location") == "")
@@ -545,17 +569,20 @@ std::string handleVersion ()
          << std::endl;
  }

-  return out.str ();
+  outs = out.str ();
+  return rc;
 }

 ////////////////////////////////////////////////////////////////////////////////
-std::string handleDelete ()
+int handleDelete (std::string &outs)
 {
+  int rc = 0;
  std::stringstream out;

+  context.disallowModification ();
+
  std::vector <Task> tasks;
  context.tdb.lock (context.config.get ("locking", true));
-  handleRecurrence ();
  Filter filter;
  context.tdb.loadPending (tasks, filter);

@@ -638,24 +665,29 @@ std::string handleDelete ()
              << std::endl;
      }
    }
-    else
+    else {
      out << "Task not deleted." << std::endl;
+      rc  = 1;
+    }
  }

  context.tdb.commit ();
  context.tdb.unlock ();

-  return out.str ();
+  outs = out.str ();
+  return rc;
 }

 ////////////////////////////////////////////////////////////////////////////////
-std::string handleStart ()
+int handleStart (std::string &outs)
 {
+  int rc = 0;
  std::stringstream out;

+  context.disallowModification ();
+
  std::vector <Task> tasks;
  context.tdb.lock (context.config.get ("locking", true));
-  handleRecurrence ();
  Filter filter;
  context.tdb.loadPending (tasks, filter);

@@ -691,23 +723,27 @@ std::string handleStart ()
          << task->get ("description")
          << "' already started."
          << std::endl;
+      rc = 1;
    }
  }

  context.tdb.commit ();
  context.tdb.unlock ();

-  return out.str ();
+  outs = out.str ();
+  return rc;
 }

 ////////////////////////////////////////////////////////////////////////////////
-std::string handleStop ()
+int handleStop (std::string &outs)
 {
+  int rc = 0;
  std::stringstream out;

+  context.disallowModification ();
+
  std::vector <Task> tasks;
  context.tdb.lock (context.config.get ("locking", true));
-  handleRecurrence ();
  Filter filter;
  context.tdb.loadPending (tasks, filter);

@@ -737,24 +773,26 @@ std::string handleStop ()
          << task->get ("description")
          << "' not started."
          << std::endl;
+      rc = 1;
    }
  }

  context.tdb.commit ();
  context.tdb.unlock ();

-  return out.str ();
+  outs = out.str ();
+  return rc;
 }

 ////////////////////////////////////////////////////////////////////////////////
-std::string handleDone ()
+int handleDone (std::string &outs)
 {
+  int rc = 0;
  int count = 0;
  std::stringstream out;

  std::vector <Task> tasks;
  context.tdb.lock (context.config.get ("locking", true));
-  handleRecurrence ();
  Filter filter;
  context.tdb.loadPending (tasks, filter);

@@ -818,6 +856,7 @@ std::string handleDone ()
          << task->get ("description")
          << "' is not pending"
          << std::endl;
+      rc = 1;
  }

  context.tdb.commit ();
@@ -831,11 +870,12 @@ std::string handleDone ()
        << " as done"
        << std::endl;

-  return out.str ();
+  outs = out.str ();
+  return rc;
 }

 ////////////////////////////////////////////////////////////////////////////////
-std::string handleExport ()
+int handleExport (std::string &outs)
 {
  std::stringstream out;

@@ -874,18 +914,18 @@ std::string handleExport ()
    }
  }

-  return out.str ();
+  outs = out.str ();
+  return 0;
 }

 ////////////////////////////////////////////////////////////////////////////////
-std::string handleModify ()
+int handleModify (std::string &outs)
 {
  int count = 0;
  std::stringstream out;

  std::vector <Task> tasks;
  context.tdb.lock (context.config.get ("locking", true));
-  handleRecurrence ();
  Filter filter;
  context.tdb.loadPending (tasks, filter);

@@ -918,6 +958,13 @@ std::string handleModify ()
           task->get ("parent") == other->get ("parent")) || // Sibling
          other->get ("uuid")   == task->get ("parent"))     // Parent
      {
+        if (task->has ("parent"))
+          std::cout << "Task "
+                    << task->id
+                    << " is a recurring task, and all other instances of this"
+                    << " task may be modified."
+                    << std::endl;
+
        Task before (*other);

        // A non-zero value forces a file write.
@@ -952,18 +999,18 @@ std::string handleModify ()
  if (context.config.get ("echo.command", true))
    out << "Modified " << count << " task" << (count == 1 ? "" : "s") << std::endl;

-  return out.str ();
+  outs = out.str ();
+  return 0;
 }

 ////////////////////////////////////////////////////////////////////////////////
-std::string handleAppend ()
+int handleAppend (std::string &outs)
 {
  int count = 0;
  std::stringstream out;

  std::vector <Task> tasks;
  context.tdb.lock (context.config.get ("locking", true));
-  handleRecurrence ();
  Filter filter;
  context.tdb.loadPending (tasks, filter);

@@ -1020,11 +1067,12 @@ std::string handleAppend ()
  if (context.config.get ("echo.command", true))
    out << "Appended " << count << " task" << (count == 1 ? "" : "s") << std::endl;

-  return out.str ();
+  outs = out.str ();
+  return 0;
 }

 ////////////////////////////////////////////////////////////////////////////////
-std::string handleDuplicate ()
+int handleDuplicate (std::string &outs)
 {
  std::stringstream out;
  int count = 0;
@@ -1089,7 +1137,8 @@ std::string handleDuplicate ()
  if (context.config.get ("echo.command", true))
    out << "Duplicated " << count << " task" << (count == 1 ? "" : "s") << std::endl;

-  return out.str ();
+  outs = out.str ();
+  return 0;
 }

 ////////////////////////////////////////////////////////////////////////////////
@@ -1168,8 +1217,9 @@ void handleShell ()
 #endif

 ////////////////////////////////////////////////////////////////////////////////
-std::string handleColor ()
+int handleColor (std::string &outs)
 {
+  int rc = 0;
  std::stringstream out;
  if (context.config.get ("color", true) || context.config.get (std::string ("_forcecolor"), false))
  {
@@ -1251,13 +1301,15 @@ std::string handleColor ()
    out << "Color is currently turned off in your .taskrc file.  "
           "To enable color, create the entry 'color=on'."
        << std::endl;
+    rc = 1;
  }

-  return out.str ();
+  outs = out.str ();
+  return rc;
 }

 ////////////////////////////////////////////////////////////////////////////////
-std::string handleAnnotate ()
+int handleAnnotate (std::string &outs)
 {
  if (!context.task.has ("description"))
    throw std::string ("Cannot apply a blank annotation.");
@@ -1301,7 +1353,8 @@ std::string handleAnnotate ()
  context.tdb.commit ();
  context.tdb.unlock ();

-  return out.str ();
+  outs = out.str ();
+  return 0;
 }

 ////////////////////////////////////////////////////////////////////////////////
@@ -1359,12 +1412,12 @@ int deltaAttributes (Task& task)

  foreach (att, context.task)
  {
-    if (att->first != "uuid"        &&
-        att->first != "description" &&
-        att->first != "tags")
+    if (att->second.name () != "uuid"        &&
+        att->second.name () != "description" &&
+        att->second.name () != "tags")
    {
      // Modifying "wait" changes status.
-      if (att->first == "wait")
+      if (att->second.name () == "wait")
      {
        if (att->second.value () == "")
          task.setStatus (Task::pending);
@@ -1373,8 +1426,9 @@ int deltaAttributes (Task& task)
      }

      if (att->second.value () == "")
-        task.remove (att->first);
+        task.remove (att->second.name ());
      else
+        // One of the few places where the compound attribute name is used.
        task.set (att->first, att->second.value ());

      ++changes;
--- a/src/custom.cpp
+++ b/src/custom.cpp
@@ -52,7 +52,7 @@ static std::vector <std::string> customReports;
 ////////////////////////////////////////////////////////////////////////////////
 // This report will eventually become the one report that many others morph into
 // via the .taskrc file.
-std::string handleCustomReport (const std::string& report)
+int handleCustomReport (const std::string& report, std::string &outs)
 {
  // Load report configuration.
  std::string columnList = context.config.get ("report." + report + ".columns");
@@ -94,21 +94,24 @@ std::string handleCustomReport (const std::string& report)
    labelList,
    sortList,
    filterList,
-    tasks);
+    tasks,
+    outs);
 }

 ////////////////////////////////////////////////////////////////////////////////
 // This report will eventually become the one report that many others morph into
 // via the .taskrc file.

-std::string runCustomReport (
+int runCustomReport (
  const std::string& report,
  const std::string& columnList,
  const std::string& labelList,
  const std::string& sortList,
  const std::string& filterList,
-  std::vector <Task>& tasks)
+  std::vector <Task>& tasks,
+  std::string &outs)
 {
+  int rc = 0;
  // Load report configuration.
  std::vector <std::string> columns;
  split (columns, columnList, ',');
@@ -547,11 +550,14 @@ std::string runCustomReport (
        << table.rowCount ()
        << (table.rowCount () == 1 ? " task" : " tasks")
        << std::endl;
-  else
+  else {
    out << "No matches."
        << std::endl;
+    rc = 1;
+  }

-  return out.str ();
+  outs = out.str ();
+  return rc;
 }

 ////////////////////////////////////////////////////////////////////////////////
--- a/src/edit.cpp
+++ b/src/edit.cpp
@@ -150,7 +150,7 @@ static std::string formatTask (Task task)
         << "  Due:               " << formatDate (task, "due")                         << std::endl
         << "  Until:             " << formatDate (task, "until")                       << std::endl
         << "  Recur:             " << task.get ("recur")                               << std::endl
-         << "  Wait until:        " << task.get ("wait")                                << std::endl
+         << "  Wait until:        " << formatDate (task, "wait")                        << std::endl
         << "  Parent:            " << task.get ("parent")                              << std::endl
         << "  Foreground color:  " << task.get ("fg")                                  << std::endl
         << "  Background color:  " << task.get ("bg")                                  << std::endl
@@ -499,7 +499,13 @@ static void parseTask (Task& task, const std::string& after)
      std::string::size_type gap = value.find (" ");
      if (gap != std::string::npos)
      {
-        Date when (value.substr (0, gap));
+        Date when (value.substr (0, gap), context.config.get ("dateformat", "m/d/Y"));
+
+        // This guarantees that if more than one annotation has the same date,
+        // that the seconds will be different, thus unique, thus not squashed.
+        // Bug #249
+        when += (const int) annotations.size ();
+
        std::stringstream name;
        name << "annotation_" << when.toEpoch ();
        std::string text = trim (value.substr (gap, std::string::npos), "\t ");
@@ -537,7 +543,7 @@ void editFile (Task& task)

  // Complete the command line.
  editor += " ";
-  editor += file.str ();
+  editor += "\"" + file.str () + "\"";

 ARE_THESE_REALLY_HARMFUL:
  // Launch the editor.
@@ -593,7 +599,7 @@ ARE_THESE_REALLY_HARMFUL:
 // Introducing the Silver Bullet.  This feature is the catch-all fixative for
 // various other ills.  This is like opening up the hood and going in with a
 // wrench.  To be used sparingly.
-std::string handleEdit ()
+int handleEdit (std::string &outs)
 {
  std::stringstream out;

@@ -637,7 +643,8 @@ std::string handleEdit ()
  context.tdb.commit ();
  context.tdb.unlock ();

-  return out.str ();
+  outs = out.str ();
+  return 0;
 }

 ////////////////////////////////////////////////////////////////////////////////
--- a/src/import.cpp
+++ b/src/import.cpp
@@ -106,16 +106,16 @@ static fileType determineFileType (const std::vector <std::string>& lines)
    {
      if (           lines[i][0] == 'x' &&
                     lines[i][1] == ' ' &&
-          ::isdigit (lines[i][2]) &&
-          ::isdigit (lines[i][3]) &&
-          ::isdigit (lines[i][4]) &&
-          ::isdigit (lines[i][5]) &&
+          isdigit (lines[i][2]) &&
+          isdigit (lines[i][3]) &&
+          isdigit (lines[i][4]) &&
+          isdigit (lines[i][5]) &&
                     lines[i][6] == '-' &&
-          ::isdigit (lines[i][7]) &&
-          ::isdigit (lines[i][8]) &&
+          isdigit (lines[i][7]) &&
+          isdigit (lines[i][8]) &&
                     lines[i][9] == '-' &&
-          ::isdigit (lines[i][10]) &&
-          ::isdigit (lines[i][11]))
+          isdigit (lines[i][10]) &&
+          isdigit (lines[i][11]))
        return todo_sh_2_0;
    }

@@ -126,13 +126,13 @@ static fileType determineFileType (const std::vector <std::string>& lines)
      // +project
      if (words[w].length () > 1 &&
          words[w][0] == '+'     &&
-          ::isalnum (words[w][1]))
+          isalnum (words[w][1]))
        return todo_sh_2_0;

      // @context
      if (words[w].length () > 1 &&
          words[w][0] == '@'     &&
-          ::isalnum (words[w][1]))
+          isalnum (words[w][1]))
        return todo_sh_2_0;
    }
  }
@@ -666,6 +666,7 @@ static std::string importTask_1_6_0 (const std::vector <std::string>& lines)
 static std::string importTaskCmdLine (const std::vector <std::string>& lines)
 {
  std::vector <std::string> failed;
+  std::string unused;

  std::vector <std::string>::const_iterator it;
  for (it = lines.begin (); it != lines.end (); ++it)
@@ -680,7 +681,7 @@ static std::string importTaskCmdLine (const std::vector <std::string>& lines)
      context.task.clear ();
      context.cmd.command = "";
      context.parse ();
-      handleAdd ();
+      (void)handleAdd (unused);
      context.clearMessages ();
    }

@@ -789,6 +790,8 @@ static std::string importTodoSh_2_0 (const std::vector <std::string>& lines)
      context.parse ();
      decorateTask (context.task);

+      context.task.set ("uuid", uuid ());
+
      if (isPending)
      {
        context.task.setStatus (Task::pending);
@@ -1144,7 +1147,7 @@ static std::string importCSV (const std::vector <std::string>& lines)
 }

 ////////////////////////////////////////////////////////////////////////////////
-std::string handleImport ()
+int handleImport (std::string &outs)
 {
  std::stringstream out;

@@ -1212,7 +1215,8 @@ std::string handleImport ()
  else
    throw std::string ("You must specify a file to import.");

-  return out.str ();
+  outs = out.str ();
+  return 0;
 }

 ////////////////////////////////////////////////////////////////////////////////
--- a/src/interactive.cpp
+++ b/src/interactive.cpp
@@ -34,7 +34,7 @@
 #include "Context.h"
 //#include "text.h"
 //#include "util.h"
-//#include "main.h"
+#include "main.h"
 #include "i18n.h"
 #include "../auto.h"

--- a/src/main.h
+++ b/src/main.h
@@ -55,25 +55,25 @@ int getDueState (const std::string&);
 bool nag (Task&);

 // command.cpp
-std::string handleAdd ();
-std::string handleAppend ();
-std::string handleExport ();
-std::string handleDone ();
-std::string handleModify ();
-std::string handleProjects ();
-std::string handleCompletionProjects ();
-std::string handleTags ();
-std::string handleCompletionTags ();
-std::string handleCompletionCommands ();
-std::string handleCompletionIDs ();
-std::string handleCompletionConfig ();
-std::string handleVersion ();
-std::string handleDelete ();
-std::string handleStart ();
-std::string handleStop ();
-std::string handleColor ();
-std::string handleAnnotate ();
-std::string handleDuplicate ();
+int handleAdd (std::string &);
+int handleAppend (std::string &);
+int handleExport (std::string &);
+int handleDone (std::string &);
+int handleModify (std::string &);
+int handleProjects (std::string &);
+int handleCompletionProjects (std::string &);
+int handleTags (std::string &);
+int handleCompletionTags (std::string &);
+int handleCompletionCommands (std::string &);
+int handleCompletionIDs (std::string &);
+int handleCompletionConfig (std::string &);
+int handleVersion (std::string &);
+int handleDelete (std::string &);
+int handleStart (std::string &);
+int handleStop (std::string &);
+int handleColor (std::string &);
+int handleAnnotate (std::string &);
+int handleDuplicate (std::string &);
 void handleUndo ();
 #ifdef FEATURE_SHELL
 void handleShell ();
@@ -85,27 +85,28 @@ int deltaAttributes (Task&);
 int deltaSubstitutions (Task&);

 // edit.cpp
-std::string handleEdit ();
+int handleEdit (std::string &);

 // report.cpp
-std::string shortUsage ();
-std::string longUsage ();
-std::string handleInfo ();
-std::string handleReportSummary ();
-std::string handleReportNext ();
-std::string handleReportHistory ();
-std::string handleReportGHistory ();
-std::string handleReportCalendar ();
-std::string handleReportStats ();
-std::string handleReportTimesheet ();
+int shortUsage (std::string &);
+int longUsage (std::string &);
+int handleInfo (std::string &);
+int handleReportSummary (std::string &);
+int handleReportNext (std::string &);
+int handleReportHistory (std::string &);
+int handleReportGHistory (std::string &);
+int handleReportCalendar (std::string &);
+int handleReportStats (std::string &);
+int handleReportTimesheet (std::string &);
 std::string getFullDescription (Task&);
 std::string getDueDate (Task&);

 // custom.cpp
-std::string handleCustomReport (const std::string&);
-std::string runCustomReport (const std::string&, const std::string&,
-                             const std::string&, const std::string&,
-                             const std::string&, std::vector <Task>&);
+int handleCustomReport (const std::string&, std::string &);
+int runCustomReport (const std::string&, const std::string&,
+                     const std::string&, const std::string&,
+                     const std::string&, std::vector <Task>&,
+		     std::string&);
 void validReportColumns (const std::vector <std::string>&);
 void validSortColumns (const std::vector <std::string>&, const std::vector <std::string>&);

@@ -118,7 +119,7 @@ std::string colorizeFootnote (const std::string&);
 std::string colorizeDebug (const std::string&);

 // import.cpp
-std::string handleImport ();
+int handleImport (std::string&);

 // list template
 ///////////////////////////////////////////////////////////////////////////////
--- a/src/report.cpp
+++ b/src/report.cpp
@@ -50,7 +50,7 @@
 extern Context context;

 ////////////////////////////////////////////////////////////////////////////////
-std::string shortUsage ()
+int shortUsage (std::string &outs)
 {
  Table table;

@@ -209,14 +209,19 @@ std::string shortUsage ()
      << std::endl
      << std::endl;

-  return out.str ();
+  outs = out.str ();
+  return 0;
 }

 ////////////////////////////////////////////////////////////////////////////////
-std::string longUsage ()
+int longUsage (std::string &outs)
 {
+  std::string shortUsageString;
  std::stringstream out;
-  out << shortUsage ()
+
+  (void)shortUsage(shortUsageString);
+
+  out << shortUsageString
      << "ID is the numeric identifier displayed by the 'task list' command. "
      << "You can specify multiple IDs for task commands, and multiple tasks "
      << "will be affected.  To specify multiple IDs make sure you use one "
@@ -276,13 +281,15 @@ std::string longUsage ()
      << "  $ ! ' \" ( ) ; \\ ` * ? { } [ ] < > | & % # ~"                    << "\n"
      << std::endl;

-  return out.str ();
+  outs = out.str();
+  return 0;
 }

 ////////////////////////////////////////////////////////////////////////////////
 // Display all information for the given task.
-std::string handleInfo ()
+int handleInfo (std::string &outs)
 {
+  int rc = 0;
  // Get all the tasks.
  std::vector <Task> tasks;
  context.tdb.lock (context.config.get ("locking", true));
@@ -497,18 +504,22 @@ std::string handleInfo ()
        << std::endl;
  }

-  if (! tasks.size ())
+  if (! tasks.size ()) {
    out << "No matches." << std::endl;
+    rc = 1;
+  }

-  return out.str ();
+  outs = out.str ();
+  return rc;
 }

 ////////////////////////////////////////////////////////////////////////////////
 // Project  Remaining  Avg Age  Complete  0%                  100%
 // A               12      13d       55%  XXXXXXXXXXXXX-----------
 // B              109   3d 12h       10%  XXX---------------------
-std::string handleReportSummary ()
+int handleReportSummary (std::string &outs)
 {
+  int rc = 0;
  // Scan the pending tasks.
  std::vector <Task> tasks;
  context.tdb.lock (context.config.get ("locking", true));
@@ -648,10 +659,13 @@ std::string handleReportSummary ()
        << table.rowCount ()
        << (table.rowCount () == 1 ? " project" : " projects")
        << std::endl;
-  else
+  else {
    out << "No projects." << std::endl;
+    rc = 1;
+  }

-  return out.str ();
+  outs = out.str ();
+  return rc;
 }

 ////////////////////////////////////////////////////////////////////////////////
@@ -673,7 +687,7 @@ std::string handleReportSummary ()
 //
 // Make the "three" tasks a configurable number
 //
-std::string handleReportNext ()
+int handleReportNext (std::string &outs)
 {
  // Load report configuration.
  std::string columnList = context.config.get ("report.next.columns");
@@ -718,7 +732,8 @@ std::string handleReportNext ()
    labelList,
    sortList,
    filterList,
-    tasks);
+    tasks,
+    outs);
 }

 ////////////////////////////////////////////////////////////////////////////////
@@ -744,8 +759,9 @@ time_t monthlyEpoch (const std::string& date)
  return 0;
 }

-std::string handleReportHistory ()
+int handleReportHistory (std::string &outs)
 {
+  int rc = 0;
  std::map <time_t, int> groups;          // Represents any month with data
  std::map <time_t, int> addedGroup;      // Additions by month
  std::map <time_t, int> completedGroup;  // Completions by month
@@ -891,15 +907,19 @@ std::string handleReportHistory ()
    out << optionalBlankLine ()
        << table.render ()
        << std::endl;
-  else
+  else {
    out << "No tasks." << std::endl;
+    rc = 1;
+  }

-  return out.str ();
+  outs = out.str ();
+  return rc;
 }

 ////////////////////////////////////////////////////////////////////////////////
-std::string handleReportGHistory ()
+int handleReportGHistory (std::string &outs)
 {
+  int rc = 0;
  std::map <time_t, int> groups;          // Represents any month with data
  std::map <time_t, int> addedGroup;      // Additions by month
  std::map <time_t, int> completedGroup;  // Completions by month
@@ -1087,14 +1107,17 @@ std::string handleReportGHistory ()
    else
      out << "Legend: + added, X completed, - deleted" << std::endl;
  }
-  else
+  else {
    out << "No tasks." << std::endl;
+    rc = 1;
+  }

-  return out.str ();
+  outs = out.str ();
+  return rc;
 }

 ////////////////////////////////////////////////////////////////////////////////
-std::string handleReportTimesheet ()
+int handleReportTimesheet (std::string &outs)
 {
  // Scan the pending tasks.
  std::vector <Task> tasks;
@@ -1264,7 +1287,8 @@ std::string handleReportTimesheet ()
    end   -= 7 * 86400;
  }

-  return out.str ();
+  outs = out.str ();
+  return 0;
 }

 ////////////////////////////////////////////////////////////////////////////////
@@ -1376,9 +1400,9 @@ std::string renderMonths (
      row = 0;

    // Loop through days in month and add to table.
-    for (int d = 1; d <= daysInMonth.at (mpl); ++d)
+    for (int d = 1; d <= daysInMonth[mpl]; ++d)
    {
-      Date temp (months.at (mpl), d, years.at (mpl));
+      Date temp (months[mpl], d, years[mpl]);
      int dow = temp.dayOfWeek ();
      int woy = temp.weekOfYear (weekStart);

@@ -1396,9 +1420,9 @@ std::string renderMonths (
      table.addCell (row, thisCol, d);

      if ((context.config.get ("color", true) || context.config.get (std::string ("_forcecolor"), false)) &&
-          today.day ()   == d              &&
-          today.month () == months.at (mpl)  &&
-          today.year ()  == years.at (mpl))
+          today.day ()   == d            &&
+          today.month () == months[mpl]  &&
+          today.year ()  == years[mpl])
        table.setCellFg (row, thisCol, Text::cyan);

      foreach (task, all)
@@ -1410,8 +1434,8 @@ std::string renderMonths (

          if ((context.config.get ("color", true) || context.config.get (std::string ("_forcecolor"), false)) &&
              due.day ()   == d               &&
-              due.month () == months.at (mpl) &&
-              due.year ()  == years.at (mpl))
+              due.month () == months[mpl] &&
+              due.year ()  == years[mpl])
          {
            table.setCellFg (row, thisCol, Text::black);
            table.setCellBg (row, thisCol, due < today ? Text::on_red : Text::on_yellow);
@@ -1423,7 +1447,7 @@ std::string renderMonths (
      int eow = 6;
      if (weekStart == 1)
        eow = 0;
-      if (dow == eow && d < daysInMonth.at (mpl))
+      if (dow == eow && d < daysInMonth[mpl])
        row++;
    }
  }
@@ -1432,7 +1456,7 @@ std::string renderMonths (
 }

 ////////////////////////////////////////////////////////////////////////////////
-std::string handleReportCalendar ()
+int handleReportCalendar (std::string &outs)
 {
  // Each month requires 28 text columns width.  See how many will actually
  // fit.  But if a preference is specified, and it fits, use it.
@@ -1604,11 +1628,12 @@ std::string handleReportCalendar ()
        << optionalBlankLine ()
        << std::endl;

-  return out.str ();
+  outs = out.str ();
+  return 0;
 }

 ////////////////////////////////////////////////////////////////////////////////
-std::string handleReportStats ()
+int handleReportStats (std::string &outs)
 {
  std::stringstream out;

@@ -1836,7 +1861,8 @@ std::string handleReportStats ()
      << table.render ()
      << optionalBlankLine ();

-  return out.str ();
+  outs = out.str ();
+  return 0;
 }

 ////////////////////////////////////////////////////////////////////////////////
--- a/src/tests/att.t.cpp
+++ b/src/tests/att.t.cpp
@@ -75,9 +75,9 @@ int main (int argc, char** argv)
  Att a5 ("name", "value");
  t.is (a5.composeF4 (), "name:\"value\"", "Att::composeF4 simple");
  a5.value ("\"");
-  t.is (a5.composeF4 (), "name:\"&quot;\"", "Att::composeF4 encoded \"");
+  t.is (a5.composeF4 (), "name:\"&dquot;\"", "Att::composeF4 encoded \"");
  a5.value ("\t\",[]:");
-  t.is (a5.composeF4 (), "name:\"&tab;&quot;&comma;&open;&close;&colon;\"", "Att::composeF4 fully encoded \\t\",[]:");
+  t.is (a5.composeF4 (), "name:\"&tab;&dquot;&comma;&open;&close;&colon;\"", "Att::composeF4 fully encoded \\t\",[]:");

  Att a6 ("name", 6);
  t.is (a6.value_int (), 6, "Att::value_int get");
@@ -174,12 +174,12 @@ int main (int argc, char** argv)

  n = Nibbler ("name:\"&quot;\"");
  a7.parse (n);
-  t.is (a7.composeF4 (), "name:\"&quot;\"",
+  t.is (a7.composeF4 (), "name:\"&dquot;\"",
         "Att::parse (name:\"&quot;\")");

  n = Nibbler ("name:\"&tab;&quot;&comma;&open;&close;&colon;\"");
  a7.parse (n);
-  t.is (a7.composeF4 (), "name:\"&tab;&quot;&comma;&open;&close;&colon;\"",
+  t.is (a7.composeF4 (), "name:\"&tab;&dquot;&comma;&open;&close;&colon;\"",
             "Att::parse (name:\"&tab;&quot;&comma;&open;&close;&colon;\")");

  n = Nibbler ("total gibberish");
@@ -216,19 +216,19 @@ int main (int argc, char** argv)
  good = true;
  try {a7.parse (n);} catch (...) {good = false;}
  t.ok (good, "Att::parse (name:\"value)");
-  t.is (a7.composeF4 (), "name:\"&quot;value\"", "Att::composeF4 -> name:\"&quot;value\"");
+  t.is (a7.composeF4 (), "name:\"&dquot;value\"", "Att::composeF4 -> name:\"&dquot;value\"");

  n = Nibbler ("name:value\"");
  good = true;
  try {a7.parse (n);} catch (...) {good = false;}
  t.ok (good, "Att::parse (name:value\")");
-  t.is (a7.composeF4 (), "name:\"value&quot;\"", "Att::composeF4 -> name:\"value&quot;\"");
+  t.is (a7.composeF4 (), "name:\"value&dquot;\"", "Att::composeF4 -> name:\"value&dquot;\"");

  n = Nibbler ("name:val\"ue");
  good = true;
  try {a7.parse (n);} catch (...) {good = false;}
  t.ok (good, "Att::parse (name:val\"ue)");
-  t.is (a7.composeF4 (), "name:\"val&quot;ue\"", "Att::composeF4 -> name:\"val&quot;ue\"");
+  t.is (a7.composeF4 (), "name:\"val&dquot;ue\"", "Att::composeF4 -> name:\"val&dquot;ue\"");

  n = Nibbler ("name\"");
  good = true;
--- a/src/tests/basic.t
+++ b/src/tests/basic.t
@@ -40,7 +40,7 @@ if (open my $fh, '>', 'basic.rc')

 # Test the usage command.
 my $output = qx{../task rc:basic.rc};
-like ($output, qr/You must specify a command, or a task ID to modify/, 'missing command and ID');
+like ($output, qr/You must specify a command, or a task ID to modify/m, 'missing command and ID');

 # Test the version command.
 $output = qx{../task rc:basic.rc version};
--- a/src/tests/bug.before.t
+++ b/src/tests/bug.before.t
@@ -43,10 +43,10 @@ if (open my $fh, '>', 'before.rc')
 # Create some exampel data directly.
 if (open my $fh, '>', 'pending.data')
 {
-  # 1230000000 = 12/22/2008
+  # 1229947200 = 12/22/2008
  # 1240000000 = 4/17/2009
  print $fh <<EOF;
-[description:"foo" entry:"1230000000" start:"1230000000" status:"pending" uuid:"27097693-91c2-4cbe-ba89-ddcc87e5582c"]
+[description:"foo" entry:"1229947200" start:"1229947200" status:"pending" uuid:"27097693-91c2-4cbe-ba89-ddcc87e5582c"]
 [description:"bar" entry:"1240000000" start:"1240000000" status:"pending" uuid:"08f72d91-964c-424b-8fd5-556434648b6b"]
 EOF

--- a/src/tests/bug.hasnt.t
+++ b/src/tests/bug.hasnt.t
@@ -0,0 +1,106 @@
+#! /usr/bin/perl
+################################################################################
+## task - a command line task list manager.
+##
+## Copyright 2006 - 2009, Paul Beckingham.
+## All rights reserved.
+##
+## This program is free software; you can redistribute it and/or modify it under
+## the terms of the GNU General Public License as published by the Free Software
+## Foundation; either version 2 of the License, or (at your option) any later
+## version.
+##
+## This program is distributed in the hope that it will be useful, but WITHOUT
+## ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS
+## FOR A PARTICULAR PURPOSE.  See the GNU General Public License for more
+## details.
+##
+## You should have received a copy of the GNU General Public License along with
+## this program; if not, write to the
+##
+##     Free Software Foundation, Inc.,
+##     51 Franklin Street, Fifth Floor,
+##     Boston, MA
+##     02110-1301
+##     USA
+##
+################################################################################
+
+use strict;
+use warnings;
+use Test::More tests => 19;
+
+# Create the rc file.
+if (open my $fh, '>', 'hasnt.rc')
+{
+  print $fh "data.location=.\n",
+            "confirmation=no\n";
+  close $fh;
+  ok (-r 'hasnt.rc', 'Created hasnt.rc');
+}
+
+# 1
+qx{../task rc:hasnt.rc add foo};
+
+# 2
+qx{../task rc:hasnt.rc add foo};
+qx{../task rc:hasnt.rc 2 annotate bar};
+
+# 3
+qx{../task rc:hasnt.rc add foo};
+qx{../task rc:hasnt.rc 3 annotate bar};
+sleep 2;
+qx{../task rc:hasnt.rc 3 annotate baz};
+
+# 4
+qx{../task rc:hasnt.rc add bar};
+
+# 5
+qx{../task rc:hasnt.rc add bar};
+qx{../task rc:hasnt.rc 5 annotate foo};
+
+# 6
+qx{../task rc:hasnt.rc add bar};
+qx{../task rc:hasnt.rc 6 annotate foo};
+sleep 2;
+qx{../task rc:hasnt.rc 6 annotate baz};
+
+#7
+qx{../task rc:hasnt.rc add one};
+qx{../task rc:hasnt.rc 7 annotate two};
+sleep 2;
+qx{../task rc:hasnt.rc 7 annotate three};
+
+my $output = qx{../task rc:hasnt.rc ls description.has:foo};
+like   ($output, qr/\n 1/, '1 has foo -> yes');
+like   ($output, qr/\n 2/, '2 has foo -> yes');
+like   ($output, qr/\n 3/, '3 has foo -> yes');
+unlike ($output, qr/\n 4/, '4 has foo -> no');
+like   ($output, qr/\n 5/, '5 has foo -> yes');
+like   ($output, qr/\n 6/, '6 has foo -> yes');
+unlike ($output, qr/\n 7/, '7 has foo -> no');
+
+$output = qx{../task rc:hasnt.rc ls description.hasnt:foo};
+unlike ($output, qr/\n 1/, '1 hasnt foo -> no');
+unlike ($output, qr/\n 2/, '2 hasnt foo -> no');
+unlike ($output, qr/\n 3/, '3 hasnt foo -> no');
+like   ($output, qr/\n 4/, '4 hasnt foo -> yes');
+unlike ($output, qr/\n 5/, '5 hasnt foo -> no');
+unlike ($output, qr/\n 6/, '6 hasnt foo -> no');
+like   ($output, qr/\n 7/, '7 hasnt foo -> yes');
+
+# Cleanup.
+unlink 'pending.data';
+ok (!-r 'pending.data', 'Removed pending.data');
+
+unlink 'completed.data';
+ok (!-r 'completed.data', 'Removed completed.data');
+
+unlink 'undo.data';
+ok (!-r 'undo.data', 'Removed undo.data');
+
+unlink 'hasnt.rc';
+ok (!-r 'hasnt.rc', 'Removed hasnt.rc');
+
+exit 0;
+
--- a/src/tests/bug.range.t
+++ b/src/tests/bug.range.t
@@ -0,0 +1,65 @@
+#! /usr/bin/perl
+################################################################################
+## task - a command line task list manager.
+##
+## Copyright 2006 - 2009, Paul Beckingham.
+## All rights reserved.
+##
+## This program is free software; you can redistribute it and/or modify it under
+## the terms of the GNU General Public License as published by the Free Software
+## Foundation; either version 2 of the License, or (at your option) any later
+## version.
+##
+## This program is distributed in the hope that it will be useful, but WITHOUT
+## ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS
+## FOR A PARTICULAR PURPOSE.  See the GNU General Public License for more
+## details.
+##
+## You should have received a copy of the GNU General Public License along with
+## this program; if not, write to the
+##
+##     Free Software Foundation, Inc.,
+##     51 Franklin Street, Fifth Floor,
+##     Boston, MA
+##     02110-1301
+##     USA
+##
+################################################################################
+
+use strict;
+use warnings;
+use Test::More tests => 8;
+
+# Create the rc file.
+if (open my $fh, '>', 'range.rc')
+{
+  print $fh "data.location=.\n",
+            "confirmation=no\n";
+  close $fh;
+  ok (-r 'range.rc', 'Created range.rc');
+}
+
+# Add three tasks, and attempt to list the middle one within a range.
+qx{../task rc:range.rc add one due:8/1/2009};
+qx{../task rc:range.rc add two due:8/3/2009};
+qx{../task rc:range.rc add three due:8/5/2009};
+my $output = qx{../task rc:range.rc ls due.after:8/2/2009 due.before:8/4/2009};
+unlike ($output, qr/one/,   'Missing prior to range');
+like   ($output, qr/two/,   'Found within range');
+unlike ($output, qr/three/, 'Missing after range');
+
+# Cleanup.
+unlink 'pending.data';
+ok (!-r 'pending.data', 'Removed pending.data');
+
+unlink 'completed.data';
+ok (!-r 'completed.data', 'Removed completed.data');
+
+unlink 'undo.data';
+ok (!-r 'undo.data', 'Removed undo.data');
+
+unlink 'range.rc';
+ok (!-r 'range.rc', 'Removed range.rc');
+
+exit 0;
+
--- a/src/tests/bug.recur.t
+++ b/src/tests/bug.recur.t
@@ -0,0 +1,61 @@
+#! /usr/bin/perl
+################################################################################
+## task - a command line task list manager.
+##
+## Copyright 2006 - 2009, Paul Beckingham.
+## All rights reserved.
+##
+## This program is free software; you can redistribute it and/or modify it under
+## the terms of the GNU General Public License as published by the Free Software
+## Foundation; either version 2 of the License, or (at your option) any later
+## version.
+##
+## This program is distributed in the hope that it will be useful, but WITHOUT
+## ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS
+## FOR A PARTICULAR PURPOSE.  See the GNU General Public License for more
+## details.
+##
+## You should have received a copy of the GNU General Public License along with
+## this program; if not, write to the
+##
+##     Free Software Foundation, Inc.,
+##     51 Franklin Street, Fifth Floor,
+##     Boston, MA
+##     02110-1301
+##     USA
+##
+################################################################################
+
+use strict;
+use warnings;
+use Test::More tests => 6;
+
+# Create the rc file.
+if (open my $fh, '>', 'recur.rc')
+{
+  print $fh "data.location=.\n",
+            "confirmation=no\n";
+  close $fh;
+  ok (-r 'recur.rc', 'Created recur.rc');
+}
+
+# Add a recurring task with no due date, look for expected error.
+qx{../task rc:recur.rc add foo recur:daily};
+my $output = qx{../task rc:recur.rc info 1};
+unlike ($output, qr/Description\s+foo/, 'task not created - missing due date');
+
+# Cleanup.
+unlink 'pending.data';
+ok (!-r 'pending.data', 'Removed pending.data');
+
+unlink 'completed.data';
+ok (!-r 'completed.data', 'Removed completed.data');
+
+unlink 'undo.data';
+ok (!-r 'undo.data', 'Removed undo.data');
+
+unlink 'recur.rc';
+ok (!-r 'recur.rc', 'Removed recur.rc');
+
+exit 0;
+
--- a/src/tests/bug.start.extra.t
+++ b/src/tests/bug.start.extra.t
@@ -0,0 +1,62 @@
+#! /usr/bin/perl
+################################################################################
+## task - a command line task list manager.
+##
+## Copyright 2006 - 2009, Paul Beckingham.
+## All rights reserved.
+##
+## This program is free software; you can redistribute it and/or modify it under
+## the terms of the GNU General Public License as published by the Free Software
+## Foundation; either version 2 of the License, or (at your option) any later
+## version.
+##
+## This program is distributed in the hope that it will be useful, but WITHOUT
+## ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS
+## FOR A PARTICULAR PURPOSE.  See the GNU General Public License for more
+## details.
+##
+## You should have received a copy of the GNU General Public License along with
+## this program; if not, write to the
+##
+##     Free Software Foundation, Inc.,
+##     51 Franklin Street, Fifth Floor,
+##     Boston, MA
+##     02110-1301
+##     USA
+##
+################################################################################
+
+use strict;
+use warnings;
+use Test::More tests => 7;
+
+# Create the rc file.
+if (open my $fh, '>', 'extra.rc')
+{
+  print $fh "data.location=.\n",
+            "confirmation=no\n";
+  close $fh;
+  ok (-r 'extra.rc', 'Created extra.rc');
+}
+
+qx{../task rc:extra.rc add foo};
+my $output = qx{../task rc:extra.rc 1 start pri:L};
+like ($output, qr/The 'start' command does not allow further modification of a task\./, 'no modifications allowed for start');
+$output = qx{../task rc:extra.rc 1 stop pro:bar};
+like ($output, qr/The 'stop' command does not allow further modification of a task\./, 'no modifications allowed for stop');
+
+# Cleanup.
+unlink 'pending.data';
+ok (!-r 'pending.data', 'Removed pending.data');
+
+unlink 'completed.data';
+ok (!-r 'completed.data', 'Removed completed.data');
+
+unlink 'undo.data';
+ok (!-r 'undo.data', 'Removed undo.data');
+
+unlink 'extra.rc';
+ok (!-r 'extra.rc', 'Removed extra.rc');
+
+exit 0;
+
--- a/src/tests/cal.t
+++ b/src/tests/cal.t
@@ -58,7 +58,7 @@ if ( $day <= 9)
 # task cal   and   task cal y
 my $output = qx{../task rc:cal.rc rc._forcecolor:on cal};
 like   ($output, qr/\[36m$day/,      'Current day is highlighted');
-like   ($output, qr/$month.* $year/, 'Current month and year are displayed');
+like   ($output, qr/$month\w+?\s+?$year/, 'Current month and year are displayed');
 qx{../task rc:cal.rc add zero};
 unlike ($output, qr/\[41m\d+/,       'No overdue tasks are present');
 unlike ($output, qr/\[43m\d+/,       'No due tasks are present');
@@ -67,9 +67,9 @@ like   ($output, qr/Su Mo Tu/,     'Week starts on Sunday');
 $output = qx{../task rc:cal.rc rc.weekstart:Monday cal};
 like   ($output, qr/Fr Sa Su/,     'Week starts on Monday'); 
 $output = qx{../task rc:cal.rc cal y};
-like   ($output, qr/$month.* $year/,         'Current month and year are displayed');
-like   ($output, qr/$prevmonth.* $nextyear/, 'Month and year one year ahead are displayed');
-unlike ($output, qr/$month.* $nextyear/,     'Current month and year ahead are not displayed');
+like   ($output, qr/$month\w+?\s+?$year/,         'Current month and year are displayed');
+like   ($output, qr/$prevmonth\w+?\s+?$nextyear/, 'Month and year one year ahead are displayed');
+unlike ($output, qr/$month\w+?\s+?$nextyear/,     'Current month and year ahead are not displayed');

 # task cal due   and   task cal due y
 qx{../task rc:cal.rc add due:20190515 one};
--- a/src/tests/custom.recur_ind.t
+++ b/src/tests/custom.recur_ind.t
@@ -42,7 +42,7 @@ if (open my $fh, '>', 'custom.rc')
  ok (-r 'custom.rc', 'Created custom.rc');
 }

-# Generate the usage screen, and locate the custom report on it.
+# Add a recurring and non-recurring task, look for the indicator.
 qx{../task rc:custom.rc add foo due:tomorrow recur:weekly};
 qx{../task rc:custom.rc add bar};
 my $output = qx{../task rc:custom.rc foo 2>&1};
--- a/src/tests/feature.exit.t
+++ b/src/tests/feature.exit.t
@@ -0,0 +1,62 @@
+#! /usr/bin/perl
+################################################################################
+## task - a command line task list manager.
+##
+## Copyright 2006 - 2009, Paul Beckingham.
+## All rights reserved.
+##
+## This program is free software; you can redistribute it and/or modify it under
+## the terms of the GNU General Public License as published by the Free Software
+## Foundation; either version 2 of the License, or (at your option) any later
+## version.
+##
+## This program is distributed in the hope that it will be useful, but WITHOUT
+## ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS
+## FOR A PARTICULAR PURPOSE.  See the GNU General Public License for more
+## details.
+##
+## You should have received a copy of the GNU General Public License along with
+## this program; if not, write to the
+##
+##     Free Software Foundation, Inc.,
+##     51 Franklin Street, Fifth Floor,
+##     Boston, MA
+##     02110-1301
+##     USA
+##
+################################################################################
+
+use strict;
+use warnings;
+use Test::More tests => 7;
+
+# Create the rc file.
+if (open my $fh, '>', 'exit.rc')
+{
+  print $fh "data.location=.\n",
+            "confirmation=no\n";
+  close $fh;
+  ok (-r 'exit.rc', 'Created exit.rc');
+}
+
+qx{../task rc:exit.rc add foo};
+my $exit_good = system ('../task rc:exit.rc ls foo 2>&1 >>/dev/null');
+is ($exit_good, 0, 'task returns 0 on success');
+my $exit_bad  = system ('../task rc:exit.rc ls bar 2>&1 >>/dev/null');
+isnt ($exit_bad, 0, 'task returns non-zero on failure');
+
+# Cleanup.
+unlink 'pending.data';
+ok (!-r 'pending.data', 'Removed pending.data');
+
+unlink 'completed.data';
+ok (!-r 'completed.data', 'Removed completed.data');
+
+unlink 'undo.data';
+ok (!-r 'undo.data', 'Removed undo.data');
+
+unlink 'exit.rc';
+ok (!-r 'exit.rc', 'Removed exit.rc');
+
+exit 0;
+
--- a/src/tests/filter.t
+++ b/src/tests/filter.t
@@ -111,52 +111,25 @@ unlike ($output, qr/six/,   'g6');
 unlike ($output, qr/seven/, 'g7');

 $output = qx{../task rc:filter.rc list -tag};
-unlike ($output, qr/one/,   'g1');
-like   ($output, qr/two/,   'g2');
-like   ($output, qr/three/, 'g3');
-like   ($output, qr/four/,  'g4');
-unlike ($output, qr/five/,  'g5');
-like   ($output, qr/six/,   'g6');
-like   ($output, qr/seven/, 'g7');
+unlike ($output, qr/one/,   'h1');
+like   ($output, qr/two/,   'h2');
+like   ($output, qr/three/, 'h3');
+like   ($output, qr/four/,  'h4');
+unlike ($output, qr/five/,  'h5');
+like   ($output, qr/six/,   'h6');
+like   ($output, qr/seven/, 'h7');

 $output = qx{../task rc:filter.rc list -missing};
-like   ($output, qr/one/,   'g1');
-like   ($output, qr/two/,   'g2');
-like   ($output, qr/three/, 'g3');
-like   ($output, qr/four/,  'g4');
-like   ($output, qr/five/,  'g5');
-like   ($output, qr/six/,   'g6');
-like   ($output, qr/seven/, 'g7');
+like   ($output, qr/one/,   'i1');
+like   ($output, qr/two/,   'i2');
+like   ($output, qr/three/, 'i3');
+like   ($output, qr/four/,  'i4');
+like   ($output, qr/five/,  'i5');
+like   ($output, qr/six/,   'i6');
+like   ($output, qr/seven/, 'i7');

 $output = qx{../task rc:filter.rc list +tag -tag};
-unlike ($output, qr/one/,   'g1');
-unlike ($output, qr/two/,   'g2');
-unlike ($output, qr/three/, 'g3');
-unlike ($output, qr/four/,  'g4');
-unlike ($output, qr/five/,  'g5');
-unlike ($output, qr/six/,   'g6');
-unlike ($output, qr/seven/, 'g7');
-
-$output = qx{../task rc:filter.rc list project:A priority:H};
-like   ($output, qr/one/,   'h1');
-like   ($output, qr/two/,   'h2');
-unlike ($output, qr/three/, 'h3');
-unlike ($output, qr/four/,  'h4');
-unlike ($output, qr/five/,  'h5');
-unlike ($output, qr/six/,   'h6');
-unlike ($output, qr/seven/, 'h7');
-
-$output = qx{../task rc:filter.rc list project:A priority:};
-unlike ($output, qr/one/,   'i1');
-unlike ($output, qr/two/,   'i2');
-like   ($output, qr/three/, 'i3');
-unlike ($output, qr/four/,  'i4');
-unlike ($output, qr/five/,  'i5');
-unlike ($output, qr/six/,   'i6');
-unlike ($output, qr/seven/, 'i7');
-
-$output = qx{../task rc:filter.rc list project:A foo};
-like   ($output, qr/one/,   'j1');
+unlike ($output, qr/one/,   'j1');
 unlike ($output, qr/two/,   'j2');
 unlike ($output, qr/three/, 'j3');
 unlike ($output, qr/four/,  'j4');
@@ -164,25 +137,25 @@ unlike ($output, qr/five/,  'j5');
 unlike ($output, qr/six/,   'j6');
 unlike ($output, qr/seven/, 'j7');

-$output = qx{../task rc:filter.rc list project:A +tag};
+$output = qx{../task rc:filter.rc list project:A priority:H};
 like   ($output, qr/one/,   'k1');
-unlike ($output, qr/two/,   'k2');
+like   ($output, qr/two/,   'k2');
 unlike ($output, qr/three/, 'k3');
 unlike ($output, qr/four/,  'k4');
 unlike ($output, qr/five/,  'k5');
 unlike ($output, qr/six/,   'k6');
 unlike ($output, qr/seven/, 'k7');

-$output = qx{../task rc:filter.rc list project:A priority:H foo};
-like   ($output, qr/one/,   'l1');
+$output = qx{../task rc:filter.rc list project:A priority:};
+unlike ($output, qr/one/,   'l1');
 unlike ($output, qr/two/,   'l2');
-unlike ($output, qr/three/, 'l3');
+like   ($output, qr/three/, 'l3');
 unlike ($output, qr/four/,  'l4');
 unlike ($output, qr/five/,  'l5');
 unlike ($output, qr/six/,   'l6');
 unlike ($output, qr/seven/, 'l7');

-$output = qx{../task rc:filter.rc list project:A priority:H +tag};
+$output = qx{../task rc:filter.rc list project:A foo};
 like   ($output, qr/one/,   'm1');
 unlike ($output, qr/two/,   'm2');
 unlike ($output, qr/three/, 'm3');
@@ -191,7 +164,7 @@ unlike ($output, qr/five/,  'm5');
 unlike ($output, qr/six/,   'm6');
 unlike ($output, qr/seven/, 'm7');

-$output = qx{../task rc:filter.rc list project:A priority:H foo +tag};
+$output = qx{../task rc:filter.rc list project:A +tag};
 like   ($output, qr/one/,   'n1');
 unlike ($output, qr/two/,   'n2');
 unlike ($output, qr/three/, 'n3');
@@ -200,14 +173,41 @@ unlike ($output, qr/five/,  'n5');
 unlike ($output, qr/six/,   'n6');
 unlike ($output, qr/seven/, 'n7');

+$output = qx{../task rc:filter.rc list project:A priority:H foo};
+like   ($output, qr/one/,   'o1');
+unlike ($output, qr/two/,   'o2');
+unlike ($output, qr/three/, 'o3');
+unlike ($output, qr/four/,  'o4');
+unlike ($output, qr/five/,  'o5');
+unlike ($output, qr/six/,   'o6');
+unlike ($output, qr/seven/, 'o7');
+
+$output = qx{../task rc:filter.rc list project:A priority:H +tag};
+like   ($output, qr/one/,   'p1');
+unlike ($output, qr/two/,   'p2');
+unlike ($output, qr/three/, 'p3');
+unlike ($output, qr/four/,  'p4');
+unlike ($output, qr/five/,  'p5');
+unlike ($output, qr/six/,   'p6');
+unlike ($output, qr/seven/, 'p7');
+
+$output = qx{../task rc:filter.rc list project:A priority:H foo +tag};
+like   ($output, qr/one/,   'q1');
+unlike ($output, qr/two/,   'q2');
+unlike ($output, qr/three/, 'q3');
+unlike ($output, qr/four/,  'q4');
+unlike ($output, qr/five/,  'q5');
+unlike ($output, qr/six/,   'q6');
+unlike ($output, qr/seven/, 'q7');
+
 $output = qx{../task rc:filter.rc list project:A priority:H foo +tag baz};
-unlike ($output, qr/one/,   'n1');
-unlike ($output, qr/two/,   'n2');
-unlike ($output, qr/three/, 'n3');
-unlike ($output, qr/four/,  'n4');
-unlike ($output, qr/five/,  'n5');
-unlike ($output, qr/six/,   'n6');
-unlike ($output, qr/seven/, 'n7');
+unlike ($output, qr/one/,   'r1');
+unlike ($output, qr/two/,   'r2');
+unlike ($output, qr/three/, 'r3');
+unlike ($output, qr/four/,  'r4');
+unlike ($output, qr/five/,  'r5');
+unlike ($output, qr/six/,   'r6');
+unlike ($output, qr/seven/, 'r7');

 # Cleanup.
 unlink 'pending.data';
--- a/src/tests/run_all
+++ b/src/tests/run_all
@@ -1,4 +1,4 @@
-#! /bin/bash
+#! /bin/sh

 date > all.log

@@ -14,7 +14,7 @@ END=`tail -1 all.log`
 OS=`uname`

 case $OS in
-  Darwin)
+  Darwin | FreeBSD)
    STARTEPOCH=`date -j -f "%a %b %d %T %Z %Y" "${START}" "+%s"`
    ENDEPOCH=`date -j -f "%a %b %d %T %Z %Y" "${END}" "+%s"`
    ;;
--- a/src/text.cpp
+++ b/src/text.cpp
@@ -209,7 +209,7 @@ std::string commify (const std::string& data)
  int i;
  for (int i = 0; i < (int) data.length (); ++i)
  {
-    if (::isdigit (data[i]))
+    if (isdigit (data[i]))
      end = i;

    if (data[i] == '.')
@@ -227,11 +227,11 @@ std::string commify (const std::string& data)
    int consecutiveDigits = 0;
    for (; i >= 0; --i)
    {
-      if (::isdigit (data[i]))
+      if (isdigit (data[i]))
      {
        result += data[i];

-        if (++consecutiveDigits == 3 && i && ::isdigit (data[i - 1]))
+        if (++consecutiveDigits == 3 && i && isdigit (data[i - 1]))
        {
          result += ',';
          consecutiveDigits = 0;
@@ -251,11 +251,11 @@ std::string commify (const std::string& data)
    int consecutiveDigits = 0;
    for (; i >= 0; --i)
    {
-      if (::isdigit (data[i]))
+      if (isdigit (data[i]))
      {
        result += data[i];

-        if (++consecutiveDigits == 3 && i && ::isdigit (data[i - 1]))
+        if (++consecutiveDigits == 3 && i && isdigit (data[i - 1]))
        {
          result += ',';
          consecutiveDigits = 0;
@@ -279,8 +279,8 @@ std::string lowerCase (const std::string& input)
 {
  std::string output = input;
  for (int i = 0; i < (int) input.length (); ++i)
-    if (::isupper (input[i]))
-      output[i] = ::tolower (input[i]);
+    if (isupper (input[i]))
+      output[i] = tolower (input[i]);

  return output;
 }
@@ -290,8 +290,8 @@ std::string upperCase (const std::string& input)
 {
  std::string output = input;
  for (int i = 0; i < (int) input.length (); ++i)
-    if (::islower (input[i]))
-      output[i] = ::toupper (input[i]);
+    if (islower (input[i]))
+      output[i] = toupper (input[i]);

  return output;
 }
@@ -302,7 +302,7 @@ std::string ucFirst (const std::string& input)
  std::string output = input;

  if (output.length () > 0)
-    output[0] = ::toupper (output[0]);
+    output[0] = toupper (output[0]);

  return output;
 }
@@ -352,7 +352,7 @@ void guess (
 bool digitsOnly (const std::string& input)
 {
  for (size_t i = 0; i < input.length (); ++i)
-    if (!::isdigit (input[i]))
+    if (!isdigit (input[i]))
      return false;

  return true;
@@ -362,7 +362,7 @@ bool digitsOnly (const std::string& input)
 bool noSpaces (const std::string& input)
 {
  for (size_t i = 0; i < input.length (); ++i)
-    if (::isspace (input[i]))
+    if (isspace (input[i]))
      return false;

  return true;
--- a/src/util.cpp
+++ b/src/util.cpp
@@ -64,7 +64,7 @@ bool confirm (const std::string& question)
              << " ";

    std::getline (std::cin, answer);
-    answer = lowerCase (trim (answer));
+    answer = std::cin.eof() ? "no" : lowerCase (trim (answer));
  }
  while (answer != "y"   && // TODO i18n
         answer != "ye"  && // TODO i18n
Author	SHA1	Message	Date
Federico Hernandez	12c4983936	Release date for 1.8.4	2009-11-17 12:00:36 +01:00
Paul Beckingham	39d9f235de	HACK - case-insensitive file system problem again.	2009-11-16 23:42:32 -05:00
Paul Beckingham	7aa0c3698a	HACK - case-insensitive file system problem again.	2009-11-16 23:42:00 -05:00
Paul Beckingham	bc40ab63b3	Bug Fix - #312 Changing one task changes another - Added a warning when modifying recurring tasks, that all instances of that task may be modified. When task confirms a bulk edit the recurrence is again indicated (thanks to Cory Donnelly).	2009-11-16 23:24:47 -05:00
Paul Beckingham	6e673d2834	Bug Fix - #313 Edit command fails when data.location includes spaces - Applied patch from Cory Donnelly.	2009-11-16 22:10:47 -05:00
Federico Hernandez	30c6dd0047	Added Joe to AUTHORS file	2009-11-13 23:32:40 +01:00
Paul Beckingham	64bc2a165a	Bug fix - hang on cygwin when task updated. - Fixed bug that caused a hang on cygwin, when a task with multiple annotations was edited (thanks to Joe Pulliam).	2009-11-09 22:35:51 -05:00
Paul Beckingham	5b96dbbce8	Bug Fix - wait date editing - The "wait" date was not being properly formatted, as are all the other dates, in the "edit" command. The result is that an epoch integer date was rendered, instead of something readable and in the preferred format.	2009-11-09 18:19:04 -05:00
Federico Hernandez	77dd930574	Fixed bug in regexp matching of whitespace between month and year.	2009-11-05 21:36:46 +01:00
Federico Hernandez	f6842941f3	Bumped version number to 1.8.4	2009-10-22 22:40:18 +02:00
Federico Hernandez	e2e0851a69	Merge branch '1.8.3'	2009-10-22 22:27:21 +02:00
Federico Hernandez	1299fe468b	Added sha1 for 1.8.3 to Changelog	2009-10-22 22:09:53 +02:00
Federico Hernandez	de50b2902c	Merge branch '1.8.3'	2009-10-22 22:02:32 +02:00
Federico Hernandez	bcdcbeeea0	Preparing release - Updating dist documentation - Release date in man pages	2009-10-21 22:53:26 +02:00
Pietro Cerutti	469cafa053	Make run_all more portable. - Changed shebang for portability - Fix date command for FreeBSD Signed-off-by: Federico Hernandez <ultrafredde@gmail.com>	2009-10-21 22:20:32 +02:00
Pietro Cerutti	fdb359c180	Fix confirm() to handle EOF Signed-off-by: Federico Hernandez <ultrafredde@gmail.com>	2009-10-21 22:19:37 +02:00
Paul Beckingham	b3d40b2554	Portability - Haiku R1/alpha1 - Added necessary include files and edits in order to build task on Haiku R1/alpha1.	2009-09-27 17:02:20 -04:00
Federico Hernandez	fba076a0d0	Bumped version number to 1.8.3	2009-09-07 16:13:37 +02:00
Federico Hernandez	09d7940dd3	Revert "Bumping version number to 1.8.3" This reverts commit `00031dc1ab`.	2009-09-07 16:04:24 +02:00
Federico Hernandez	00031dc1ab	Bumping version number to 1.8.3	2009-09-07 15:52:51 +02:00
Paul Beckingham	3ef844de5f	Packaging - Adding updated package files for OSX, from the 1.8.2 release.	2009-09-07 09:51:20 -04:00
Federico Hernandez	bb45d91ddb	Added sha1 of 1.8.2 to ChangLog	2009-09-07 15:41:51 +02:00
Federico Hernandez	f243f0ed44	Release date in man pages	2009-09-07 10:18:44 +02:00
Federico Hernandez	b305cc0a60	Release date in ChangeLog	2009-09-07 09:57:09 +02:00
Paul Beckingham	53609b2837	Bug Fix - #288 - Fixed bug which failed to propagate rc overrides from the command line to the default command (thanks to Zach Frazier).	2009-09-06 16:27:28 -04:00
Federico Hernandez	bdaa0f89d9	Platform update	2009-09-02 12:50:43 +02:00
Federico Hernandez	2fd8d8aa83	Bug Fix #291 - Fixed bug #291 which generated a false warning about an unrecognized variable when enabling default.projects in .taskrc (thanks to Thomas@BIC)	2009-09-02 10:43:04 +02:00
Federico Hernandez	e69fb81b2b	Documentation update in Changelog for #289	2009-09-01 21:12:49 +02:00
Federico Hernandez	d2aa0f31b0	Bug Fix - #289 - Fixed bug #289 which imported task from todo.sh without valid uuids (thanks to Ben Jackson).	2009-09-01 21:08:13 +02:00
Paul Beckingham	05f67db429	Bug Fix - #287 - Fixed bug #287 which caused color control codes to be written to the shadow file (thanks to Richard Querin).	2009-08-29 13:55:59 -04:00
Paul Beckingham	62be3f8acb	Feature #282 - Applied large patch to make task return meaningful exit codes. - Added unit tests to prove this. - Thanks to Pietro Cerutti.	2009-08-29 09:14:26 -04:00
Federico Hernandez	cc5c99c0a1	Pushing version number to 1.8.2	2009-08-21 01:09:41 +02:00
Federico Hernandez	35792e7874	Added release date for 1.8.1 to ChangeLog	2009-08-20 22:48:22 +02:00
Federico Hernandez	083407f789	Pushing package build scripts for fedora, ubuntu and cygwin to 1.8.1	2009-08-19 23:50:47 +02:00
Federico Hernandez	e1146cdd20	Changed epoch to be 12/22 for UTC	2009-08-18 13:46:19 +02:00
Paul Beckingham	f04205be4c	Documentation Update - Added a load of missing ChangeLog entries, by running the timesheet report and comparing.	2009-08-16 23:10:21 -04:00
Paul Beckingham	07da2396fc	Bug Fix - #251 - Fixed bug #251 that caused the .hasnt attribute modifier to fail when annotations were present. Thanks to John Florian.	2009-08-16 22:38:33 -04:00
Paul Beckingham	ec7f7cc939	Bug Fix - #260 - Fixed bug whereby the start, stop and delete commands were not complaining when filter arguments were specified, even though they were ignored. Thanks to Charles T. Yun.	2009-08-16 18:39:35 -04:00
Paul Beckingham	1b28d8714b	Bug Fix - #249 - Fixed bug that caused two annotations with the same date to be parsed identically, and therefore were not considered unique annotations after the "edit" command.	2009-08-16 17:22:17 -04:00
Paul Beckingham	79f59f12ae	Bug Fix - #252 - Fixed bug that prevented using end.after: and end.before: together to effect a range. - Required differentiation between positive and negative attribute modifiers, and special handling.	2009-08-16 13:01:28 -04:00
Paul Beckingham	f595bc4731	Bug Fix - #259 - Fixed broken build on Snow Leopard - missing include.	2009-08-15 14:20:20 -04:00
Federico Hernandez	972efc58f4	Pushing man pages to 1.8.1	2009-08-12 14:43:05 +02:00
Paul Beckingham	dc9ba6d6b8	Bug Fix - #250 - Fixed bug #250 whereby rc.dateformat was not observed when parsing the creation date of an annotation (thanks to Federico Hernandez).	2009-08-07 16:53:25 -06:00
Paul Beckingham	a7196ca181	Bug Fix - #257 - Fixed bug #257 where an extant ~/.taskrc file prevented the override and automatic creation of an alternate rc file (thanks to Zach Frazier).	2009-08-07 16:11:09 -06:00
Paul Beckingham	1355571876	Bug Fix - #252 - Fixed bug that prevented the chaining of two attributes with different modifiers to effect a date range, such as: task ls due.after:8/1/2009 due.before:8/31/2009 Thanks to John Florian.	2009-08-05 10:15:33 -06:00
Paul Beckingham	d3fcd40279	Bug Fix - Bug #248 - Fixed bug where both single and double quotes are stored as ampersand-quote-semi-colon, leading to the gradual conversion of one to the other. Fix is backward compatible. Thanks to John Florian.	2009-08-05 07:24:00 -06:00
Paul Beckingham	c4dcdbff0f	Unit Tests - range - Added unit tests to check the behavior when a data range is created with "task list due.before:... due.after:...".	2009-08-04 17:24:06 -06:00
Paul Beckingham	d39e45841d	Bug Fix - Bug #241 - Fixed bug #241 that caused redirected output to retain color control codes for colored header and footnotes (thanks to John Florian).	2009-08-04 16:59:52 -06:00
Paul Beckingham	b28575625e	Bug Fix - Bug #243 - Fixed bug that caused due dates and recurrence periods to be quoted on export, which then caused confusion on a subsequent import (thanks to John Florian).	2009-08-04 16:47:49 -06:00
Paul Beckingham	fc8f8957c4	Bug Fix - recurring task with no due date - A recurring task may not be added without a due date. Previously, this restriction only applied to modified tasks.	2009-08-04 16:26:23 -06:00
Paul Beckingham	0499bfff49	Bug Fix - Bug #242 - Fixed bug that caused the reported ID after an add to be incorrect.	2009-08-04 16:08:56 -06:00
Paul Beckingham	af49ccf508	Bug Fix - wrong .taskrc file in message - When a .taskrc file is overridden (with rc:...) task still refers to the default file name in output.	2009-08-04 16:01:20 -06:00
Paul Beckingham	7b0edfdeba	Unit Test - recurring task with no due date - Should generate error, but doesn't.	2009-08-03 07:38:22 -06:00
Federico Hernandez	e5d56e3075	Enhancement: * Remove trailing whitespace from completions. * Complete rc.<variable> with ":'.	2009-08-01 01:24:46 +02:00
John Florian	ede746ba16	Fixed bug #241 Completion behavior has changed from bash-3 to bash-4. Even the colon character appears in COMP_WORDBREAKS of both versions, only bash-4 seems to honor it. This appears to be a bug in bash-3. Changes made here work around the problem so that completion of project names works for either version of bash.	2009-07-30 23:19:10 -04:00
Paul Beckingham	0bad0277fe	Documentation Update - Added bug fix info to ChangeLog. This is now ready to release, if necessary.	2009-07-26 16:31:40 -04:00
Paul Beckingham	8488cbd1a6	Bug Fix - #231 time_t/int collision on 32-bit systems - Fixed bug #231 by removing the operator int () cast from the Duration class. On a 32-bit system, "time_t" may be defined as "int", in which case the int cast and time_t cast are essentially duplicates. Thanks to Pietro Cerutti, who also provided a patch.	2009-07-26 16:11:14 -04:00
Paul Beckingham	6f9356da64	Updated version to 1.8.1 - Changed program version number. - Changed NEWS file to refer only to 1.8, not 1.8.1. This means we now only need to update the NEWS file on major releases, not patches. This is the right thing to do now that we have only "whats new" in the NEWS file.	2009-07-26 16:06:14 -04:00
Paul Beckingham	90d53245c3	Merge branch '1.8.0' Conflicts: .gitignore AUTHORS ChangeLog DEVELOPERS Makefile.am NEWS README configure.ac doc/man/task.1 doc/man/taskrc.5 src/T.cpp src/T.h src/TDB.cpp src/TDB.h src/command.cpp src/edit.cpp src/import.cpp src/parse.cpp src/report.cpp src/rules.cpp src/task.cpp src/task.h task_completion.sh	2009-07-21 19:15:54 -04:00
Paul Beckingham	1422a15cbc	Merge branch '1.7.1' Conflicts: doc/man5/taskrc.5	2009-06-09 19:49:36 -04:00
Paul Beckingham	9c15457d7f	Merge branch '1.7.1'	2009-06-09 19:35:24 -04:00
Federico Hernandez	20d8124a1d	Fixed documentation for 1.7.1	2009-06-09 17:30:09 +02:00
Federico Hernandez	db6de54ccc	* Updated project homepage reference and license text	2009-06-09 09:32:23 +02:00
Federico Hernandez	eb864820cf	* Removed hard-coded version number	2009-06-09 09:31:44 +02:00
Federico Hernandez	0291e4c72e	Changed some more references from beckingham.net to taskwarrior.org	2009-06-09 08:56:59 +02:00
Paul Beckingham	6e27feb8a3	Code Cleanup - Removed Makefile.in from repository. This was done in 1.8.0, but wasn't in 1.7.0. It's time. - Added Makefile.in to the .gitignore files.	2009-06-08 23:24:09 -04:00
Paul Beckingham	1dcba4619e	Documentation Update - Bumped version to 1.7.1. - Changed references to email addresses, URLs.	2009-06-08 23:14:38 -04:00
Paul Beckingham	7538b43c68	Bug Fix - Build failure on OpenBSD - Fixed build failure on OpenBSD (thanks to Mike Adonay).	2009-06-08 23:03:30 -04:00