[Buildroot] [PATCH 28/29] package/qt: drop option for license approved

[Yann E. MORIN <yann.morin.1998@free.fr>]

Arnout, All,

On 2017-04-23 22:59 +0200, Arnout Vandecappelle spake thusly:
> On 23-04-17 22:31, Yann E. MORIN wrote:
> > As stated in previous commits, Buildroot only ever uses the free
> > license, and we do not have a click-wrap approval for any such free
> > license for any other package.
> > 
> > The option to approve the license now has no prompt and is no longer
> > used anywhere. Drop it and make it a legacy entry.
> > 
> > Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> > Cc: Julien Corjon <corjon.j@ecagroup.com>
> > Cc: Peter Seiderer <ps.report@gmx.net>
> > ---
> >  Config.in.legacy     | 6 ++++++
> >  package/qt/Config.in | 4 ----
> >  2 files changed, 6 insertions(+), 4 deletions(-)
> > 
> > diff --git a/Config.in.legacy b/Config.in.legacy
> > index 8e9f3d175d..52cda9808e 100644
> > --- a/Config.in.legacy
> > +++ b/Config.in.legacy
> > @@ -145,6 +145,12 @@ endif
> >  ###############################################################################
> >  comment "Legacy options removed in 2017.05"
> >  
> > +config BR2_PACKAGE_QT_LICENSE_APPROVED
> > +	bool "Qt free license now always approved"
> > +	select BR2_LEGACY
> > +	help
> > +	  Use of Qt in Buildroot now always use the free license of Qt.
> 
>  IMO this doesn't bring any value. If the user already has selected to approve
> the free license, nothing really changes for them so there's no point drawing
> their attention to legacy.
> 
>  We should instead give a legacy warning when the user *didn't* have the option
> selected. But that's not possible, of course.
> 
>  Therefore, I'd say we drop the legacy handling for this option.

I'm fine with this.

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.  |
'------------------------------^-------^------------------^--------------------'