# HG changeset patch # User Stu Tomlinson # Date 1130445823 0 # Node ID f8cd0675375519ffbfb9bb157f8573eb1ab4cd0d # Parent 83e2fa10df1c474fe28a0496bd52c0f8087f5949 [gaim-migrate @ 14153] Make "make distcheck" work, which also makes the perl plugin loader loadable when installed to a non-standard prefix (it still can't find Gaim.pm though) Add some extra warning CFLAGS if a) --enable-debug was used and b) your compiler supports them. Plug a couple of leaks The gaimrc plugin's GtkTreeView expander_size setting now works (but doesn't seem to instant-apply - should it?) staticify some functions that don't need to be exposed probably something else I forgot about committer: Tailor Script diff -r 83e2fa10df1c -r f8cd06753755 ChangeLog --- a/ChangeLog Thu Oct 27 20:22:27 2005 +0000 +++ b/ChangeLog Thu Oct 27 20:43:43 2005 +0000 @@ -54,6 +54,7 @@ (Peter McCurdy) * Control-Shift-Tab will reverse cycle through the conversation tabs (James Vega) + * Mono plugin loader (Eoin Coffey) Bug fixes: * People using input methods can now use Enter again. diff -r 83e2fa10df1c -r f8cd06753755 Makefile.am --- a/Makefile.am Thu Oct 27 20:22:27 2005 +0000 +++ b/Makefile.am Thu Oct 27 20:43:43 2005 +0000 @@ -48,3 +48,9 @@ @echo "doxygen was not found during configure. Aborting." @echo; endif + +# perl's MakeMaker uninstall foo doesn't work well with DESTDIR set, which +# breaks "make distcheck" unless we ignore perl things + +distuninstallcheck_listfiles = \ + find . -type f -print | grep -v perl diff -r 83e2fa10df1c -r f8cd06753755 configure.ac --- a/configure.ac Thu Oct 27 20:22:27 2005 +0000 +++ b/configure.ac Thu Oct 27 20:43:43 2005 +0000 @@ -239,7 +239,7 @@ AM_CONDITIONAL(STATIC_YAHOO, test "x$static_yahoo" = "xyes") AM_CONDITIONAL(STATIC_ZEPHYR, test "x$static_zephyr" = "xyes") AC_SUBST(STATIC_LINK_LIBS) -AC_DEFINE_UNQUOTED(STATIC_PROTO_INIT, $extern_init void static_proto_init() { $load_proto }, +AC_DEFINE_UNQUOTED(STATIC_PROTO_INIT, $extern_init static void static_proto_init() { $load_proto }, [Loads static protocol plugin module initialization functions.]) AC_ARG_WITH(dynamic_prpls, [AC_HELP_STRING([--with-dynamic-prpls], [specify which protocols to build dynamically])], [DYNAMIC_PRPLS=`echo $withval | $sedpath 's/,/ /g'`]) @@ -307,6 +307,19 @@ if test "$enable_debug" = yes ; then AC_DEFINE(DEBUG, 1, [Define if debugging is enabled.]) + for newflag in "-Werror-implicit-function-declaration" "-Wdeclaration-after-statement"; do + orig_CFLAGS="$CFLAGS" + CFLAGS="$CFLAGS $newflag" + AC_MSG_CHECKING(for $newflag option to gcc) + AC_TRY_COMPILE([], [ + void main() {}; + ], [ + AC_MSG_RESULT(yes) + ], [ + AC_MSG_RESULT(no) + CFLAGS="$orig_CFLAGS" + ]) + done fi if test "x$enable_deprecated" = no; then diff -r 83e2fa10df1c -r f8cd06753755 plugins/Makefile.am --- a/plugins/Makefile.am Thu Oct 27 20:22:27 2005 +0000 +++ b/plugins/Makefile.am Thu Oct 27 20:43:43 2005 +0000 @@ -27,12 +27,12 @@ docklet \ $(GEVOLUTION_DIR) \ gestures \ + $(MONO_DIR) \ + $(MUSICMESSAGING_DIR) \ $(PERL_DIR) \ ssl \ $(TCL_DIR) \ - ticker \ - $(MUSICMESSAGING_DIR) \ - $(MONO_DIR) + ticker plugindir = $(libdir)/gaim diff -r 83e2fa10df1c -r f8cd06753755 plugins/gaimrc.c --- a/plugins/gaimrc.c Thu Oct 27 20:22:27 2005 +0000 +++ b/plugins/gaimrc.c Thu Oct 27 20:43:43 2005 +0000 @@ -110,7 +110,7 @@ if (gaim_prefs_get_bool(widget_size_prefs_set[i])) { prefbase = g_path_get_basename(widget_size_prefs[i]); g_string_append_printf(style_string, - "%s = \"%d\"\n", prefbase, + "%s = %d\n", prefbase, gaim_prefs_get_int(widget_size_prefs[i])); g_free(prefbase); } @@ -166,6 +166,7 @@ if (value) gaimrc_make_changes(); g_string_free(style_string, TRUE); + g_free(prefbase); return; } else { diff -r 83e2fa10df1c -r f8cd06753755 plugins/perl/Makefile.am --- a/plugins/perl/Makefile.am Thu Oct 27 20:22:27 2005 +0000 +++ b/plugins/perl/Makefile.am Thu Oct 27 20:43:43 2005 +0000 @@ -2,10 +2,10 @@ perl_dirs = common -plugin_LTLIBRARIES = perl.la +plugin_LTLIBRARIES = perl.la libgaimperl.la perl_la_LDFLAGS = -module -avoid-version $(GLIB_LIBS) $(PERL_LIBS) -perl_la_LIBADD = $(PERL_LIBS) -L. -lgaimperl +perl_la_LIBADD = $(PERL_LIBS) -L.libs -lgaimperl perl_la_SOURCES = \ perl.c \ perl-common.c \ @@ -16,10 +16,10 @@ perl_la_DEPENDENCIES = \ .libs/libperl_orig.a \ .libs/DynaLoader.a \ - libgaimperl.so + libgaimperl.la -libgaimperl.so: - $(CC) -shared -fPIC -olibgaimperl.so libgaimperl.c +libgaimperl_la_LDFLAGS = -module -avoid-version $(GLIB_LIBS) +libgaimperl_la_SOURCES = libgaimperl.c .libs/libperl_orig.a: @mkdir -p .libs @@ -114,9 +114,9 @@ $(MAKE) install; \ cd ..; \ done - cp libgaimperl.so $(libdir) # Evil Hack (TM) +# ... which doesn't work with DESTDIR installs. FIXME? uninstall-local: @for dir in $(perl_dirs); do \ cd $$dir && \ diff -r 83e2fa10df1c -r f8cd06753755 plugins/perl/libgaimperl.c --- a/plugins/perl/libgaimperl.c Thu Oct 27 20:22:27 2005 +0000 +++ b/plugins/perl/libgaimperl.c Thu Oct 27 20:43:43 2005 +0000 @@ -1,3 +1,4 @@ +#include void __attribute__ ((constructor)) my_init(void) { /* Very evil hack...puts perl.so's symbols in the global table */ /* but does not create a circular dependancy because g_module_open */ diff -r 83e2fa10df1c -r f8cd06753755 src/gtkimhtmltoolbar.c --- a/src/gtkimhtmltoolbar.c Thu Oct 27 20:22:27 2005 +0000 +++ b/src/gtkimhtmltoolbar.c Thu Oct 27 20:43:43 2005 +0000 @@ -579,7 +579,7 @@ struct smiley_button_list *next; }; -struct smiley_button_list * +static struct smiley_button_list * sort_smileys(struct smiley_button_list *ls, GtkIMHtmlToolbar *toolbar, int *width, char *filename, char *face) { GtkWidget *image; diff -r 83e2fa10df1c -r f8cd06753755 src/protocols/msn/notification.c --- a/src/protocols/msn/notification.c Thu Oct 27 20:22:27 2005 +0000 +++ b/src/protocols/msn/notification.c Thu Oct 27 20:43:43 2005 +0000 @@ -523,9 +523,13 @@ reason = g_strdup_printf(_("%s is not a valid passport account."), passport); } + else if (error == 500) + { + reason = g_strdup(_("Service Temporarily Unavailable.")); + } else { - reason = g_strdup_printf(_("Unknown error.")); + reason = g_strdup(_("Unknown error.")); } } diff -r 83e2fa10df1c -r f8cd06753755 src/protocols/sametime/Makefile.am --- a/src/protocols/sametime/Makefile.am Thu Oct 27 20:22:27 2005 +0000 +++ b/src/protocols/sametime/Makefile.am Thu Oct 27 20:43:43 2005 +0000 @@ -36,7 +36,7 @@ $(GLIB_CFLAGS) \ $(DEBUG_CFLAGS) \ -I$(top_srcdir)/src \ - -Imeanwhile + -I$(top_srcdir)/src/protocols/sametime/meanwhile AM_CPPFLAGS = \ diff -r 83e2fa10df1c -r f8cd06753755 src/session.c --- a/src/session.c Thu Oct 27 20:22:27 2005 +0000 +++ b/src/session.c Thu Oct 27 20:43:43 2005 +0000 @@ -165,7 +165,7 @@ /* SM callback handlers */ -void session_save_yourself(SmcConn conn, SmPointer data, int save_type, +static void session_save_yourself(SmcConn conn, SmPointer data, int save_type, Bool shutdown, int interact_style, Bool fast) { if (had_first_save == FALSE && save_type == SmSaveLocal && interact_style == SmInteractStyleNone && !shutdown && @@ -192,18 +192,18 @@ SmcSaveYourselfDone(conn, True); } -void session_die(SmcConn conn, SmPointer data) { +static void session_die(SmcConn conn, SmPointer data) { gaim_debug(GAIM_DEBUG_INFO, "Session Management", "Received die\n"); gaim_core_quit(); } -void session_save_complete(SmcConn conn, SmPointer data) { +static void session_save_complete(SmcConn conn, SmPointer data) { gaim_debug(GAIM_DEBUG_INFO, "Session Management", "Received save_complete\n"); } -void session_shutdown_cancelled(SmcConn conn, SmPointer data) { +static void session_shutdown_cancelled(SmcConn conn, SmPointer data) { gaim_debug(GAIM_DEBUG_INFO, "Session Management", "Received shutdown_cancelled\n"); }