[Buildroot] [PATCH 2/2] package/mono: new package

[Thomas Petazzoni <thomas.petazzoni@free-electrons.com>]

Dear Angelo Compagnucci,

On Sun, 12 Oct 2014 11:21:20 +0200, Angelo Compagnucci wrote:

> > Then, the conditional shouldn't be here, but around the dependency of
> > mono on host-mono.
> >
> > That being said, I would recommend you to keep things simple for now,
> > and handle only this use case as a follow-up contribution.
> 
> It is already:
> 
> ifneq ($(MONO_ASSEMBLY_INCLUDED),)
> define MONO_INSTALL_LIBS
> rsync -av --exclude=*.so --exclude=*.mdb --exclude=[0-9].[0-9] \
> $(HOST_DIR)/usr/lib/mono $(TARGET_DIR)/usr/lib/
> rsync -av --exclude=*.so --exclude=*.mdb $(MONO_ASSEMBLY_INCLUDED) \
> $(TARGET_DIR)/usr/lib/mono/
> endef
> MONO_DEPENDENCIES += host-mono
> MONO_POST_INSTALL_TARGET_HOOKS += MONO_INSTALL_LIBS
> endif
> 
> This way the dependency is met only when MONO_ASSEMBLY_INCLUDED.
> By the way, I'll remove this logic, make 4.5 as default version and if
> nothing selected, I'll default evenly on 4.5.

Ok, right. Then don't add a condition around the host-autotools-package
call, this is useless.

Thanks,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com