diff options
| author | spwhitton <spwhitton@web> | 2015-12-28 16:58:41 +0000 |
|---|---|---|
| committer | admin <admin@branchable.com> | 2015-12-28 16:58:41 +0000 |
| commit | 0b0e8aaf827b0de5232b648142873e29504f2076 (patch) | |
| tree | 4429b768236f63e1958e1d0c5ac78ba13a8bcbad | |
| parent | 65b598f6c8083725e43b2cb5c2219b5f52154bdc (diff) | |
pull request
| -rw-r--r-- | doc/todo/silence_xargs_when_hothasktags_not_installed.mdwn | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/doc/todo/silence_xargs_when_hothasktags_not_installed.mdwn b/doc/todo/silence_xargs_when_hothasktags_not_installed.mdwn new file mode 100644 index 00000000..ff34006a --- /dev/null +++ b/doc/todo/silence_xargs_when_hothasktags_not_installed.mdwn @@ -0,0 +1,7 @@ +Please consider merging branch `hothasktags` of my repo `https://git.spwhitton.name/propellor` + +Moves the `2>/dev/null` redirection in the Makefile rule for `tags` in order to silence xargs if hothasktags is not installed (avoids `xargs: hothasktags: No such file or directory`). + +I assume that your originally intention was not to cause an error if hothasktags is not installed, since you have `|| true` at the end of the line. However, someone trying to build propellor's Debian package saw the error output from xargs and thought it meant hothasktags should be a build-dep. This patch prevents someone thinking that. + +-- [[spwhitton|https://spwhitton.name]] |