diff mbox series

[1/1] package/libev: use host-xxx-package macros

Message ID 20191001091628.7760-1-heiko.thiery@gmail.com
State Superseded
Headers show
Series [1/1] package/libev: use host-xxx-package macros | expand

Commit Message

Heiko Thiery Oct. 1, 2019, 9:16 a.m. UTC 
From: Heiko Thiery <heiko.thiery@kontron.com>

Signed-off-by: Heiko Thiery <heiko.thiery@kontron.com>
---
 package/libev/libev.mk | 1 +
 1 file changed, 1 insertion(+)

Comments

Thomas Petazzoni Oct. 1, 2019, 9:37 a.m. UTC | #1 
On Tue,  1 Oct 2019 11:16:28 +0200
heiko.thiery@gmail.com wrote:

> From: Heiko Thiery <heiko.thiery@kontron.com>
> 
> Signed-off-by: Heiko Thiery <heiko.thiery@kontron.com>

Why ? Why do you need the host variant of this library ?

Thanks,

Thomas
Heiko Thiery Oct. 1, 2019, 11:24 a.m. UTC | #2 
>
> Why ? Why do you need the host variant of this library ?
>

I need this(these) library(ies) as a dependency for a tool that is also
used in the installation step to prepare binary data.

I detail it is used for the integration of a netconf server. This server
has to prepare some YANG models during the installation phase where the
host tool of the sysrep project is needed as requirement.

I know the better solution would be to only compile the needed tool (that
in fact does not need the libev and libssh) but the build environment of
this package has no option to only compile the helper tools.

--
Heiko
Thomas Petazzoni Oct. 1, 2019, 1:48 p.m. UTC | #3 
On Tue, 1 Oct 2019 13:24:54 +0200
Heiko Thiery <heiko.thiery@gmail.com> wrote:

> I need this(these) library(ies) as a dependency for a tool that is also
> used in the installation step to prepare binary data.
> 
> I detail it is used for the integration of a netconf server. This server
> has to prepare some YANG models during the installation phase where the
> host tool of the sysrep project is needed as requirement.
> 
> I know the better solution would be to only compile the needed tool (that
> in fact does not need the libev and libssh) but the build environment of
> this package has no option to only compile the helper tools.

Is this tool part of a package that you intend to submit in Buildroot ?

Thomas
Heiko Thiery Oct. 1, 2019, 1:50 p.m. UTC | #4 
> Is this tool part of a package that you intend to submit in Buildroot ?
yes
Thomas Petazzoni Oct. 1, 2019, 2:01 p.m. UTC | #5 
On Tue, 1 Oct 2019 15:50:49 +0200
Heiko Thiery <heiko.thiery@gmail.com> wrote:

> > Is this tool part of a package that you intend to submit in Buildroot ?  
> yes

Then could you send those patches as part of a patch series together
with the patch adding the package that actually uses those host
libraries ?

Thanks!

Thomas
Heiko Thiery Oct. 1, 2019, 2:32 p.m. UTC | #6 
> Then could you send those patches as part of a patch series together
> with the patch adding the package that actually uses those host
> libraries ?

Just send the patches for all of the pending packages.
diff mbox series

Patch

diff --git a/package/libev/libev.mk b/package/libev/libev.mk
index 2b2d5db004..38b0530f62 100644
--- a/package/libev/libev.mk
+++ b/package/libev/libev.mk
@@ -18,3 +18,4 @@  endef
 LIBEV_POST_PATCH_HOOKS += LIBEV_DISABLE_EVENT_H_INSTALL
 
 $(eval $(autotools-package))
+$(eval $(host-autotools-package))