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:3.734 / U:0.028 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site