Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] package/openocd: add missing host-pkgconf dependency
@ 2018-11-15 16:46 Thomas Petazzoni
  2018-11-16 22:29 ` Peter Korsgaard
  2018-11-26 11:46 ` Peter Korsgaard
  0 siblings, 2 replies; 3+ messages in thread
From: Thomas Petazzoni @ 2018-11-15 16:46 UTC (permalink / raw)
  To: buildroot

The configure.ac script uses PKG_CHECK_MODULES(), and we autoreconf
the package, so host-pkgconf should be listed in the dependencies.

This issue is seen either with per-package folders, or by doing a
clean build with just "make openocd":

>>> openocd 0.10.0 Configuring
>>> openocd 0.10.0 Autoreconfiguring
[...]
configure.ac:12: error: possibly undefined macro: AC_MSG_WARN
      If this token and others are legitimate, please use m4_pattern_allow.
      See the Autoconf documentation.
configure.ac:201: error: possibly undefined macro: AC_DEFINE
configure.ac:582: error: possibly undefined macro: AC_MSG_NOTICE

Even if the message seems unrelated, it's really the lack of pkg.m4
from host-pkgconf that causes the issue.

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
---
 package/openocd/openocd.mk | 1 +
 1 file changed, 1 insertion(+)

diff --git a/package/openocd/openocd.mk b/package/openocd/openocd.mk
index 548d8b81a0..ca13d99366 100644
--- a/package/openocd/openocd.mk
+++ b/package/openocd/openocd.mk
@@ -31,6 +31,7 @@ OPENOCD_CONF_OPTS = \
 # the dependencies they need.
 
 OPENOCD_DEPENDENCIES = \
+	host-pkgconf \
 	$(if $(BR2_PACKAGE_LIBFTDI1),libftdi1) \
 	$(if $(BR2_PACKAGE_LIBUSB),libusb) \
 	$(if $(BR2_PACKAGE_LIBUSB_COMPAT),libusb-compat) \
-- 
2.19.1

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

* [Buildroot] [PATCH] package/openocd: add missing host-pkgconf dependency
  2018-11-15 16:46 [Buildroot] [PATCH] package/openocd: add missing host-pkgconf dependency Thomas Petazzoni
@ 2018-11-16 22:29 ` Peter Korsgaard
  2018-11-26 11:46 ` Peter Korsgaard
  1 sibling, 0 replies; 3+ messages in thread
From: Peter Korsgaard @ 2018-11-16 22:29 UTC (permalink / raw)
  To: buildroot

>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni@bootlin.com> writes:

 > The configure.ac script uses PKG_CHECK_MODULES(), and we autoreconf
 > the package, so host-pkgconf should be listed in the dependencies.

 > This issue is seen either with per-package folders, or by doing a
 > clean build with just "make openocd":

 >>>> openocd 0.10.0 Configuring
 >>>> openocd 0.10.0 Autoreconfiguring
 > [...]
 > configure.ac:12: error: possibly undefined macro: AC_MSG_WARN
 >       If this token and others are legitimate, please use m4_pattern_allow.
 >       See the Autoconf documentation.
 > configure.ac:201: error: possibly undefined macro: AC_DEFINE
 > configure.ac:582: error: possibly undefined macro: AC_MSG_NOTICE

 > Even if the message seems unrelated, it's really the lack of pkg.m4
 > from host-pkgconf that causes the issue.

 > Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>


Committed, thanks.

-- 
Bye, Peter Korsgaard

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

* [Buildroot] [PATCH] package/openocd: add missing host-pkgconf dependency
  2018-11-15 16:46 [Buildroot] [PATCH] package/openocd: add missing host-pkgconf dependency Thomas Petazzoni
  2018-11-16 22:29 ` Peter Korsgaard
@ 2018-11-26 11:46 ` Peter Korsgaard
  1 sibling, 0 replies; 3+ messages in thread
From: Peter Korsgaard @ 2018-11-26 11:46 UTC (permalink / raw)
  To: buildroot

>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni@bootlin.com> writes:

 > The configure.ac script uses PKG_CHECK_MODULES(), and we autoreconf
 > the package, so host-pkgconf should be listed in the dependencies.

 > This issue is seen either with per-package folders, or by doing a
 > clean build with just "make openocd":

 >>>> openocd 0.10.0 Configuring
 >>>> openocd 0.10.0 Autoreconfiguring
 > [...]
 > configure.ac:12: error: possibly undefined macro: AC_MSG_WARN
 >       If this token and others are legitimate, please use m4_pattern_allow.
 >       See the Autoconf documentation.
 > configure.ac:201: error: possibly undefined macro: AC_DEFINE
 > configure.ac:582: error: possibly undefined macro: AC_MSG_NOTICE

 > Even if the message seems unrelated, it's really the lack of pkg.m4
 > from host-pkgconf that causes the issue.

 > Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>

Committed to 2018.02.x and 2018.08.x, thanks.

-- 
Bye, Peter Korsgaard

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

end of thread, other threads:[~2018-11-26 11:46 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-11-15 16:46 [Buildroot] [PATCH] package/openocd: add missing host-pkgconf dependency Thomas Petazzoni
2018-11-16 22:29 ` Peter Korsgaard
2018-11-26 11:46 ` Peter Korsgaard

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox