diff mbox

[1/1] libtirpc: rpcgen is only built for host, don't install to target

Message ID 1446584983-29703-1-git-send-email-phil@zankapfel.net
State Accepted
Headers show

Commit Message

Phil Eichinger Nov. 3, 2015, 9:09 p.m. UTC
rpcgen is only used to generate headers needed for compilation.
Move it to noinst_PROGRAMS otherwise you may end up with a host-arch
binary in your target.

Signed-off-by: Phil Eichinger <phil@zankapfel.net>
---
 package/libtirpc/0003-Add-rpcgen-program-from-nfs-utils-sources.patch | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Arnout Vandecappelle Nov. 3, 2015, 10:43 p.m. UTC | #1
On 03-11-15 22:09, Phil Eichinger wrote:
> rpcgen is only used to generate headers needed for compilation.
> Move it to noinst_PROGRAMS otherwise you may end up with a host-arch
> binary in your target.
> 
> Signed-off-by: Phil Eichinger <phil@zankapfel.net>

Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>

 Note that nfs-utils-1.3.3 actually defines it in noinst_PROGRAMS.

 Regards,
 Arnout

> ---
>  package/libtirpc/0003-Add-rpcgen-program-from-nfs-utils-sources.patch | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/package/libtirpc/0003-Add-rpcgen-program-from-nfs-utils-sources.patch b/package/libtirpc/0003-Add-rpcgen-program-from-nfs-utils-sources.patch
> index ad14312..1cf8614 100644
> --- a/package/libtirpc/0003-Add-rpcgen-program-from-nfs-utils-sources.patch
> +++ b/package/libtirpc/0003-Add-rpcgen-program-from-nfs-utils-sources.patch
> @@ -87,7 +87,7 @@ index 0000000..2277b6f
>  +	$(CPPFLAGS_FOR_BUILD) $(AM_CFLAGS) $(CFLAGS_FOR_BUILD)
>  +LINK = $(CC_FOR_BUILD) $(AM_CFLAGS) $(CFLAGS_FOR_BUILD) $(AM_LDFLAGS) $(LDFLAGS_FOR_BUILD) -o $@
>  +
> -+bin_PROGRAMS = rpcgen
> ++noinst_PROGRAMS = rpcgen
>  +
>  +rpcgen_SOURCES = \
>  +	rpc_clntout.c \
>
Thomas Petazzoni Nov. 3, 2015, 10:49 p.m. UTC | #2
Dear Phil Eichinger,

On Tue,  3 Nov 2015 22:09:43 +0100, Phil Eichinger wrote:
> rpcgen is only used to generate headers needed for compilation.
> Move it to noinst_PROGRAMS otherwise you may end up with a host-arch
> binary in your target.
> 
> Signed-off-by: Phil Eichinger <phil@zankapfel.net>
> ---
>  package/libtirpc/0003-Add-rpcgen-program-from-nfs-utils-sources.patch | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Applied, thanks.

Thomas
diff mbox

Patch

diff --git a/package/libtirpc/0003-Add-rpcgen-program-from-nfs-utils-sources.patch b/package/libtirpc/0003-Add-rpcgen-program-from-nfs-utils-sources.patch
index ad14312..1cf8614 100644
--- a/package/libtirpc/0003-Add-rpcgen-program-from-nfs-utils-sources.patch
+++ b/package/libtirpc/0003-Add-rpcgen-program-from-nfs-utils-sources.patch
@@ -87,7 +87,7 @@  index 0000000..2277b6f
 +	$(CPPFLAGS_FOR_BUILD) $(AM_CFLAGS) $(CFLAGS_FOR_BUILD)
 +LINK = $(CC_FOR_BUILD) $(AM_CFLAGS) $(CFLAGS_FOR_BUILD) $(AM_LDFLAGS) $(LDFLAGS_FOR_BUILD) -o $@
 +
-+bin_PROGRAMS = rpcgen
++noinst_PROGRAMS = rpcgen
 +
 +rpcgen_SOURCES = \
 +	rpc_clntout.c \