Opened 3 years ago

Closed 3 years ago

#11039 closed defect (fixed)

Fix mutter build on i686

Reported by: Douglas R. Reno Owned by: Douglas R. Reno
Priority: normal Milestone: 8.4
Component: BOOK Version: SVN
Severity: normal Keywords:
Cc:

Description

I received a report over email recently that mutter was broken on i686. I now have a moderately-fast (SBU=120) dual-core i686 system (Intel Core Duo w/ 4GB of RAM), and I'm going to fix this.

See http://lists.linuxfromscratch.org/pipermail/blfs-dev/2018-August/034679.html

Change History (11)

comment:1 by Douglas R. Reno, 3 years ago

Owner: changed from blfs-book to Douglas R. Reno
Status: newassigned

comment:2 by Douglas R. Reno, 3 years ago

Milestone: 8.38.4

Moving to 8.4

comment:3 by Douglas R. Reno, 3 years ago

I will test this with GNOME 3.30. If it turns out that there is still a problem, I'll also modify the errata for 8.3.

comment:4 by Xi Ruoyao, 3 years ago

Still occurs with Mutter 3.30.

comment:5 by Douglas R. Reno, 3 years ago

Hey Xi,

Just wanted to say thank you for confirming!

comment:6 by Xi Ruoyao, 3 years ago

Adding configure option --enable-compile-warnings=minimum can workaround this (tested with mutter-3.30.0).

comment:7 by Douglas R. Reno, 3 years ago

Minor changes in 3.30.5:
* Fix a bunch of compiler warnings that could prevent building (Ernestas Kulik)
* Fix location entry icon not reacting to clicks (Ernestas Kulik)


Might not even have to use that option!

comment:8 by Douglas R. Reno, 3 years ago

I should have this done by the end of the week at the latest, with my Dell Latitude D820's current pace.

Note that large package builds are painful on this machine, so Qt5/WebKitGTK/rustc will take much longer than on my other systems (even my PPC is faster sometimes).

comment:9 by Douglas R. Reno, 3 years ago

  CC       backends/meta-input-settings.lo
backends/meta-input-settings.c: In function ‘lookup_tool_settings’:
backends/meta-input-settings.c:1287:72: error: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 2 has type ‘guint64’ {aka ‘long long unsigned int’} [-Werror=format=]
       path = g_strdup_printf ("/org/gnome/desktop/peripherals/stylus/%lx/", serial);
                                                                      ~~^    ~~~~~~
                                                                      %llx
cc1: all warnings being treated as errors
make[4]: *** [Makefile:2839: backends/meta-input-settings.lo] Error 1
make[4]: Leaving directory '/sources/mutter-3.30.2/mutter-3.30.2/src'
make[3]: *** [Makefile:3016: all-recursive] Error 1
make[3]: Leaving directory '/sources/mutter-3.30.2/mutter-3.30.2/src'
make[2]: *** [Makefile:1856: all] Error 2
make[2]: Leaving directory '/sources/mutter-3.30.2/mutter-3.30.2/src'
make[1]: *** [Makefile:492: all-recursive] Error 1
make[1]: Leaving directory '/sources/mutter-3.30.2/mutter-3.30.2'
make: *** [Makefile:422: all] Error 2

Time to go give that flag a shot.

comment:10 by Douglas R. Reno, 3 years ago

That switch worked! Thanks again, Xi.

comment:11 by Douglas R. Reno, 3 years ago

Resolution: fixed
Status: assignedclosed

Fixed at r21018

Note: See TracTickets for help on using tickets.