lkml.org 
[lkml]   [2014]   [Feb]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 24/28] Remove DEPRECATED
On 02/09/2014 01:38 PM, Richard Weinberger wrote:
> Am 09.02.2014 22:35, schrieb Guenter Roeck:
>> I could understand if you would replace DEPRECATED with BROKEN,
>> but causing a potentially large number of broken builds and
>> then leave it to others to clean up the resulting mess doesn't
>> make any sense to me and should, in my opinion, be rejected.
>
> Currently the mess is hidden by bad Kconfig dependencies.
> We have to face it. :)
>

You can make the breakage obvious by using BROKEN instead
of DEPRECATED. Then it can be fixed.

Otherwise you could, using your logic, remove all BROKEN
dependencies as well (all 90 or so of them). That symbol,
after all, is just as orphan as DEPRECATED.

The point here is to not unnecessarily introduce build problems.
There are people out there trying to ensure that no new build
breakages are introduced. By making X configurations
non-buildable, all you accomplish is to ensure that no one will
be able to discover real new build problems anymore, because
those new problems will all disappear in the build breakage
noise you introduced.

So, in practice, all you accomplish is to make the kernel
less maintainable. If that is your goal, mission accomplished.
Otherwise I would kindly ask you to reconsider.

Note that I would not mind some big cleanup effort to simply
remove all code marked as BROKEN, and I would be more than
happy to assist in such an effort.

However, breaking builds on purpose in the hope that someone
would magically show up and fix it is just plain wrong.

Note that I do not refer to a specific platform.

Guenter



\
 
 \ /
  Last update: 2014-02-10 00:41    [W:0.127 / U:0.408 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site