Skip to contentSkip to conversation

matchbox-lib: Append -lX11 to the linker

PATCH New Markos Chandras 3 participants 8 messages Oct 30, 2012 – Nov 30, 2012

lore
Share & tools
lore

Markos Chandras

author
#0 lore raw

PATCH 1/1 matchbox-lib: Append -lX11 to the linker

From: Markos Chandras <[email protected]>

Fixes building with binutils-2.22
http://autobuild.buildroot.net/results/b4b26dd9c7d7bebe45638ba125fb49a5f5d0cc14/build-end.log

Signed-off-by Markos Chandras <[email protected]>
---
 package/matchbox/matchbox-lib/matchbox-lib.mk |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)
package/matchbox/matchbox-lib/matchbox-lib.mk +1 -0
index de11dfc..339f512 100644
@@ -10,6 +10,7 @@ MATCHBOX_LIB_SITE = http://matchbox-project.org/sources/libmatchbox/$(MATCHBOX_L
 MATCHBOX_LIB_INSTALL_STAGING = YES
 MATCHBOX_LIB_DEPENDENCIES = host-pkgconf expat xlib_libXext
 MATCHBOX_LIB_CONF_OPT = --enable-expat --disable-doxygen-docs
+MATCHBOX_LIB_CONF_ENV = LIBS="-lX11"
 
 define MATCHBOX_LIB_POST_INSTALL_FIXES
  $(SED) 's:-I[^$$].*/usr/include/freetype2:-I/usr/include/freetype2:' $(STAGING_DIR)/usr/lib/pkgconfig/libmb.pc
Signature · 1 line
1.7.1
#1 lore raw

PATCH 1/1 matchbox-lib: Append -lX11 to the linker

Dear Markos Chandras,

Quoted 15 lines · Markos Chandras

From: Markos Chandras <[email protected]>

Fixes building with binutils-2.22
http://autobuild.buildroot.net/results/b4b26dd9c7d7bebe45638ba125fb49a5f5d0cc14/build-end.log

Signed-off-by Markos Chandras <[email protected]>
---
 package/matchbox/matchbox-lib/matchbox-lib.mk |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)
package/matchbox/matchbox-lib/matchbox-lib.mk +0 -0
index de11dfc..339f512 100644

package/matchbox/matchbox-lib/matchbox-lib.mk MATCHBOX_LIB_SITE = http://matchbox-project.org/sources/libmatchbox/$(MATCHBOX_L · line 13

Quoted 4 lines
package/matchbox/matchbox-lib/matchbox-lib.mk +1 -0
@@ -10,6 +10,7 @@ MATCHBOX_LIB_SITE = http://matchbox-project.org/sources/libmatchbox/$(MATCHBOX_L
 MATCHBOX_LIB_INSTALL_STAGING = YES
 MATCHBOX_LIB_DEPENDENCIES = host-pkgconf expat xlib_libXext
 MATCHBOX_LIB_CONF_OPT = --enable-expat --disable-doxygen-docs
+MATCHBOX_LIB_CONF_ENV = LIBS="-lX11"

This works but is a hack. Has the problem been fixed in newer upstream releases? If not, would it be possible to cook a patch to send upstream?

Ditto for the matchbox-desktop patch.

Thanks,

Thomas

Signature · 4 lines
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
author
#2 lore raw

PATCH 1/1 matchbox-lib: Append -lX11 to the linker

Quoted 28 lines · Thomas Petazzoni

Dear Markos Chandras,

Quoted 15 lines · Markos Chandras

From: Markos Chandras <[email protected]>

Fixes building with binutils-2.22
http://autobuild.buildroot.net/results/b4b26dd9c7d7bebe45638ba125fb49a5f5d0cc14/build-end.log

Signed-off-by Markos Chandras <[email protected]>
---
 package/matchbox/matchbox-lib/matchbox-lib.mk |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)
package/matchbox/matchbox-lib/matchbox-lib.mk +0 -0
index de11dfc..339f512 100644

package/matchbox/matchbox-lib/matchbox-lib.mk MATCHBOX_LIB_SITE = http://matchbox-project.org/sources/libmatchbox/$(MATCHBOX_L · line 13

Quoted 4 lines
package/matchbox/matchbox-lib/matchbox-lib.mk +1 -0
@@ -10,6 +10,7 @@ MATCHBOX_LIB_SITE = http://matchbox-project.org/sources/libmatchbox/$(MATCHBOX_L
 MATCHBOX_LIB_INSTALL_STAGING = YES
 MATCHBOX_LIB_DEPENDENCIES = host-pkgconf expat xlib_libXext
 MATCHBOX_LIB_CONF_OPT = --enable-expat --disable-doxygen-docs
+MATCHBOX_LIB_CONF_ENV = LIBS="-lX11"

This works but is a hack. Has the problem been fixed in newer upstream releases? If not, would it be possible to cook a patch to send upstream?

Ditto for the matchbox-desktop patch.

Quoted 8 lines

Thanks,

Thomas
--
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

Hi Thomas,

There is no matchbox upstream. Latest release was in 2007 and the latest post in the mailing list was in 2007 as well. Their svn repo seems dead too

http://svn.o-hand.com/view/matchbox/trunk/

Signature · 2 lines
Regards,
Markos
#3 lore raw

PATCH 1/1 matchbox-lib: Append -lX11 to the linker

Dear Markos Chandras,

Quoted 5 lines · Markos Chandras

There is no matchbox upstream. Latest release was in 2007 and the latest post in the mailing list was in 2007 as well. Their svn repo seems dead too

http://svn.o-hand.com/view/matchbox/trunk/

Hum, right. Do we want to keep those packages in Buildroot then? :-)

Thomas

Signature · 4 lines
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
author
#4 lore raw

PATCH 1/1 matchbox-lib: Append -lX11 to the linker

Quoted 4 lines · Thomas Petazzoni

Dear Markos Chandras,

On Tue, 30 Oct 2012 13:21:22 +0000, Markos Chandras wrote:

Quoted 8 lines
Quoted 5 lines

There is no matchbox upstream. Latest release was in 2007 and the latest post in the mailing list was in 2007 as well. Their svn repo seems dead too

http://svn.o-hand.com/view/matchbox/trunk/

Hum, right. Do we want to keep those packages in Buildroot then? :-)

Well at least they build now :) There are other alternatives of course (openbox, fluxbox) but I think matchbox
targets different devices (like PDAs or devices with small screens). My personal opinion is to keep them
around until we reach a point where patching them becomes a non-trivial process.

Signature · 2 lines
Regards,
Markos
author
#5 lore raw

PATCH 1/1 matchbox-lib: Append -lX11 to the linker

Quoted 21 lines · Markos Chandras
Quoted 4 lines · Thomas Petazzoni

Dear Markos Chandras,

On Tue, 30 Oct 2012 13:21:22 +0000, Markos Chandras wrote:

Quoted 8 lines
Quoted 5 lines

There is no matchbox upstream. Latest release was in 2007 and the latest post in the mailing list was in 2007 as well. Their svn repo seems dead too

http://svn.o-hand.com/view/matchbox/trunk/

Hum, right. Do we want to keep those packages in Buildroot then? :-)

Well at least they build now :) There are other alternatives of course (openbox, fluxbox) but I think matchbox
targets different devices (like PDAs or devices with small screens). My personal opinion is to keep them
around until we reach a point where patching them becomes a non-trivial process.

Quoted 3 lines

--
Regards,
Markos

Hi,

Ping? :)

Signature · 2 lines
Regards,
Markos
#6 lore raw

PATCH 1/1 matchbox-lib: Append -lX11 to the linker

In reply to Markos Chandras

Peter,

Could you apply this one to 2012.11 ? Thanks!

Quoted 6 lines · Markos Chandras

From: Markos Chandras <[email protected]>

Fixes building with binutils-2.22
http://autobuild.buildroot.net/results/b4b26dd9c7d7bebe45638ba125fb49a5f5d0cc14/build-end.log

Signed-off-by Markos Chandras <[email protected]>
Acked-by Thomas Petazzoni <[email protected]>
Tested-by Thomas Petazzoni <[email protected]>

Thomas

Signature · 4 lines
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
#7 lore raw

PATCH 1/1 matchbox-lib: Append -lX11 to the linker

In reply to Markos Chandras
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line

"Markos" == Markos Chandras <[email protected]> writes:

Markos> From: Markos Chandras <[email protected]>
Markos> Fixes building with binutils-2.22
Markos> http://autobuild.buildroot.net/results/b4b26dd9c7d7bebe45638ba125fb49a5f5d0cc14/build-end.log

Committed, thanks.

Signature · 1 line
Bye, Peter Korsgaard