[PATCH 00/39] Windows port, base and archival/

Nguyen Thai Ngoc Duy pclouds at gmail.com
Tue May 4 19:45:07 UTC 2010


On Tue, Apr 27, 2010 at 9:33 AM, Denys Vlasenko
<vda.linux at googlemail.com> wrote:
> On Saturday 17 April 2010 21:48, Nguyen Thai Ngoc Duy wrote:
>> 2010/4/17 Denys Vlasenko <vda.linux at googlemail.com>:
>> > I think a positive form, "depends on TARGET_PLATFORM_POSIX",
>> > is better - you do not need to touch it for a gazillion of
>> > other weird platforms it does not run on (if you ever add them).
>> > Also, at this step, add it to every applet, not to applet groups.
>> > The patch will be bigger, but further pathes will be smaller,
>> > since they will only need to touch one entry whenever you make one
>> > more applet to work on WIN - you add " || TARGET_PLATFORM_NATIVE_WIN"
>> > for that applet.
>>
>> How about general features that do not depend on any applets? There
>> are some features that won't be supported on Windows, should it be
>> "depends on !PLATFORM_MINGW32" or "depends on PLATFORM_POSIX" for
>> them?
>
> depends on PLATFORM_POSIX

I've been thinking about this. Eventually any supported platform (or
whatever you call it) must support most applets, if not all. That
would make Config.in really messy, with most configs having "depends
on PLATFORM_POSIX || PLATFORM_NATIVE_WIN || PLATFORM_HURD". How about
this appoach instead? With KCONFIG_SUPPORT=defconfig.mingw32, all
items not listed in that file will be marked "BROKEN". This helps
users of new platforms identify which applets are good while keeping
Config.in clean.
-- 
Duy
-------------- next part --------------
A non-text attachment was scrubbed...
Name: broken.patch
Type: application/octet-stream
Size: 3531 bytes
Desc: not available
URL: <http://lists.busybox.net/pipermail/busybox/attachments/20100504/5d9f44d9/attachment-0001.obj>


More information about the busybox mailing list