[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