diff mbox

package/sysdig: New package

Message ID 1426499014-15851-1-git-send-email-angelo.compagnucci@gmail.com
State Superseded
Headers show

Commit Message

Angelo Compagnucci March 16, 2015, 9:43 a.m. UTC 
Sysdig is open source, system-level exploration:
capture system state and activity from a running Linux
instance, then save, filter and analyze.

Signed-off-by: Angelo Compagnucci <angelo.compagnucci@gmail.com>
---
 package/Config.in        |  1 +
 package/sysdig/Config.in | 14 ++++++++++++++
 package/sysdig/sysdig.mk | 22 ++++++++++++++++++++++
 3 files changed, 37 insertions(+)
 create mode 100644 package/sysdig/Config.in
 create mode 100644 package/sysdig/sysdig.mk

Comments

Baruch Siach March 17, 2015, 10:13 a.m. UTC | #1 
Hi Angelo,

On Mon, Mar 16, 2015 at 10:43:34AM +0100, Angelo Compagnucci wrote:
> diff --git a/package/sysdig/Config.in b/package/sysdig/Config.in
> new file mode 100644
> index 0000000..ff873a8
> --- /dev/null
> +++ b/package/sysdig/Config.in
> @@ -0,0 +1,14 @@
> +config BR2_PACKAGE_SYSDIG
> +	bool "sysdig"
> +	depends on BR2_PACKAGE_ZLIB
> +	depends on BR2_PACKAGE_LUAJIT
> +	depends on BR2_PACKAGE_LIBJSON

We generally 'select' packages that are mandatory dependencies, and copy their 
dependencies here. See 
http://nightly.buildroot.org/manual.html#depends-on-vs-select.

> +	depends on BR2_LINUX_KERNEL

This is OK, though. But please also add a comment as described in 
http://nightly.buildroot.org/manual.html#_dependencies_on_a_linux_kernel_built_by_buildroot.

baruch
diff mbox

Patch

diff --git a/package/Config.in b/package/Config.in
index e221deb..019d40b 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -94,6 +94,7 @@  endif
 	source "package/strace/Config.in"
 	source "package/stress/Config.in"
 	source "package/sysprof/Config.in"
+	source "package/sysdig/Config.in"
 	source "package/tinymembench/Config.in"
 	source "package/trace-cmd/Config.in"
 	source "package/trinity/Config.in"
diff --git a/package/sysdig/Config.in b/package/sysdig/Config.in
new file mode 100644
index 0000000..ff873a8
--- /dev/null
+++ b/package/sysdig/Config.in
@@ -0,0 +1,14 @@ 
+config BR2_PACKAGE_SYSDIG
+	bool "sysdig"
+	depends on BR2_PACKAGE_ZLIB
+	depends on BR2_PACKAGE_LUAJIT
+	depends on BR2_PACKAGE_LIBJSON
+	depends on BR2_LINUX_KERNEL
+	help
+	  Sysdig is open source, system-level exploration:
+	  capture system state and activity from a running Linux instance,
+	  then save, filter and analyze.
+	  Think of it as strace + tcpdump + lsof + awesome sauce.
+	  With a little Lua cherry on top.
+
+	  http://sysdig.org
diff --git a/package/sysdig/sysdig.mk b/package/sysdig/sysdig.mk
new file mode 100644
index 0000000..62b9344
--- /dev/null
+++ b/package/sysdig/sysdig.mk
@@ -0,0 +1,22 @@ 
+#############################################################
+#
+# sysdig
+#
+#############################################################
+
+SYSDIG_VERSION = 0.1.98
+SYSDIG_SITE = $(call github,draios,sysdig,$(SYSDIG_VERSION))
+SYSDIG_LICENSE = GPLv2
+SYSDIG_LICENSE_FILES = COPYING
+SYSDIG_CONF_OPTS = -DUSE_BUNDLED_LUAJIT=OFF -DUSE_BUNDLED_ZLIB=OFF
+SYSDIG_DEPENDENCIES += zlib luajit libjson linux
+SYSDIG_SUPPORTS_IN_SOURCE_BUILD = NO
+
+define SYSDIG_INSTALL_DRIVER
+	$(MAKE) $(LINUX_MAKE_FLAGS) -C $(LINUX_DIR) M=$(@D)/driver modules
+	$(MAKE) $(LINUX_MAKE_FLAGS) -C $(LINUX_DIR) M=$(@D)/driver modules_install
+endef
+
+SYSDIG_POST_INSTALL_TARGET_HOOKS += SYSDIG_INSTALL_DRIVER
+
+$(eval $(cmake-package))