* [Buildroot] [PATCH 1/1] package/x11r7/xserver_xorg-server: libunwind support is broken on mips64 @ 2014-11-29 22:10 Bernd Kuhls 2014-11-30 20:40 ` Peter Korsgaard 0 siblings, 1 reply; 3+ messages in thread From: Bernd Kuhls @ 2014-11-29 22:10 UTC (permalink / raw) To: buildroot Fixes http://autobuild.buildroot.net/results/e8f/e8f68163802f31359cee15fa3ca52547e06d1f27/ http://autobuild.buildroot.net/results/994/9947222da8682a358f81f8ac924c12596fd09ee8/ http://autobuild.buildroot.net/results/df9/df92d9b682976b77d9e1e4c39afca52b7512d847/ Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de> --- package/x11r7/xserver_xorg-server/xserver_xorg-server.mk | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/package/x11r7/xserver_xorg-server/xserver_xorg-server.mk b/package/x11r7/xserver_xorg-server/xserver_xorg-server.mk index 4db8b09..cae06e6 100644 --- a/package/x11r7/xserver_xorg-server/xserver_xorg-server.mk +++ b/package/x11r7/xserver_xorg-server/xserver_xorg-server.mk @@ -144,7 +144,9 @@ ifeq ($(BR2_PACKAGE_FREETYPE),y) XSERVER_XORG_SERVER_DEPENDENCIES += freetype endif -ifeq ($(BR2_PACKAGE_LIBUNWIND),y) +# libunwind support is broken on mips64 +# https://www.libreoffice.org/bugzilla/show_bug.cgi?id=79939 +ifeq ($(BR2_PACKAGE_LIBUNWIND)$(BR2_mips64el),y) XSERVER_XORG_SERVER_DEPENDENCIES += libunwind else XSERVER_XORG_SERVER_CONF_OPTS += --disable-libunwind -- 1.7.10.4 ^ permalink raw reply related [flat|nested] 3+ messages in thread
* [Buildroot] [PATCH 1/1] package/x11r7/xserver_xorg-server: libunwind support is broken on mips64 2014-11-29 22:10 [Buildroot] [PATCH 1/1] package/x11r7/xserver_xorg-server: libunwind support is broken on mips64 Bernd Kuhls @ 2014-11-30 20:40 ` Peter Korsgaard 2014-12-01 10:23 ` Vicente Olivert Riera 0 siblings, 1 reply; 3+ messages in thread From: Peter Korsgaard @ 2014-11-30 20:40 UTC (permalink / raw) To: buildroot >>>>> "Bernd" == Bernd Kuhls <bernd.kuhls@t-online.de> writes: > Fixes > http://autobuild.buildroot.net/results/e8f/e8f68163802f31359cee15fa3ca52547e06d1f27/ > http://autobuild.buildroot.net/results/994/9947222da8682a358f81f8ac924c12596fd09ee8/ > http://autobuild.buildroot.net/results/df9/df92d9b682976b77d9e1e4c39afca52b7512d847/ > Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de> > --- > package/x11r7/xserver_xorg-server/xserver_xorg-server.mk | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > diff --git a/package/x11r7/xserver_xorg-server/xserver_xorg-server.mk > b/package/x11r7/xserver_xorg-server/xserver_xorg-server.mk > index 4db8b09..cae06e6 100644 > --- a/package/x11r7/xserver_xorg-server/xserver_xorg-server.mk > +++ b/package/x11r7/xserver_xorg-server/xserver_xorg-server.mk > @@ -144,7 +144,9 @@ ifeq ($(BR2_PACKAGE_FREETYPE),y) > XSERVER_XORG_SERVER_DEPENDENCIES += freetype > endif > -ifeq ($(BR2_PACKAGE_LIBUNWIND),y) > +# libunwind support is broken on mips64 > +# https://www.libreoffice.org/bugzilla/show_bug.cgi?id=79939 This doesn't have anything to do with libreoffice. That URL is apparently an alias for bugs.freedesktop.org (the official bugtracker for xorg), so I've used that as URL instead. > +ifeq ($(BR2_PACKAGE_LIBUNWIND)$(BR2_mips64el),y) > XSERVER_XORG_SERVER_DEPENDENCIES += libunwind That's not good as builds on mips64el WITHOUT libunwind now ends up pulling in libunwind (and breaking the build). We need to match on LIBUNWIND and !mips64el, so there needs so be a seperator in between, like: ifeq ($(BR2_PACKAGE_LIBUNWIND)-$(BR2_mips64el),y-) Even if it looks a bit odd. Committed with those fixes, thanks. -- Bye, Peter Korsgaard ^ permalink raw reply [flat|nested] 3+ messages in thread
* [Buildroot] [PATCH 1/1] package/x11r7/xserver_xorg-server: libunwind support is broken on mips64 2014-11-30 20:40 ` Peter Korsgaard @ 2014-12-01 10:23 ` Vicente Olivert Riera 0 siblings, 0 replies; 3+ messages in thread From: Vicente Olivert Riera @ 2014-12-01 10:23 UTC (permalink / raw) To: buildroot Dear Bernd Kuhls, Peter Korsgaard, On 11/30/2014 08:40 PM, Peter Korsgaard wrote: >>>>>> "Bernd" == Bernd Kuhls <bernd.kuhls@t-online.de> writes: > > > Fixes > > http://autobuild.buildroot.net/results/e8f/e8f68163802f31359cee15fa3ca52547e06d1f27/ > > http://autobuild.buildroot.net/results/994/9947222da8682a358f81f8ac924c12596fd09ee8/ > > http://autobuild.buildroot.net/results/df9/df92d9b682976b77d9e1e4c39afca52b7512d847/ > > > Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de> > > --- > > package/x11r7/xserver_xorg-server/xserver_xorg-server.mk | 4 +++- > > 1 file changed, 3 insertions(+), 1 deletion(-) > > > diff --git a/package/x11r7/xserver_xorg-server/xserver_xorg-server.mk > > b/package/x11r7/xserver_xorg-server/xserver_xorg-server.mk > > index 4db8b09..cae06e6 100644 > > --- a/package/x11r7/xserver_xorg-server/xserver_xorg-server.mk > > +++ b/package/x11r7/xserver_xorg-server/xserver_xorg-server.mk > > @@ -144,7 +144,9 @@ ifeq ($(BR2_PACKAGE_FREETYPE),y) > > XSERVER_XORG_SERVER_DEPENDENCIES += freetype > > endif > > -ifeq ($(BR2_PACKAGE_LIBUNWIND),y) > > +# libunwind support is broken on mips64 > > +# https://www.libreoffice.org/bugzilla/show_bug.cgi?id=79939 > > This doesn't have anything to do with libreoffice. That URL is > apparently an alias for bugs.freedesktop.org (the official bugtracker > for xorg), so I've used that as URL instead. > > > > +ifeq ($(BR2_PACKAGE_LIBUNWIND)$(BR2_mips64el),y) > > XSERVER_XORG_SERVER_DEPENDENCIES += libunwind > > That's not good as builds on mips64el WITHOUT libunwind now ends up > pulling in libunwind (and breaking the build). > > We need to match on LIBUNWIND and !mips64el, so there needs so be a > seperator in between, like: > > ifeq ($(BR2_PACKAGE_LIBUNWIND)-$(BR2_mips64el),y-) > > Even if it looks a bit odd. > > Committed with those fixes, thanks. > as you may see in the bug report, I'm the one who reported the bug. https://bugs.freedesktop.org/show_bug.cgi?id=79939 Xorg with libunwind support is broken for MIPS64 n32, but MIPS64 n64 is ok, so disabing libunwind support for mips64el is wrong. Also, I'm trying to fix the problem in libunwind. I reported the bug and sent a patch which we are discussing right now: https://savannah.nongnu.org/bugs/?43695 Cheers, -- Vicente Olivert Riera Graduate Software Engineer, MIPS Platforms Imagination Technologies Limited t: +44 (0)113 2429814 www.imgtec.com ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-12-01 10:23 UTC | newest] Thread overview: 3+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2014-11-29 22:10 [Buildroot] [PATCH 1/1] package/x11r7/xserver_xorg-server: libunwind support is broken on mips64 Bernd Kuhls 2014-11-30 20:40 ` Peter Korsgaard 2014-12-01 10:23 ` Vicente Olivert Riera
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox