GNOME Bugzilla – Bug 336605
Cannot compile metacity with --disable-xsync specified
Last modified: 2006-04-15 22:58:40 UTC
Please describe the problem: I tried to compile Metacity 2.14.1 with the --disable-xsync flag set, but unfortunately the compile fails with the following error: edge-resistance.c: In function ‘meta_window_edge_resistance_for_move’: edge-resistance.c:1116: error: ‘struct _MetaDisplay’ has no member named ‘grab_last_user_action_was_snap’ edge-resistance.c: In function ‘meta_window_edge_resistance_for_resize’: edge-resistance.c:1205: error: ‘struct _MetaDisplay’ has no member named ‘grab_last_user_action_was_snap’ make[4]: *** [edge-resistance.o] Error 1 make[4]: Leaving directory `/home/retro/newmet/metacity-2.14.1/src' make[3]: *** [all-recursive] Error 1 make[3]: Leaving directory `/home/retro/newmet/metacity-2.14.1/src' make[2]: *** [all] Error 2 make[2]: Leaving directory `/home/retro/newmet/metacity-2.14.1/src' make[1]: *** [all-recursive] Error 1 make[1]: Leaving directory `/home/retro/newmet/metacity-2.14.1' make: *** [all] Error 2 Steps to reproduce: 1. ./configure --disable-xsync 2. make Actual results: compile fails Expected results: Does this happen every time? yes Other information: I am building this on Ubuntu Dapper, and the build completes as expected without this flag set. The Metacity sources are from the Gnome servers, not the Ubuntu patched version.
This should be a relatively easy fix for someone looking to get involved in Gnome -- it just looks like I didn't pay attention to where all the #ifdef's were when I added the variable to display.h. It probably just needs to be moved. There may be a couple other vars that need to be moved to get it to compile, but it'll probably be the same kind of thing.
Created attachment 63252 [details] [review] Move grab_last_user_action_was_snap to fix compile No idea if this variable is in the correct place for readability, but it does at least fix the compile. :) Andy.
Looks good, thanks. Do you have a CVS account? If not, can you post a ChangeLog entry in a comment here and I'll commit it for you?