On Thu, Jan 31, 2019 at 06:26:41PM -0500, Chas Williams wrote:
>
>
> On 1/31/19 4:32 AM, Bruce Richardson wrote:
> > On Wed, Jan 30, 2019 at 09:18:58PM -0500, Chas Williams wrote:
> > > Allow users and packagers to override the default RTE_MAX_ETHPORTS.
> > > This adds a new meson option, max_ethports which defaults to the
> > > current value.
> > >
> > > Signed-off-by: Chas Williams <3ch...@gmail.com>
> >
> > Agree with the idea. Just one small comment below re alphabetical order.
> >
> > Otherwise:
> > Acked-by: Bruce Richardson <bruce.richard...@intel.com>
> >
> ...
> >
> > We are trying to keep the options in alphabetical order, so this setting
> > should be just before "max_lcores" rather than at the end.
>
> If you look at the last few lines in this file:
>
> option('use_hpet', type: 'boolean', value: false,
> description: 'use HPET timer in EAL')
> option('tests', type: 'boolean', value: true,
> description: 'build unit tests')
>
> I couldn't detect any ordering so I just appended. I can submit a v2.
Yes, that was an oversight that was missed when we were adding those
options, or I just got my alphabet scrambled in my mind. :-) Thanks for the
v2.
>
> > Interestingly, I see that since 0.45 there is now the option of an
> > "integer" option type. We should probably switch to use that once we bump
> > our minimum meson version. [Luca, perhaps another change for your set?]
> > Ref: http://mesonbuild.com/Build-options.html#integers
>
> I just followed along with what everyone else was doing.
Yep, no problem here. Since we still support from 0.41 onwards we can't use
integer anyway, so what you did is perfectly correct.
