All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] package/openvmtools: fix procps-ng library name
@ 2015-04-17 11:23 Karoly Kasza
  2015-04-18 17:16 ` Romain Naour
  2015-04-19  8:16 ` Thomas Petazzoni
  0 siblings, 2 replies; 3+ messages in thread
From: Karoly Kasza @ 2015-04-17 11:23 UTC (permalink / raw)
  To: buildroot

procps-ng libraries are now accesible on the same name as procps, thus
using the CUSTOM_PROCPS_NAME environment variable breaks compilation.

This fixes autobuild error 2a19f84c35de0ff008c2cd1aea96f6a4f2767ee1.

Signed-off-by: Karoly Kasza <kaszak@gmail.com>
---
 package/openvmtools/openvmtools.mk |    1 -
 1 file changed, 1 deletion(-)

diff --git a/package/openvmtools/openvmtools.mk b/package/openvmtools/openvmtools.mk
index f180f6f..ef6544a 100644
--- a/package/openvmtools/openvmtools.mk
+++ b/package/openvmtools/openvmtools.mk
@@ -33,7 +33,6 @@ endif
 
 ifeq ($(BR2_PACKAGE_OPENVMTOOLS_PROCPS),y)
 OPENVMTOOLS_CONF_OPTS += --with-procps
-OPENVMTOOLS_CONF_ENV += CUSTOM_PROCPS_NAME=procps
 OPENVMTOOLS_DEPENDENCIES += procps-ng
 else
 OPENVMTOOLS_CONF_OPTS += --without-procps
-- 
1.7.10.4

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [Buildroot] [PATCH] package/openvmtools: fix procps-ng library name
  2015-04-17 11:23 [Buildroot] [PATCH] package/openvmtools: fix procps-ng library name Karoly Kasza
@ 2015-04-18 17:16 ` Romain Naour
  2015-04-19  8:16 ` Thomas Petazzoni
  1 sibling, 0 replies; 3+ messages in thread
From: Romain Naour @ 2015-04-18 17:16 UTC (permalink / raw)
  To: buildroot

Hi Karoly,

Le 17/04/2015 13:23, Karoly Kasza a ?crit :
> procps-ng libraries are now accesible on the same name as procps, thus
> using the CUSTOM_PROCPS_NAME environment variable breaks compilation.
> 
> This fixes autobuild error 2a19f84c35de0ff008c2cd1aea96f6a4f2767ee1.

Usually we use a complete path like:
http://autobuild.buildroot.net/results/2a1/2a19f84c35de0ff008c2cd1aea96f6a4f2767ee1

> 
> Signed-off-by: Karoly Kasza <kaszak@gmail.com>
> ---

Issue reproduced locally.

Reviewed-by: Romain Naour <romain.naour@openwide.fr>
Tested-by: Romain Naour <romain.naour@openwide.fr>

Best regards,
Romain

>  package/openvmtools/openvmtools.mk |    1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/package/openvmtools/openvmtools.mk b/package/openvmtools/openvmtools.mk
> index f180f6f..ef6544a 100644
> --- a/package/openvmtools/openvmtools.mk
> +++ b/package/openvmtools/openvmtools.mk
> @@ -33,7 +33,6 @@ endif
>  
>  ifeq ($(BR2_PACKAGE_OPENVMTOOLS_PROCPS),y)
>  OPENVMTOOLS_CONF_OPTS += --with-procps
> -OPENVMTOOLS_CONF_ENV += CUSTOM_PROCPS_NAME=procps
>  OPENVMTOOLS_DEPENDENCIES += procps-ng
>  else
>  OPENVMTOOLS_CONF_OPTS += --without-procps
> 

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [Buildroot] [PATCH] package/openvmtools: fix procps-ng library name
  2015-04-17 11:23 [Buildroot] [PATCH] package/openvmtools: fix procps-ng library name Karoly Kasza
  2015-04-18 17:16 ` Romain Naour
@ 2015-04-19  8:16 ` Thomas Petazzoni
  1 sibling, 0 replies; 3+ messages in thread
From: Thomas Petazzoni @ 2015-04-19  8:16 UTC (permalink / raw)
  To: buildroot

Dear Karoly Kasza,

On Fri, 17 Apr 2015 13:23:00 +0200, Karoly Kasza wrote:
> procps-ng libraries are now accesible on the same name as procps, thus
> using the CUSTOM_PROCPS_NAME environment variable breaks compilation.
> 
> This fixes autobuild error 2a19f84c35de0ff008c2cd1aea96f6a4f2767ee1.
> 
> Signed-off-by: Karoly Kasza <kaszak@gmail.com>
> ---
>  package/openvmtools/openvmtools.mk |    1 -
>  1 file changed, 1 deletion(-)

Applied after fixing the autobuilder reference, thanks!

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

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2015-04-19  8:16 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-04-17 11:23 [Buildroot] [PATCH] package/openvmtools: fix procps-ng library name Karoly Kasza
2015-04-18 17:16 ` Romain Naour
2015-04-19  8:16 ` Thomas Petazzoni

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.