[LEDE-DEV] [PATCH] iftop: Update to latest version, and drop patch
Russell Senior
russell at personaltelco.net
Sat May 7 11:11:45 PDT 2016
>>>>> "Bert" == Bert Vermeulen <bert at biot.com> writes:
Bert> On 05/07/2016 12:46 PM, Felix Fietkau wrote:
>> On 2016-05-07 12:40, Bert Vermeulen wrote:
>>> On 05/07/2016 12:01 PM, Felix Fietkau wrote:
>>>> We should probably just move everything to libncursesw and drop
>>>> libncurses entirely.
>>>
>>> Actually I'd gotten this wrong: libncursesw is an extra build option
>>> to libncurses, so you only get it if you explicitly want it. In that
>>> case though, you specifically DO want your applications to use
>>> ncursesw instead of regular ncurses.
>> It's a build variant, meaning both can be built and staged at the
>> same time.
>>
>>> So regardless of whether you keep libncurses around or not, I think
>>> patches preferring one or the other should be dropped, right?
>> What the package ends up using absolutely needs to match the
>> dependencies that the package specifies (otherwise you get a build
>> error). Allowing the package source build system to simply pick one
>> will result in very quirky behavior.
Bert> Ok, so will you drop regular libncurses? Or do you need me to
Bert> submit a patch to do it?
In this particular case, as the author of the patch involved, I would
suggest "if it isn't broken, don't fix it". That is, iftop doesn't
really need ncursesw, the patch lets it use plain ncurses even if
ncursesw is available. I don't see the problem leaving things like
that. It means carrying a small patch. Seems like not that big a
problem.
--
Russell Senior, President
russell at personaltelco.net
More information about the Lede-dev
mailing list