[Buildroot] [PATCH 1/1] omniorb: add COS Naming Service
Arnout Vandecappelle
arnout at mind.be
Tue Nov 12 21:25:40 UTC 2013
On 12/11/13 21:31, Matt Weber 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
> diff --git a/package/omniorb/omniorb.mk b/package/omniorb/omniorb.mk
> index 490ff93..cad50f8 100644
> --- a/package/omniorb/omniorb.mk
> +++ b/package/omniorb/omniorb.mk
> @@ -24,6 +24,14 @@ OMNIORB_INSTALL_TARGET = YES
> OMNIORB_CONF_OPT += --disable-longdouble
> HOST_OMNIORB_CONF_OPT += --disable-longdouble
>
> +define OMNIORB_ENABLE_SERVICES
> + $(SED) 's:SUBDIRS += lib:SUBDIRS += lib services:g' $(@D)/src/dir.mk
> +endef
> +
> +ifeq ($(BR2_PACKAGE_OMNIORB_WITH_SERVICES),y)
> + OMNIORB_POST_CONFIGURE_HOOKS += OMNIORB_ENABLE_SERVICES
We normally don't indent variable assignments within conditions (the
indentation of the sed statement is OK, because that's a shell command).
Also, wouldn't it be better to do this as post-patch hook?
Regards,
Arnout
> +endif
> +
> # omniORB is not completely cross-compile friendly and has some
> # assumptions where a couple host tools must be built and then
> # used by the target build. The host tools generate code from
>
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286500
Essensium/Mind http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
More information about the buildroot
mailing list