> -----Original Message-----
> From: Thomas Monjalon <[email protected]>
> Sent: Thursday, March 18, 2021 6:01 AM
> To: Zhang, Qi Z <[email protected]>; Andrew Rybchenko
> <[email protected]>; Yigit, Ferruh <[email protected]>
> Cc: [email protected]; Richardson, Bruce <[email protected]>; Wang,
> Xiao W <[email protected]>; Wu, Jingjing <[email protected]>; Xing,
> Beilei <[email protected]>; Guo, Jia <[email protected]>; Yang, Qiming
> <[email protected]>; Wang, Haiyue <[email protected]>
> Subject: Re: [PATCH v3 2/9] net: replace build option
>
> 17/03/2021 22:47, Ferruh Yigit:
> > On 3/17/2021 11:15 AM, Qi Zhang wrote:
> > > Replace RTE_LIBRTE_ETHDEV_DEBUG with RTE_ETHDEV_DEBUG.
> > >
> > > Signed-off-by: Qi Zhang <[email protected]>
> >
> > Hi Qi, Thomas,
> >
> > I see Thomas mentioned 'LIBRTE_' is redundant but not sure about
> > renaming the existing compile time flag without notice.
> >
> > The flag is to be used for users, it is not internal, and it may be
> > already used in some scripts and automation etc, this rename can break
> them.
> >
> > 'LIBRTE_' is already used commonly, I think all library flags has it,
> > do we really need to drop it from this current one? Even if we drop
> > it, what about keeping it as alias for backward compatibility?
>
> Yes I did not ask for removing now :)
> Let's keep old name as alias and start using the new name (if you agree with
> the new name without LIBRTE_).
OK, adding an alias makes sense to me
Thanks
Qi
>
