* [Buildroot] [PATCH v2] package/sysdig: New package
@ 2015-03-17 10:56 Angelo Compagnucci
2015-03-17 11:23 ` Baruch Siach
0 siblings, 1 reply; 6+ messages in thread
From: Angelo Compagnucci @ 2015-03-17 10:56 UTC (permalink / raw)
To: buildroot
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>
---
Changes v1 -> v2:
- Changed "depends on" with "select" for dependencies (suggested by Baruch)
- Added comment "sysdig needs a Linux kernel to be built" (suggested by Baruch)
- Upgreded to recently released 0.1.99
package/Config.in | 1 +
package/sysdig/Config.in | 17 +++++++++++++++++
package/sysdig/sysdig.mk | 22 ++++++++++++++++++++++
3 files changed, 40 insertions(+)
create mode 100644 package/sysdig/Config.in
create mode 100644 package/sysdig/sysdig.mk
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..94b088f
--- /dev/null
+++ b/package/sysdig/Config.in
@@ -0,0 +1,17 @@
+comment "sysdig needs a Linux kernel to be built"
+ depends on !BR2_LINUX_KERNEL
+
+config BR2_PACKAGE_SYSDIG
+ bool "sysdig"
+ select BR2_PACKAGE_ZLIB
+ select BR2_PACKAGE_LUAJIT
+ select 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.99
+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))
--
1.9.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [Buildroot] [PATCH v2] package/sysdig: New package
2015-03-17 10:56 [Buildroot] [PATCH v2] package/sysdig: New package Angelo Compagnucci
@ 2015-03-17 11:23 ` Baruch Siach
2015-03-17 11:31 ` Angelo Compagnucci
0 siblings, 1 reply; 6+ messages in thread
From: Baruch Siach @ 2015-03-17 11:23 UTC (permalink / raw)
To: buildroot
Hi Angelo,
On Tue, Mar 17, 2015 at 11:56:48AM +0100, Angelo Compagnucci wrote:
> diff --git a/package/sysdig/Config.in b/package/sysdig/Config.in
> new file mode 100644
> index 0000000..94b088f
> --- /dev/null
> +++ b/package/sysdig/Config.in
> @@ -0,0 +1,17 @@
> +comment "sysdig needs a Linux kernel to be built"
> + depends on !BR2_LINUX_KERNEL
> +
> +config BR2_PACKAGE_SYSDIG
> + bool "sysdig"
> + select BR2_PACKAGE_ZLIB
> + select BR2_PACKAGE_LUAJIT
> + select BR2_PACKAGE_LIBJSON
This packages also needs to depend on all the dependencies of these packages.
Please read the whole section under
http://nightly.buildroot.org/manual.html#depends-on-vs-select.
> + 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
A dependencies comment is still missing here
(http://nightly.buildroot.org/manual.html#_dependencies_on_a_linux_kernel_built_by_buildroot).
baruch
--
http://baruch.siach.name/blog/ ~. .~ Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
- baruch at tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il -
^ permalink raw reply [flat|nested] 6+ messages in thread* [Buildroot] [PATCH v2] package/sysdig: New package
2015-03-17 11:23 ` Baruch Siach
@ 2015-03-17 11:31 ` Angelo Compagnucci
2015-03-17 11:48 ` Baruch Siach
0 siblings, 1 reply; 6+ messages in thread
From: Angelo Compagnucci @ 2015-03-17 11:31 UTC (permalink / raw)
To: buildroot
Hi Baruch,
2015-03-17 12:23 GMT+01:00 Baruch Siach <baruch@tkos.co.il>:
> Hi Angelo,
>
> On Tue, Mar 17, 2015 at 11:56:48AM +0100, Angelo Compagnucci wrote:
>> diff --git a/package/sysdig/Config.in b/package/sysdig/Config.in
>> new file mode 100644
>> index 0000000..94b088f
>> --- /dev/null
>> +++ b/package/sysdig/Config.in
>> @@ -0,0 +1,17 @@
>> +comment "sysdig needs a Linux kernel to be built"
>> + depends on !BR2_LINUX_KERNEL
>> +
>> +config BR2_PACKAGE_SYSDIG
>> + bool "sysdig"
>> + select BR2_PACKAGE_ZLIB
>> + select BR2_PACKAGE_LUAJIT
>> + select BR2_PACKAGE_LIBJSON
>
> This packages also needs to depend on all the dependencies of these packages.
> Please read the whole section under
> http://nightly.buildroot.org/manual.html#depends-on-vs-select.
Now I understand because Thomas Petazzoni suggests to use depends on
insted of select!
>
>> + 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
>
> A dependencies comment is still missing here
> (http://nightly.buildroot.org/manual.html#_dependencies_on_a_linux_kernel_built_by_buildroot).
The comment is there:
@@ -0,0 +1,17 @@
+comment "sysdig needs a Linux kernel to be built"
+ depends on !BR2_LINUX_KERNEL
Sincerely, Angelo
>
> baruch
>
> --
> http://baruch.siach.name/blog/ ~. .~ Tk Open Systems
> =}------------------------------------------------ooO--U--Ooo------------{=
> - baruch at tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il -
--
Profile: http://it.linkedin.com/in/compagnucciangelo
^ permalink raw reply [flat|nested] 6+ messages in thread* [Buildroot] [PATCH v2] package/sysdig: New package
2015-03-17 11:31 ` Angelo Compagnucci
@ 2015-03-17 11:48 ` Baruch Siach
2015-03-17 11:59 ` Angelo Compagnucci
0 siblings, 1 reply; 6+ messages in thread
From: Baruch Siach @ 2015-03-17 11:48 UTC (permalink / raw)
To: buildroot
Hi Angelo,
On Tue, Mar 17, 2015 at 12:31:48PM +0100, Angelo Compagnucci wrote:
> 2015-03-17 12:23 GMT+01:00 Baruch Siach <baruch@tkos.co.il>:
> > On Tue, Mar 17, 2015 at 11:56:48AM +0100, Angelo Compagnucci wrote:
> >> + 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
> >
> > A dependencies comment is still missing here
> > (http://nightly.buildroot.org/manual.html#_dependencies_on_a_linux_kernel_built_by_buildroot).
>
> The comment is there:
>
> @@ -0,0 +1,17 @@
> +comment "sysdig needs a Linux kernel to be built"
> + depends on !BR2_LINUX_KERNEL
You are right, I missed that. It's just that I'm used to see these comments
below the main 'config' section. Putting the comment above is also fine,
though. Don't forget to update this comment when adding indirect dependencies.
baruch
--
http://baruch.siach.name/blog/ ~. .~ Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
- baruch at tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il -
^ permalink raw reply [flat|nested] 6+ messages in thread* [Buildroot] [PATCH v2] package/sysdig: New package
2015-03-17 11:48 ` Baruch Siach
@ 2015-03-17 11:59 ` Angelo Compagnucci
2015-03-17 13:12 ` Thomas Petazzoni
0 siblings, 1 reply; 6+ messages in thread
From: Angelo Compagnucci @ 2015-03-17 11:59 UTC (permalink / raw)
To: buildroot
Dear Baruch Siach,
2015-03-17 12:48 GMT+01:00 Baruch Siach <baruch@tkos.co.il>:
> Hi Angelo,
>
> On Tue, Mar 17, 2015 at 12:31:48PM +0100, Angelo Compagnucci wrote:
>> 2015-03-17 12:23 GMT+01:00 Baruch Siach <baruch@tkos.co.il>:
>> > On Tue, Mar 17, 2015 at 11:56:48AM +0100, Angelo Compagnucci wrote:
>> >> + 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
>> >
>> > A dependencies comment is still missing here
>> > (http://nightly.buildroot.org/manual.html#_dependencies_on_a_linux_kernel_built_by_buildroot).
>>
>> The comment is there:
>>
>> @@ -0,0 +1,17 @@
>> +comment "sysdig needs a Linux kernel to be built"
>> + depends on !BR2_LINUX_KERNEL
>
> You are right, I missed that. It's just that I'm used to see these comments
> below the main 'config' section. Putting the comment above is also fine,
> though. Don't forget to update this comment when adding indirect dependencies.
I think that if I use select, the dependencies of this package will
explode. I think I can use select for simple, non problematic
packages, and use depends on for others.
I'll submit a patch in minutes!
Sincerely, Angelo
>
> baruch
>
> --
> http://baruch.siach.name/blog/ ~. .~ Tk Open Systems
> =}------------------------------------------------ooO--U--Ooo------------{=
> - baruch at tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il -
--
Profile: http://it.linkedin.com/in/compagnucciangelo
^ permalink raw reply [flat|nested] 6+ messages in thread* [Buildroot] [PATCH v2] package/sysdig: New package
2015-03-17 11:59 ` Angelo Compagnucci
@ 2015-03-17 13:12 ` Thomas Petazzoni
0 siblings, 0 replies; 6+ messages in thread
From: Thomas Petazzoni @ 2015-03-17 13:12 UTC (permalink / raw)
To: buildroot
Dear Angelo Compagnucci,
On Tue, 17 Mar 2015 12:59:11 +0100, Angelo Compagnucci wrote:
> > You are right, I missed that. It's just that I'm used to see these comments
> > below the main 'config' section. Putting the comment above is also fine,
> > though. Don't forget to update this comment when adding indirect dependencies.
>
> I think that if I use select, the dependencies of this package will
> explode. I think I can use select for simple, non problematic
> packages, and use depends on for others.
No. Please follow what Baruch said: use 'select' for BR2_PACKAGE_ZLIB,
BR2_PACKAGE_LUAJIT and BR2_PACKAGE_LIBJSON, and a 'depends on' for
BR2_LINUX_KERNEL.
Also, are you sure sysdig only works with LuaJIT, or does it also work
with the "normal" Lua interpreter?
Thanks,
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2015-03-17 13:12 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-03-17 10:56 [Buildroot] [PATCH v2] package/sysdig: New package Angelo Compagnucci
2015-03-17 11:23 ` Baruch Siach
2015-03-17 11:31 ` Angelo Compagnucci
2015-03-17 11:48 ` Baruch Siach
2015-03-17 11:59 ` Angelo Compagnucci
2015-03-17 13:12 ` Thomas Petazzoni
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox