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

Thomas De Schampheleire patrickdepinguin at gmail.com
Tue Nov 12 20:54:50 UTC 2013


Hi Matt,

Matt Weber <mlweber1 at rockwellcollins.com> wrote:
>
>Signed-off-by: Matt Weber <mlweber1 at rockwellcollins.com>
>---
> package/omniorb/Config.in  |    9 +++++++++
> package/omniorb/omniorb.mk |    8 ++++++++
> 2 files changed, 17 insertions(+), 0 deletions(-)
>
>diff --git a/package/omniorb/Config.in b/package/omniorb/Config.in
>index 6326688..22527fc 100644
>--- a/package/omniorb/Config.in
>+++ b/package/omniorb/Config.in
>@@ -10,5 +10,14 @@ config BR2_PACKAGE_OMNIORB
> 
> 	  http://omniorb.sourceforge.net/
> 
>+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...

Best regards,
Thomas



More information about the buildroot mailing list