[Buildroot] [PATCH 1/1] Added new package ncdu

[Buildroot] [PATCH 1/1] Added new package ncdu

[Bogdan Radulescu]

Signed-off-by: Bogdan Radulescu <bogdan@nimblex.net>
---
?package/Config.in????? |? 1 +
?package/ncdu/Config.in |? 7 +++++++
?package/ncdu/ncdu.mk?? | 13 +++++++++++++
?3 files changed, 21 insertions(+)
?create mode 100644 package/ncdu/Config.in
?create mode 100644 package/ncdu/ncdu.mk

diff --git a/package/Config.in b/package/Config.in
index 171db8f..2d12cdc 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -732,6 +732,7 @@ if BR2_PACKAGE_BUSYBOX_SHOW_OTHERS
?source "package/module-init-tools/Config.in"
?endif
?source "package/monit/Config.in"
+source "package/ncdu/Config.in"
?if BR2_PACKAGE_BUSYBOX_SHOW_OTHERS
?source "package/procps/Config.in"
?source "package/psmisc/Config.in"
diff --git a/package/ncdu/Config.in b/package/ncdu/Config.in
new file mode 100644
index 0000000..d8b4db4
--- /dev/null
+++ b/package/ncdu/Config.in
@@ -0,0 +1,7 @@
+config BR2_PACKAGE_NCDU
+?????? bool "ncdu"
+?????? select BR2_PACKAGE_NCURSES
+?????? help
+???????? ncdu is a disk usage analyzer with an ncurses interface
+
+???????? http://dev.yorhel.nl/ncdu
diff --git a/package/ncdu/ncdu.mk b/package/ncdu/ncdu.mk
new file mode 100644
index 0000000..26f3b5e
--- /dev/null
+++ b/package/ncdu/ncdu.mk
@@ -0,0 +1,13 @@
+#############################################################
+#
+# ncdu
+#
+#############################################################
+NCDU_VERSION=1.9
+NCDU_SITE=http://dev.yorhel.nl/download/
+NCDU_INSTALL_STAGING_OPT = instroot=$(STAGING_DIR) install
+NCDU_INSTALL_TARGET_OPT = instroot=$(TARGET_DIR) install
+
+NCDU_DEPENDENCIES = ncurses
+
+$(eval $(autotools-package))
-- 
1.7.12.1
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ncdu.diff
Type: application/octet-stream
Size: 1812 bytes
Desc: not available
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20121217/42140369/attachment.obj>

[Buildroot] [PATCH 1/1] Added new package ncdu

[Arnout Vandecappelle]

  Dear Bogdan,

  Thanks for your patch.

  We prefer generally prefer patches to be sent in-line with the correct 
spacing - using git send-email is the easiest way to send a patch.

  Some additional comments below.

On 17/12/12 21:04, Bogdan Radulescu wrote:
>
> Signed-off-by: Bogdan Radulescu<bogdan@nimblex.net>
[snip]
> diff --git a/package/ncdu/Config.in b/package/ncdu/Config.in
> new file mode 100644
> index 0000000..d8b4db4
> --- /dev/null
> +++ b/package/ncdu/Config.in
> @@ -0,0 +1,7 @@
> +config BR2_PACKAGE_NCDU
> +       bool "ncdu"

  Indentation in Config.in should be a single TAB.

> +       select BR2_PACKAGE_NCURSES
> +       help
> +         ncdu is a disk usage analyzer with an ncurses interface

  ... except for help which should be one TAB + 2 spaces.

> +
> +         http://dev.yorhel.nl/ncdu
> diff --git a/package/ncdu/ncdu.mk b/package/ncdu/ncdu.mk
> new file mode 100644
> index 0000000..26f3b5e
> --- /dev/null
> +++ b/package/ncdu/ncdu.mk
> @@ -0,0 +1,13 @@
> +#############################################################
> +#
> +# ncdu
> +#
> +#############################################################
> +NCDU_VERSION=1.9

  We prefer spaces around the =

NCDU_VERSION = 1.9

> +NCDU_SITE=http://dev.yorhel.nl/download/
> +NCDU_INSTALL_STAGING_OPT = instroot=$(STAGING_DIR) install

  Since you don't install to staging, this is redundant.

> +NCDU_INSTALL_TARGET_OPT = instroot=$(TARGET_DIR) install

  I don't understand where this comes from... It looks like it should be 
DESTDIR=$(TARGET_DIR), which is the default, so this line is redundant as 
well.

  It would be nice to also add the licensing:

NCDU_LICENSE = MIT
NCDU_LICENSE_FILES = COPYING

(test with 'make legal-info')

  Regards,
  Arnout

> +
> +NCDU_DEPENDENCIES = ncurses
> +
> +$(eval $(autotools-package))

-- 
Arnout Vandecappelle                               arnout at mind be
Senior Embedded Software Architect                 +32-16-286540
Essensium/Mind                                     http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium                BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F

[Buildroot] [PATCH 1/1] Added new package ncdu

[Peter Korsgaard]

>>>>> "Arnout" == Arnout Vandecappelle <arnout@mind.be> writes:

Hi Bogdan,

 Arnout>  We prefer generally prefer patches to be sent in-line with the
 Arnout> correct spacing - using git send-email is the easiest way to send a
 Arnout> patch.

 Arnout>  Some additional comments below.

Could you please send an updated patch taking Arnout's comment into
consideration?

-- 
Bye, Peter Korsgaard