<tt><font size=2>Thomas De Schampheleire <patrickdepinguin@gmail.com>
wrote on 11/12/2013 02:54:50 PM:<br>
<br>
> From: Thomas De Schampheleire <patrickdepinguin@gmail.com></font></tt>
<br><tt><font size=2>> To: Matt Weber <mlweber1@rockwellcollins.com>,
buildroot@busybox.net</font></tt>
<br><tt><font size=2>> Date: 11/12/2013 02:55 PM</font></tt>
<br><tt><font size=2>> Subject: Re: [Buildroot] [PATCH 1/1] omniorb:
add COS Naming Service</font></tt>
<br><tt><font size=2>> <br>
> Hi Matt,<br>
> <br>
> Matt Weber <mlweber1@rockwellcollins.com> wrote:<br>
> ><br>
> >Signed-off-by: Matt Weber <mlweber1@rockwellcollins.com><br>
> >---<br>
> > package/omniorb/Config.in  |    9 +++++++++<br>
> > package/omniorb/omniorb.mk |    8 ++++++++<br>
> > 2 files changed, 17 insertions(+), 0 deletions(-)<br>
> ><br>
> >diff --git a/package/omniorb/Config.in b/package/omniorb/Config.in<br>
> >index 6326688..22527fc 100644<br>
> >--- a/package/omniorb/Config.in<br>
> >+++ b/package/omniorb/Config.in<br>
> >@@ -10,5 +10,14 @@ config BR2_PACKAGE_OMNIORB<br>
> > <br>
> >      </font></tt><a href=http://omniorb.sourceforge.net/><tt><font size=2>http://omniorb.sourceforge.net/</font></tt></a><tt><font size=2><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.<br>
> I know that many packages do not follow this, but I'm planning on
<br>
> fixing that...<br>
> <br>
> Best regards,<br>
> Thomas<br>
> <br>
Sure that works.  I noticed the indenting of the new cfg option doesn't
seem </font></tt>
<br><tt><font size=2>to happen in the menuconfig if I move that comment
below the original package.</font></tt>
<br><tt><font size=2>I'll have to dig into why that happens.</font></tt>
<br>
<br><tt><font size=2>Thanks,</font></tt>
<br><tt><font size=2>Matt </font></tt>
<br>
<br>