[Buildroot] [PATCH 6/8] ti-sgx-km: rename options to have proper prefix

Yann E. MORIN yann.morin.1998 at free.fr
Mon May 28 20:45:01 UTC 2018


Arnout, Peter, All,

On 2018-05-22 12:42 +0200, Arnout Vandecappelle spake thusly:
> On 21-05-18 23:16, Peter Korsgaard wrote:
> >>>>>> "Arnout" == Arnout Vandecappelle <arnout at mind.be> writes:
> > 
> >  > On 16-05-18 09:00, Thomas Petazzoni wrote:
> >  >> Hello,
> >  >> 
> >  >> On Tue, 15 May 2018 23:52:33 +0200, Arnout Vandecappelle wrote:
> >  >> 
> >  >>>> What is nice with this solution, is that we have the new choice that is
> >  >>>> right near the legacy option so we can chek it, and it automatically
> >  >>>> gets the old setting.
> >  >>>> 
> >  >>>> But I am not too convinced, however, because we do not usually use the
> >  >>>> name of a choice elsewhere...  
> >  >>> 
> >  >>> Legacy is pretty special anyway. I do think it's worthwhile it to do this properly.
> >  >>
> >  >> Is this an Acked-by to the approach this patch series has taken in
> >  >> terms of handling the Config.in choice option renaming ?
> > 
> >  >  Er, no, it's a NACK. I *do* think it's worthwhile to properly select the new
> >  > option.

Yeah, but as we've gone without support for legacy choices so far
(almost 6 years now), I don't think this is an urgent matter.

Next time, maybe I'll just shut up instead of suggesting sophisticated
and imaginative solutions. ;-P

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'



More information about the buildroot mailing list