<tt><font size=2>Peter Korsgaard <jacmet@gmail.com> wrote on 11/12/2013
04:24:10 PM:<br>
<br>
> From: Peter Korsgaard <jacmet@uclibc.org></font></tt>
<br><tt><font size=2>> To: Thomas De Schampheleire <patrickdepinguin@gmail.com></font></tt>
<br><tt><font size=2>> Cc: Matt Weber <mlweber1@rockwellcollins.com>,
buildroot@busybox.net</font></tt>
<br><tt><font size=2>> Date: 11/12/2013 04:24 PM</font></tt>
<br><tt><font size=2>> Subject: Re: [PATCH 1/1] omniorb: add COS Naming
Service</font></tt>
<br><tt><font size=2>> Sent by: Peter Korsgaard <jacmet@gmail.com></font></tt>
<br><tt><font size=2>> <br>
> >>>>> "Thomas" == Thomas De Schampheleire
<patrickdepinguin@gmail.com> writes:<br>
> <br>
> Hi,<br>
> <br>
> >> +if BR2_PACKAGE_OMNIORB<br>
> >> +<br>
> >> +config BR2_PACKAGE_OMNIORB_WITH_SERVICES<br>
> >> +        bool "COS Naming Service"<br>
> >> +        default y<br>
> >> +        help<br>
> >> +          omniORB COS Naming Service<br>
> >> +endif<br>
> >> +<br>
> >> comment "omniORB needs a toolchain w/ C++"<br>
> >> depends on !BR2_INSTALL_LIBSTDCPP<br>
> <br>
> > I think it makes more sense to keep this comment close to the
config<br>
> > option it applies to, thus moving the new cos option below it.
 I know<br>
> > that many packages do not follow this, but I'm planning on fixing<br>
> > that...<br>
> <br>
> Careful, kconfig needs sub options to be directly under the main option,<br>
> otherwise they don't get indented correctly in menuconfig, so your<br>
> options are either:</font></tt>
<br>
<br><tt><font size=2>Thanks for the clarification, I was wondering if that
was needed.</font></tt>
<br><tt><font size=2>I just tested moving it above and that seems to be
more readable then below.</font></tt>
<br>
<br><tt><font size=2>Thanks,</font></tt>
<br><tt><font size=2>Matt</font></tt>