Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] [autobuild] package/lightning: fix build for static-only
@ 2015-08-13 14:58 Yann E. MORIN
  2015-08-13 19:55 ` Baruch Siach
  2015-08-18  9:20 ` Thomas Petazzoni
  0 siblings, 2 replies; 4+ messages in thread
From: Yann E. MORIN @ 2015-08-13 14:58 UTC (permalink / raw)
  To: buildroot

Fixes:
    http://autobuild.buildroot.org/results/019/019b6f2ac034766a4266e20e5dcc229bec4a7222/
    http://autobuild.buildroot.org/results/55d/55d30db7212863d9f898bb929f0c75d529036bad/
    http://autobuild.buildroot.org/results/459/459ab12bc3eeb493f39efa2450bc829a351ee3a3/
    ...

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Cc: Paul Cercueil <paul@crapouillou.net>
---
 package/lightning/0003-no-check-no-doc.patch | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)
 create mode 100644 package/lightning/0003-no-check-no-doc.patch

diff --git a/package/lightning/0003-no-check-no-doc.patch b/package/lightning/0003-no-check-no-doc.patch
new file mode 100644
index 0000000..91a1684
--- /dev/null
+++ b/package/lightning/0003-no-check-no-doc.patch
@@ -0,0 +1,21 @@
+Makefile: do not build checks or docs
+
+checks require a toolchain with shared libs, so they FTBFS when the
+build is static-only.
+
+Docs we don't care about in Buildroot.
+
+Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
+
+diff -durN lightning-2.0.5.orig/Makefile.am lightning-2.0.5/Makefile.am
+--- lightning-2.0.5.orig/Makefile.am	2013-09-24 05:37:46.000000000 +0200
++++ lightning-2.0.5/Makefile.am	2015-08-13 16:27:20.626730913 +0200
+@@ -17,8 +17,6 @@
+ ACLOCAL_AMFLAGS = -I m4
+ 
+ SUBDIRS =		\
+-	check		\
+-	doc		\
+ 	include		\
+ 	lib
+ 
-- 
1.9.1

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

* [Buildroot] [PATCH] [autobuild] package/lightning: fix build for static-only
  2015-08-13 14:58 [Buildroot] [PATCH] [autobuild] package/lightning: fix build for static-only Yann E. MORIN
@ 2015-08-13 19:55 ` Baruch Siach
  2015-08-18  9:20 ` Thomas Petazzoni
  1 sibling, 0 replies; 4+ messages in thread
From: Baruch Siach @ 2015-08-13 19:55 UTC (permalink / raw)
  To: buildroot

Hi Yann,

On Thu, Aug 13, 2015 at 04:58:28PM +0200, Yann E. MORIN wrote:
> Fixes:
>     http://autobuild.buildroot.org/results/019/019b6f2ac034766a4266e20e5dcc229bec4a7222/
>     http://autobuild.buildroot.org/results/55d/55d30db7212863d9f898bb929f0c75d529036bad/
>     http://autobuild.buildroot.org/results/459/459ab12bc3eeb493f39efa2450bc829a351ee3a3/
>     ...
> 
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> Cc: Paul Cercueil <paul@crapouillou.net>
> ---
>  package/lightning/0003-no-check-no-doc.patch | 21 +++++++++++++++++++++
>  1 file changed, 21 insertions(+)
>  create mode 100644 package/lightning/0003-no-check-no-doc.patch
> 
> diff --git a/package/lightning/0003-no-check-no-doc.patch b/package/lightning/0003-no-check-no-doc.patch
> new file mode 100644
> index 0000000..91a1684
> --- /dev/null
> +++ b/package/lightning/0003-no-check-no-doc.patch
> @@ -0,0 +1,21 @@
> +Makefile: do not build checks or docs
> +
> +checks require a toolchain with shared libs, so they FTBFS when the
> +build is static-only.
> +
> +Docs we don't care about in Buildroot.

In http://patchwork.ozlabs.org/patch/505871/ I opted for disabling static 
build support entirely. I see the value in preserving static build ability, 
but this patch is not upstreamable in its current form. I'm fine both ways.

> +
> +Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> +
> +diff -durN lightning-2.0.5.orig/Makefile.am lightning-2.0.5/Makefile.am
> +--- lightning-2.0.5.orig/Makefile.am	2013-09-24 05:37:46.000000000 +0200
> ++++ lightning-2.0.5/Makefile.am	2015-08-13 16:27:20.626730913 +0200
> +@@ -17,8 +17,6 @@
> + ACLOCAL_AMFLAGS = -I m4
> + 
> + SUBDIRS =		\
> +-	check		\
> +-	doc		\
> + 	include		\
> + 	lib

baruch

-- 
     http://baruch.siach.name/blog/                  ~. .~   Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
   - baruch at tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il -

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

* [Buildroot] [PATCH] [autobuild] package/lightning: fix build for static-only
  2015-08-13 14:58 [Buildroot] [PATCH] [autobuild] package/lightning: fix build for static-only Yann E. MORIN
  2015-08-13 19:55 ` Baruch Siach
@ 2015-08-18  9:20 ` Thomas Petazzoni
  2015-08-18  9:36   ` Yann E. MORIN
  1 sibling, 1 reply; 4+ messages in thread
From: Thomas Petazzoni @ 2015-08-18  9:20 UTC (permalink / raw)
  To: buildroot

Dear Yann E. MORIN,

On Thu, 13 Aug 2015 16:58:28 +0200, Yann E. MORIN wrote:
> Fixes:
>     http://autobuild.buildroot.org/results/019/019b6f2ac034766a4266e20e5dcc229bec4a7222/
>     http://autobuild.buildroot.org/results/55d/55d30db7212863d9f898bb929f0c75d529036bad/
>     http://autobuild.buildroot.org/results/459/459ab12bc3eeb493f39efa2450bc829a351ee3a3/
>     ...
> 
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> Cc: Paul Cercueil <paul@crapouillou.net>
> ---
>  package/lightning/0003-no-check-no-doc.patch | 21 +++++++++++++++++++++
>  1 file changed, 21 insertions(+)
>  create mode 100644 package/lightning/0003-no-check-no-doc.patch

I've applied Baruch proposal of making the package unavailable for
static builds. If you are really interested in having lightning
available for static builds, implementing an upstream-acceptable
solution would be nice.

Thanks!

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

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

* [Buildroot] [PATCH] [autobuild] package/lightning: fix build for static-only
  2015-08-18  9:20 ` Thomas Petazzoni
@ 2015-08-18  9:36   ` Yann E. MORIN
  0 siblings, 0 replies; 4+ messages in thread
From: Yann E. MORIN @ 2015-08-18  9:36 UTC (permalink / raw)
  To: buildroot

Thomas, All,

On 2015-08-18 11:20 +0200, Thomas Petazzoni spake thusly:
> On Thu, 13 Aug 2015 16:58:28 +0200, Yann E. MORIN wrote:
> > Fixes:
> >     http://autobuild.buildroot.org/results/019/019b6f2ac034766a4266e20e5dcc229bec4a7222/
> >     http://autobuild.buildroot.org/results/55d/55d30db7212863d9f898bb929f0c75d529036bad/
> >     http://autobuild.buildroot.org/results/459/459ab12bc3eeb493f39efa2450bc829a351ee3a3/
> >     ...
> > 
> > Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> > Cc: Paul Cercueil <paul@crapouillou.net>
> > ---
> >  package/lightning/0003-no-check-no-doc.patch | 21 +++++++++++++++++++++
> >  1 file changed, 21 insertions(+)
> >  create mode 100644 package/lightning/0003-no-check-no-doc.patch
> 
> I've applied Baruch proposal of making the package unavailable for
> static builds. If you are really interested in having lightning
> available for static builds, implementing an upstream-acceptable
> solution would be nice.

No, I don't care about lightning being staticaly buildable.

<humor>
For once, I decided to go the extra length of actually fixing the stuff,
not just do my usual lazy we-dont-care-static-just-disable-it type of
fix. Just to get this kind of cautery actually applied.
</humor>

;-)

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

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

end of thread, other threads:[~2015-08-18  9:36 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-08-13 14:58 [Buildroot] [PATCH] [autobuild] package/lightning: fix build for static-only Yann E. MORIN
2015-08-13 19:55 ` Baruch Siach
2015-08-18  9:20 ` Thomas Petazzoni
2015-08-18  9:36   ` Yann E. MORIN

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