* [Buildroot] [PATCH/next v2 1/1] package/lzo: switch to cmake
@ 2017-08-20 8:07 Bernd Kuhls
2017-08-21 0:33 ` Ricardo Martincoski
` (2 more replies)
0 siblings, 3 replies; 5+ messages in thread
From: Bernd Kuhls @ 2017-08-20 8:07 UTC (permalink / raw)
To: buildroot
autoreconf for the host package is broken after bumping to 2.10:
http://lists.busybox.net/pipermail/buildroot/2017-August/200634.html
To solve the problem we switch to cmake.
Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
---
v2: patch is for next branch
package/lzo/lzo.mk | 8 +++-----
1 file changed, 3 insertions(+), 5 deletions(-)
diff --git a/package/lzo/lzo.mk b/package/lzo/lzo.mk
index 5b8efdd54..dba965fcd 100644
--- a/package/lzo/lzo.mk
+++ b/package/lzo/lzo.mk
@@ -9,9 +9,7 @@ LZO_SITE = http://www.oberhumer.com/opensource/lzo/download
LZO_LICENSE = GPL-2.0+
LZO_LICENSE_FILES = COPYING
LZO_INSTALL_STAGING = YES
-# Ships a beta libtool version hence our patch doesn't apply.
-# Run autoreconf to regenerate ltmain.sh.
-LZO_AUTORECONF = YES
+LZO_SUPPORTS_IN_SOURCE_BUILD = NO
-$(eval $(autotools-package))
-$(eval $(host-autotools-package))
+$(eval $(cmake-package))
+$(eval $(host-cmake-package))
--
2.11.0
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH/next v2 1/1] package/lzo: switch to cmake
2017-08-20 8:07 [Buildroot] [PATCH/next v2 1/1] package/lzo: switch to cmake Bernd Kuhls
@ 2017-08-21 0:33 ` Ricardo Martincoski
2017-08-21 19:35 ` Arnout Vandecappelle
2017-08-21 21:29 ` Thomas Petazzoni
2 siblings, 0 replies; 5+ messages in thread
From: Ricardo Martincoski @ 2017-08-21 0:33 UTC (permalink / raw)
To: buildroot
Hello,
On Sun, Aug 20, 2017 at 05:07 AM, Bernd Kuhls wrote:
> autoreconf for the host package is broken after bumping to 2.10:
> http://lists.busybox.net/pipermail/buildroot/2017-August/200634.html
>
> To solve the problem we switch to cmake.
>
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
[It fixes the scenarios tested by the test infra in the next branch
https://gitlab.com/RicardoMartincoski/buildroot/-/jobs/29798693 ]
Tested-by: Ricardo Martincoski <ricardo.martincoski@gmail.com>
Regards,
Ricardo
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH/next v2 1/1] package/lzo: switch to cmake
2017-08-20 8:07 [Buildroot] [PATCH/next v2 1/1] package/lzo: switch to cmake Bernd Kuhls
2017-08-21 0:33 ` Ricardo Martincoski
@ 2017-08-21 19:35 ` Arnout Vandecappelle
2017-08-21 21:29 ` Thomas Petazzoni
2017-08-21 21:29 ` Thomas Petazzoni
2 siblings, 1 reply; 5+ messages in thread
From: Arnout Vandecappelle @ 2017-08-21 19:35 UTC (permalink / raw)
To: buildroot
On 20-08-17 10:07, Bernd Kuhls wrote:
> +LZO_SUPPORTS_IN_SOURCE_BUILD = NO
Why doesn't it support in-source build? Best to explain that in the commit message.
Regards,
Arnout
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286500
Essensium/Mind http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH/next v2 1/1] package/lzo: switch to cmake
2017-08-21 19:35 ` Arnout Vandecappelle
@ 2017-08-21 21:29 ` Thomas Petazzoni
0 siblings, 0 replies; 5+ messages in thread
From: Thomas Petazzoni @ 2017-08-21 21:29 UTC (permalink / raw)
To: buildroot
Hello,
On Mon, 21 Aug 2017 21:35:30 +0200, Arnout Vandecappelle wrote:
> On 20-08-17 10:07, Bernd Kuhls wrote:
> > +LZO_SUPPORTS_IN_SOURCE_BUILD = NO
>
> Why doesn't it support in-source build? Best to explain that in the commit message.
It's actually quite common for cmake packages to not support in-tree
build:
package/gnuradio/gnuradio.mk:GNURADIO_SUPPORTS_IN_SOURCE_BUILD = NO
package/gr-osmosdr/gr-osmosdr.mk:GR_OSMOSDR_SUPPORTS_IN_SOURCE_BUILD = NO
package/jasper/jasper.mk:JASPER_SUPPORTS_IN_SOURCE_BUILD = NO
package/lapack/lapack.mk:LAPACK_SUPPORTS_IN_SOURCE_BUILD = NO
package/lensfun/lensfun.mk:LENSFUN_SUPPORTS_IN_SOURCE_BUILD = NO
package/libcodec2/libcodec2.mk:LIBCODEC2_SUPPORTS_IN_SOURCE_BUILD = NO
package/libssh/libssh.mk:LIBSSH_SUPPORTS_IN_SOURCE_BUILD = NO
package/lzo/lzo.mk:LZO_SUPPORTS_IN_SOURCE_BUILD = NO
package/racehound/racehound.mk:RACEHOUND_SUPPORTS_IN_SOURCE_BUILD = NO
package/sysdig/sysdig.mk:SYSDIG_SUPPORTS_IN_SOURCE_BUILD = NO
package/zxing-cpp/zxing-cpp.mk:ZXING_CPP_SUPPORTS_IN_SOURCE_BUILD = NO
and we've never really asked people to explain why. I wouldn't mind a
comment on top of the SUPPORTS_IN_SOURCE_BUILD = NO line in the .mk
file, but I'm not sure I want to make this absolutely necessary.
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH/next v2 1/1] package/lzo: switch to cmake
2017-08-20 8:07 [Buildroot] [PATCH/next v2 1/1] package/lzo: switch to cmake Bernd Kuhls
2017-08-21 0:33 ` Ricardo Martincoski
2017-08-21 19:35 ` Arnout Vandecappelle
@ 2017-08-21 21:29 ` Thomas Petazzoni
2 siblings, 0 replies; 5+ messages in thread
From: Thomas Petazzoni @ 2017-08-21 21:29 UTC (permalink / raw)
To: buildroot
Hello,
On Sun, 20 Aug 2017 10:07:44 +0200, Bernd Kuhls wrote:
> autoreconf for the host package is broken after bumping to 2.10:
> http://lists.busybox.net/pipermail/buildroot/2017-August/200634.html
>
> To solve the problem we switch to cmake.
>
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
> ---
> v2: patch is for next branch
Applied to next, thanks.
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2017-08-21 21:29 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-08-20 8:07 [Buildroot] [PATCH/next v2 1/1] package/lzo: switch to cmake Bernd Kuhls
2017-08-21 0:33 ` Ricardo Martincoski
2017-08-21 19:35 ` Arnout Vandecappelle
2017-08-21 21:29 ` Thomas Petazzoni
2017-08-21 21:29 ` 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.