diff mbox

[v2,1/1] taskd: new package

Message ID 1457577520-11211-1-git-send-email-mathstuf@gmail.com
State Accepted
Headers show

Commit Message

Ben Boeckel March 10, 2016, 2:38 a.m. UTC
Taskwarrior is a task management suite of tools. Taskd is the
synchronization server for it.

Signed-off-by: Ben Boeckel <mathstuf@gmail.com>
---
 package/Config.in       |  1 +
 package/taskd/Config.in | 11 +++++++++++
 package/taskd/taskd.mk  | 14 ++++++++++++++
 3 files changed, 26 insertions(+)
 create mode 100644 package/taskd/Config.in
 create mode 100644 package/taskd/taskd.mk

Comments

Arnout Vandecappelle March 10, 2016, 10:31 p.m. UTC | #1
On 03/10/16 03:38, Ben Boeckel wrote:
> Taskwarrior is a task management suite of tools. Taskd is the
> synchronization server for it.
>
> Signed-off-by: Ben Boeckel <mathstuf@gmail.com>
> ---
>   package/Config.in       |  1 +
>   package/taskd/Config.in | 11 +++++++++++
>   package/taskd/taskd.mk  | 14 ++++++++++++++
>   3 files changed, 26 insertions(+)
>   create mode 100644 package/taskd/Config.in
>   create mode 100644 package/taskd/taskd.mk
>
> diff --git a/package/Config.in b/package/Config.in
> index d916154..35a24ba 100644
> --- a/package/Config.in
> +++ b/package/Config.in
> @@ -1489,6 +1489,7 @@ endif
>   	source "package/sshpass/Config.in"
>   	source "package/strongswan/Config.in"
>   	source "package/stunnel/Config.in"
> +	source "package/taskd/Config.in"

  I don't think networking tools is the appropriate location. Miscellaneous maybe.

>   	source "package/tcpdump/Config.in"
>   	source "package/tcping/Config.in"
>   	source "package/tcpreplay/Config.in"
> diff --git a/package/taskd/Config.in b/package/taskd/Config.in
> new file mode 100644
> index 0000000..5b357f5
> --- /dev/null
> +++ b/package/taskd/Config.in
> @@ -0,0 +1,11 @@
> +config BR2_PACKAGE_TASKD
> +	bool "taskd"
> +	select BR2_PACKAGE_GNUTLS
> +	select BR2_USE_MMU
> +	select BR2_USE_WCHAR

  You cannot select MMU and WCHAR, you need to depend on it. See 
https://buildroot.org/downloads/manual/manual.html#depends-on-vs-select
The depends lines should come immediately after bool "taskd". There should also 
be a comment at the end of the file that tells the user that taskd is only 
available with wchar.

  In addition to these two, it also needs C++11. So definitely 
BR2_INSTALL_LIBSTDCPP, and probably also some minimal gcc version. Try with the 
Arago ARMv7 2011.09 if 4.5 works; if it does, add 
BR2_TOOLCHAIN_GCC_AT_LEAST_4_5, and so on until you find the minimal version.


> +	select BR2_PACKAGE_UTIL_LINUX
> +	select BR2_PACKAGE_UTIL_LINUX_LIBUUID
> +	help
> +	  Task management synchronization daemon.
> +
> +	  http://taskwarrior.org/
> diff --git a/package/taskd/taskd.mk b/package/taskd/taskd.mk
> new file mode 100644
> index 0000000..0bdf81d
> --- /dev/null
> +++ b/package/taskd/taskd.mk
> @@ -0,0 +1,14 @@
> +################################################################################
> +#
> +# taskd
> +#
> +################################################################################
> +
> +TASKD_VERSION = 1.1.0
> +TASKD_SOURCE = taskd-$(TASKD_VERSION).tar.gz

  This is the default, not needed.

> +TASKD_SITE = http://taskwarrior.org/download
> +TASKD_LICENSE = MIT
> +TASKD_LICENSE_FILES = COPYING LICENSE

  AFAICS both files are identical, so specifying only one of them (COPYING) is 
sufficient.

  Regards,
  Arnout


> +TASKD_DEPENDENCIES = gnutls util-linux
> +
> +$(eval $(cmake-package))
>
Thomas Petazzoni March 10, 2016, 10:31 p.m. UTC | #2
Ben,

On Wed,  9 Mar 2016 21:38:40 -0500, Ben Boeckel wrote:
> Taskwarrior is a task management suite of tools. Taskd is the
> synchronization server for it.
> 
> Signed-off-by: Ben Boeckel <mathstuf@gmail.com>

Thanks for this updated version! I've applied, but there were still a
few things to fix. Here are the changes I did:

    [Thomas:
     - use 'depends on' for BR2_USE_MMU, and add a comment indicating that
       the dependency is due to the use of fork()
     - use 'depends on' for BR2_USE_WCHAR, and add a comment indicating
       that the dependency is due to gnutls and util-linux.
     - add a hash file.]

Also, it would be a good idea to send a new patch that adds a simple
init script to start this server at boot time. This way, taskd can work
"out of the box".

Thanks!

Thomas
diff mbox

Patch

diff --git a/package/Config.in b/package/Config.in
index d916154..35a24ba 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -1489,6 +1489,7 @@  endif
 	source "package/sshpass/Config.in"
 	source "package/strongswan/Config.in"
 	source "package/stunnel/Config.in"
+	source "package/taskd/Config.in"
 	source "package/tcpdump/Config.in"
 	source "package/tcping/Config.in"
 	source "package/tcpreplay/Config.in"
diff --git a/package/taskd/Config.in b/package/taskd/Config.in
new file mode 100644
index 0000000..5b357f5
--- /dev/null
+++ b/package/taskd/Config.in
@@ -0,0 +1,11 @@ 
+config BR2_PACKAGE_TASKD
+	bool "taskd"
+	select BR2_PACKAGE_GNUTLS
+	select BR2_USE_MMU
+	select BR2_USE_WCHAR
+	select BR2_PACKAGE_UTIL_LINUX
+	select BR2_PACKAGE_UTIL_LINUX_LIBUUID
+	help
+	  Task management synchronization daemon.
+
+	  http://taskwarrior.org/
diff --git a/package/taskd/taskd.mk b/package/taskd/taskd.mk
new file mode 100644
index 0000000..0bdf81d
--- /dev/null
+++ b/package/taskd/taskd.mk
@@ -0,0 +1,14 @@ 
+################################################################################
+#
+# taskd
+#
+################################################################################
+
+TASKD_VERSION = 1.1.0
+TASKD_SOURCE = taskd-$(TASKD_VERSION).tar.gz
+TASKD_SITE = http://taskwarrior.org/download
+TASKD_LICENSE = MIT
+TASKD_LICENSE_FILES = COPYING LICENSE
+TASKD_DEPENDENCIES = gnutls util-linux
+
+$(eval $(cmake-package))