All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] iproute2: enable previously unselected utilities
@ 2015-11-30 12:11 Maxin B. John
  2015-12-01  9:27 ` Anders Darander
  0 siblings, 1 reply; 6+ messages in thread
From: Maxin B. John @ 2015-11-30 12:11 UTC (permalink / raw)
  To: openembedded-core

Include below listed utilities that are not yet enabled/packaged
in the iproute2 recipe:

lnstat
ifstat
genl
rtacct
nstat
ss

Signed-off-by: Maxin B. John <maxin.john@intel.com>
---
 meta/recipes-connectivity/iproute2/iproute2.inc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-connectivity/iproute2/iproute2.inc b/meta/recipes-connectivity/iproute2/iproute2.inc
index 29f9062..abaff3f 100644
--- a/meta/recipes-connectivity/iproute2/iproute2.inc
+++ b/meta/recipes-connectivity/iproute2/iproute2.inc
@@ -13,7 +13,7 @@ DEPENDS = "flex-native bison-native iptables elfutils"
 
 inherit update-alternatives
 
-EXTRA_OEMAKE = "CC='${CC}' KERNEL_INCLUDE=${STAGING_INCDIR} DOCDIR=${docdir}/iproute2 SUBDIRS='lib tc ip' SBINDIR='${base_sbindir}' LIBDIR='${libdir}'"
+EXTRA_OEMAKE = "CC='${CC}' KERNEL_INCLUDE=${STAGING_INCDIR} DOCDIR=${docdir}/iproute2 SUBDIRS='lib tc ip misc genl' SBINDIR='${base_sbindir}' LIBDIR='${libdir}'"
 
 do_configure_append () {
     sh configure ${STAGING_INCDIR}
-- 
2.4.0



^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH] iproute2: enable previously unselected utilities
  2015-12-01  9:29   ` Khem Raj
@ 2015-12-01  5:58     ` Maxin B. John
  2016-02-10  2:47       ` Randy MacLeod
  0 siblings, 1 reply; 6+ messages in thread
From: Maxin B. John @ 2015-12-01  5:58 UTC (permalink / raw)
  To: Khem Raj; +Cc: Patches and discussions about the oe-core layer

Hi,

On Tue, Dec 01, 2015 at 01:29:43AM -0800, Khem Raj wrote:
> On Tue, Dec 1, 2015 at 1:27 AM, Anders Darander <anders@chargestorm.se> wrote:
> > Hi,
> >
> > Just a couple of questions.
> >
> > * Maxin B. John <maxin.john@intel.com> [151130 17:59]:
> >> Include below listed utilities that are not yet enabled/packaged
> >> in the iproute2 recipe:
> >
> >> lnstat
> >> ifstat
> >> genl
> >> rtacct
> >> nstat
> >> ss
> >
> > When you enable those, how much does the size ofthe iproute2 package
> > increase?
> >

The increase in size is around 268K. 

$ du -sh iproute2
476K    iproute2

$ du -sh iproute2-new
744K    iproute2-new

> > Perhaps those extra tools should be placed in e.g. ${PN}-tools, or
> > some package like that?
> 
> I think its better to package each one in separate package of its own.
>

Ok. I will split these binaries in separate packages.

> >
> > Cheers,
> > Anders
> >
> >> Signed-off-by: Maxin B. John <maxin.john@intel.com>
> >> ---
> >>  meta/recipes-connectivity/iproute2/iproute2.inc | 2 +-
> >>  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> >> diff --git a/meta/recipes-connectivity/iproute2/iproute2.inc b/meta/recipes-connectivity/iproute2/iproute2.inc
> >> index 29f9062..abaff3f 100644
> >> --- a/meta/recipes-connectivity/iproute2/iproute2.inc
> >> +++ b/meta/recipes-connectivity/iproute2/iproute2.inc
> >> @@ -13,7 +13,7 @@ DEPENDS = "flex-native bison-native iptables elfutils"
> >
> >>  inherit update-alternatives
> >
> >> -EXTRA_OEMAKE = "CC='${CC}' KERNEL_INCLUDE=${STAGING_INCDIR} DOCDIR=${docdir}/iproute2 SUBDIRS='lib tc ip' SBINDIR='${base_sbindir}' LIBDIR='${libdir}'"
> >> +EXTRA_OEMAKE = "CC='${CC}' KERNEL_INCLUDE=${STAGING_INCDIR} DOCDIR=${docdir}/iproute2 SUBDIRS='lib tc ip misc genl' SBINDIR='${base_sbindir}' LIBDIR='${libdir}'"
> >
> >>  do_configure_append () {
> >>      sh configure ${STAGING_INCDIR}
> > --
> > Anders Darander, Senior System Architect
> > ChargeStorm AB / eStorm AB
> > --

Best Regards,
Maxin



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] iproute2: enable previously unselected utilities
  2015-11-30 12:11 [PATCH] iproute2: enable previously unselected utilities Maxin B. John
@ 2015-12-01  9:27 ` Anders Darander
  2015-12-01  9:29   ` Khem Raj
  0 siblings, 1 reply; 6+ messages in thread
From: Anders Darander @ 2015-12-01  9:27 UTC (permalink / raw)
  To: openembedded-core

Hi,

Just a couple of questions.

* Maxin B. John <maxin.john@intel.com> [151130 17:59]:
> Include below listed utilities that are not yet enabled/packaged
> in the iproute2 recipe:

> lnstat
> ifstat
> genl
> rtacct
> nstat
> ss

When you enable those, how much does the size ofthe iproute2 package
increase?

Perhaps those extra tools should be placed in e.g. ${PN}-tools, or
some package like that?

Cheers,
Anders

> Signed-off-by: Maxin B. John <maxin.john@intel.com>
> ---
>  meta/recipes-connectivity/iproute2/iproute2.inc | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

> diff --git a/meta/recipes-connectivity/iproute2/iproute2.inc b/meta/recipes-connectivity/iproute2/iproute2.inc
> index 29f9062..abaff3f 100644
> --- a/meta/recipes-connectivity/iproute2/iproute2.inc
> +++ b/meta/recipes-connectivity/iproute2/iproute2.inc
> @@ -13,7 +13,7 @@ DEPENDS = "flex-native bison-native iptables elfutils"

>  inherit update-alternatives

> -EXTRA_OEMAKE = "CC='${CC}' KERNEL_INCLUDE=${STAGING_INCDIR} DOCDIR=${docdir}/iproute2 SUBDIRS='lib tc ip' SBINDIR='${base_sbindir}' LIBDIR='${libdir}'"
> +EXTRA_OEMAKE = "CC='${CC}' KERNEL_INCLUDE=${STAGING_INCDIR} DOCDIR=${docdir}/iproute2 SUBDIRS='lib tc ip misc genl' SBINDIR='${base_sbindir}' LIBDIR='${libdir}'"

>  do_configure_append () {
>      sh configure ${STAGING_INCDIR}
-- 
Anders Darander, Senior System Architect
ChargeStorm AB / eStorm AB


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] iproute2: enable previously unselected utilities
  2015-12-01  9:27 ` Anders Darander
@ 2015-12-01  9:29   ` Khem Raj
  2015-12-01  5:58     ` Maxin B. John
  0 siblings, 1 reply; 6+ messages in thread
From: Khem Raj @ 2015-12-01  9:29 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

On Tue, Dec 1, 2015 at 1:27 AM, Anders Darander <anders@chargestorm.se> wrote:
> Hi,
>
> Just a couple of questions.
>
> * Maxin B. John <maxin.john@intel.com> [151130 17:59]:
>> Include below listed utilities that are not yet enabled/packaged
>> in the iproute2 recipe:
>
>> lnstat
>> ifstat
>> genl
>> rtacct
>> nstat
>> ss
>
> When you enable those, how much does the size ofthe iproute2 package
> increase?
>
> Perhaps those extra tools should be placed in e.g. ${PN}-tools, or
> some package like that?

I think its better to package each one in separate package of its own.

>
> Cheers,
> Anders
>
>> Signed-off-by: Maxin B. John <maxin.john@intel.com>
>> ---
>>  meta/recipes-connectivity/iproute2/iproute2.inc | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>
>> diff --git a/meta/recipes-connectivity/iproute2/iproute2.inc b/meta/recipes-connectivity/iproute2/iproute2.inc
>> index 29f9062..abaff3f 100644
>> --- a/meta/recipes-connectivity/iproute2/iproute2.inc
>> +++ b/meta/recipes-connectivity/iproute2/iproute2.inc
>> @@ -13,7 +13,7 @@ DEPENDS = "flex-native bison-native iptables elfutils"
>
>>  inherit update-alternatives
>
>> -EXTRA_OEMAKE = "CC='${CC}' KERNEL_INCLUDE=${STAGING_INCDIR} DOCDIR=${docdir}/iproute2 SUBDIRS='lib tc ip' SBINDIR='${base_sbindir}' LIBDIR='${libdir}'"
>> +EXTRA_OEMAKE = "CC='${CC}' KERNEL_INCLUDE=${STAGING_INCDIR} DOCDIR=${docdir}/iproute2 SUBDIRS='lib tc ip misc genl' SBINDIR='${base_sbindir}' LIBDIR='${libdir}'"
>
>>  do_configure_append () {
>>      sh configure ${STAGING_INCDIR}
> --
> Anders Darander, Senior System Architect
> ChargeStorm AB / eStorm AB
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] iproute2: enable previously unselected utilities
  2015-12-01  5:58     ` Maxin B. John
@ 2016-02-10  2:47       ` Randy MacLeod
  2016-02-10  9:13         ` Maxin B. John
  0 siblings, 1 reply; 6+ messages in thread
From: Randy MacLeod @ 2016-02-10  2:47 UTC (permalink / raw)
  To: Maxin B. John; +Cc: Patches and discussions about the oe-core layer

On 2015-12-01 12:58 AM, Maxin B. John wrote:
> Hi,
>
> On Tue, Dec 01, 2015 at 01:29:43AM -0800, Khem Raj wrote:
>> On Tue, Dec 1, 2015 at 1:27 AM, Anders Darander <anders@chargestorm.se> wrote:
>>> Hi,
>>>
>>> Just a couple of questions.
>>>
>>> * Maxin B. John <maxin.john@intel.com> [151130 17:59]:
>>>> Include below listed utilities that are not yet enabled/packaged
>>>> in the iproute2 recipe:
>>>
>>>> lnstat
>>>> ifstat
>>>> genl
>>>> rtacct
>>>> nstat
>>>> ss
>>>
>>> When you enable those, how much does the size ofthe iproute2 package
>>> increase?
>>>
>
> The increase in size is around 268K.
>
> $ du -sh iproute2
> 476K    iproute2
>
> $ du -sh iproute2-new
> 744K    iproute2-new
>
>>> Perhaps those extra tools should be placed in e.g. ${PN}-tools, or
>>> some package like that?
>>
>> I think its better to package each one in separate package of its own.
>>
>
> Ok. I will split these binaries in separate packages.

Maxin,

Will you be able to do this as part of the 4.4 upgrade that
you have just sent or would you like Robert to find someone
to do it when he is back from holiday next week?

../Randy
>
>>>
>>> Cheers,
>>> Anders
>>>
>>>> Signed-off-by: Maxin B. John <maxin.john@intel.com>
>>>> ---
>>>>   meta/recipes-connectivity/iproute2/iproute2.inc | 2 +-
>>>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>>> diff --git a/meta/recipes-connectivity/iproute2/iproute2.inc b/meta/recipes-connectivity/iproute2/iproute2.inc
>>>> index 29f9062..abaff3f 100644
>>>> --- a/meta/recipes-connectivity/iproute2/iproute2.inc
>>>> +++ b/meta/recipes-connectivity/iproute2/iproute2.inc
>>>> @@ -13,7 +13,7 @@ DEPENDS = "flex-native bison-native iptables elfutils"
>>>
>>>>   inherit update-alternatives
>>>
>>>> -EXTRA_OEMAKE = "CC='${CC}' KERNEL_INCLUDE=${STAGING_INCDIR} DOCDIR=${docdir}/iproute2 SUBDIRS='lib tc ip' SBINDIR='${base_sbindir}' LIBDIR='${libdir}'"
>>>> +EXTRA_OEMAKE = "CC='${CC}' KERNEL_INCLUDE=${STAGING_INCDIR} DOCDIR=${docdir}/iproute2 SUBDIRS='lib tc ip misc genl' SBINDIR='${base_sbindir}' LIBDIR='${libdir}'"
>>>
>>>>   do_configure_append () {
>>>>       sh configure ${STAGING_INCDIR}
>>> --
>>> Anders Darander, Senior System Architect
>>> ChargeStorm AB / eStorm AB
>>> --
>
> Best Regards,
> Maxin
>


-- 
# Randy MacLeod. SMTS, Linux, Wind River
Direct: 613.963.1350 | 350 Terry Fox Drive, Suite 200, Ottawa, ON, 
Canada, K2K 2W5


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] iproute2: enable previously unselected utilities
  2016-02-10  2:47       ` Randy MacLeod
@ 2016-02-10  9:13         ` Maxin B. John
  0 siblings, 0 replies; 6+ messages in thread
From: Maxin B. John @ 2016-02-10  9:13 UTC (permalink / raw)
  To: Randy MacLeod; +Cc: Patches and discussions about the oe-core layer

Hi Randy,

On Tue, Feb 09, 2016 at 09:47:35PM -0500, Randy MacLeod wrote:
> On 2015-12-01 12:58 AM, Maxin B. John wrote:
> >Hi,
> >
> >On Tue, Dec 01, 2015 at 01:29:43AM -0800, Khem Raj wrote:
> >>On Tue, Dec 1, 2015 at 1:27 AM, Anders Darander <anders@chargestorm.se> wrote:
> >>>Hi,
> >>>
> >>>Just a couple of questions.
> >>>
> >>>* Maxin B. John <maxin.john@intel.com> [151130 17:59]:
> >>>>Include below listed utilities that are not yet enabled/packaged
> >>>>in the iproute2 recipe:
> >>>
> >>>>lnstat
> >>>>ifstat
> >>>>genl
> >>>>rtacct
> >>>>nstat
> >>>>ss
> >>>
> >>>When you enable those, how much does the size ofthe iproute2 package
> >>>increase?
> >>>
> >
> >The increase in size is around 268K.
> >
> >$ du -sh iproute2
> >476K    iproute2
> >
> >$ du -sh iproute2-new
> >744K    iproute2-new
> >
> >>>Perhaps those extra tools should be placed in e.g. ${PN}-tools, or
> >>>some package like that?
> >>
> >>I think its better to package each one in separate package of its own.
> >>
> >
> >Ok. I will split these binaries in separate packages.
> 
> Maxin,
> 
> Will you be able to do this as part of the 4.4 upgrade that
> you have just sent or would you like Robert to find someone
> to do it when he is back from holiday next week?
>
Ah, I forgot about that change (thinking that it is already there).

Will send a v2 with those changes.


> ../Randy

Best Regards,
Maxin

> >>>
> >>>Cheers,
> >>>Anders
> >>>
> >>>>Signed-off-by: Maxin B. John <maxin.john@intel.com>
> >>>>---
> >>>>  meta/recipes-connectivity/iproute2/iproute2.inc | 2 +-
> >>>>  1 file changed, 1 insertion(+), 1 deletion(-)
> >>>
> >>>>diff --git a/meta/recipes-connectivity/iproute2/iproute2.inc b/meta/recipes-connectivity/iproute2/iproute2.inc
> >>>>index 29f9062..abaff3f 100644
> >>>>--- a/meta/recipes-connectivity/iproute2/iproute2.inc
> >>>>+++ b/meta/recipes-connectivity/iproute2/iproute2.inc
> >>>>@@ -13,7 +13,7 @@ DEPENDS = "flex-native bison-native iptables elfutils"
> >>>
> >>>>  inherit update-alternatives
> >>>
> >>>>-EXTRA_OEMAKE = "CC='${CC}' KERNEL_INCLUDE=${STAGING_INCDIR} DOCDIR=${docdir}/iproute2 SUBDIRS='lib tc ip' SBINDIR='${base_sbindir}' LIBDIR='${libdir}'"
> >>>>+EXTRA_OEMAKE = "CC='${CC}' KERNEL_INCLUDE=${STAGING_INCDIR} DOCDIR=${docdir}/iproute2 SUBDIRS='lib tc ip misc genl' SBINDIR='${base_sbindir}' LIBDIR='${libdir}'"
> >>>
> >>>>  do_configure_append () {
> >>>>      sh configure ${STAGING_INCDIR}
> >>>--
> >>>Anders Darander, Senior System Architect
> >>>ChargeStorm AB / eStorm AB
> >>>--
> >
> >Best Regards,
> >Maxin
> >
> 
> 
> -- 
> # Randy MacLeod. SMTS, Linux, Wind River
> Direct: 613.963.1350 | 350 Terry Fox Drive, Suite 200, Ottawa, ON, Canada,
> K2K 2W5


^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2016-02-10  9:10 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-11-30 12:11 [PATCH] iproute2: enable previously unselected utilities Maxin B. John
2015-12-01  9:27 ` Anders Darander
2015-12-01  9:29   ` Khem Raj
2015-12-01  5:58     ` Maxin B. John
2016-02-10  2:47       ` Randy MacLeod
2016-02-10  9:13         ` Maxin B. John

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.