[Buildroot] [PATCH v2] package/sysdig: New package

Angelo Compagnucci angelo.compagnucci at gmail.com
Tue Mar 17 11:59:11 UTC 2015


Dear Baruch Siach,

2015-03-17 12:48 GMT+01:00 Baruch Siach <baruch at 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 at 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


More information about the buildroot mailing list