[Buildroot] [PATCH 1/1] omniorb: add COS Naming Service

[Peter Korsgaard]

>>>>> "Thomas" == Thomas De Schampheleire <patrickdepinguin at gmail.com> writes:

Hi,

>> +if BR2_PACKAGE_OMNIORB
>> +
>> +config BR2_PACKAGE_OMNIORB_WITH_SERVICES
>> +        bool "COS Naming Service"
>> +        default y
>> +        help
>> +          omniORB COS Naming Service
>> +endif
>> +
>> comment "omniORB needs a toolchain w/ C++"
>> depends on !BR2_INSTALL_LIBSTDCPP

> I think it makes more sense to keep this comment close to the config
> option it applies to, thus moving the new cos option below it.  I know
> that many packages do not follow this, but I'm planning on fixing
> that...

Careful, kconfig needs sub options to be directly under the main option,
otherwise they don't get indented correctly in menuconfig, so your
options are either:

- Comment before main option
- Comment after all the sub options

-- 
Bye, Peter Korsgaard