[PATCH] Fix double use of mail directory in parsemail-batch.sh

Stephen Finucane stephen at that.guru
Wed Feb 8 20:25:04 AEDT 2017


On Wed, 2017-02-08 at 15:16 +1100, Daniel Axtens wrote:
> 2d142b2c0a27 ("bin: Run scripts through shellcheck") changes an
> ls to a find. find output includes the directory name, unlike ls.
> This breaks the script, because it prepends the mail directory
> later on.
> 
> Remove the prepending in the script - leave it exclusively to find.
> 
> Fixes: 2d142b2c0a27 ("bin: Run scripts through shellcheck")
> Cc: stephen at that.guru
> Signed-off-by: Daniel Axtens <dja at axtens.net>

Oops, good spot. We could do with tests for these scripts but that's a
problem for another day.

Reviewed-by: Stephen Finucane <stephen at that.guru>

...and applied.


More information about the Patchwork mailing list