/pidgin/main: 30107eb2ff81: Fix compiler warnings, make build ev...
Tomasz Wasilczyk
tomkiewicz at cpw.pidgin.im
Tue Aug 13 06:14:25 EDT 2013
Changeset: 30107eb2ff81255b430848b6b24f34b88aee0794
Author: Tomasz Wasilczyk <tomkiewicz at cpw.pidgin.im>
Date: 2013-08-13 12:14 +0200
Branch: default
URL: https://hg.pidgin.im/pidgin/main/rev/30107eb2ff81
Description:
Fix compiler warnings, make build even more silent
diffstat:
finch/libgnt/Makefile.am | 8 ++++--
libpurple/plugins/Makefile.am | 6 +++-
libpurple/plugins/perl/Makefile.am | 7 +++--
libpurple/tag.sh | 29 ++++++++++++++++---------
pidgin/gtkutils.c | 2 +-
pidgin/plugins/musicmessaging/Makefile.am | 6 +++-
pidgin/plugins/musicmessaging/musicmessaging.c | 2 +
pidgin/plugins/perl/Makefile.am | 7 +++--
8 files changed, 42 insertions(+), 25 deletions(-)
diffs (206 lines):
diff --git a/finch/libgnt/Makefile.am b/finch/libgnt/Makefile.am
--- a/finch/libgnt/Makefile.am
+++ b/finch/libgnt/Makefile.am
@@ -73,11 +73,13 @@ CLEANFILES = \
gntmarshal.c
gntmarshal.c: $(srcdir)/genmarshal gntmarshal.h
- echo "#include \"gntmarshal.h\"" > $@
- glib-genmarshal --prefix=gnt_closure_marshal --body $(srcdir)/genmarshal >> $@
+ @echo -e " GEN\t$@"
+ @echo "#include \"gntmarshal.h\"" > $@
+ @glib-genmarshal --prefix=gnt_closure_marshal --body $(srcdir)/genmarshal >> $@
gntmarshal.h: $(srcdir)/genmarshal
- glib-genmarshal --prefix=gnt_closure_marshal --header $(srcdir)/genmarshal > $@
+ @echo -e " GEN\t$@"
+ @glib-genmarshal --prefix=gnt_closure_marshal --header $(srcdir)/genmarshal > $@
libgnt_laincludedir=$(includedir)/gnt
libgnt_lainclude_HEADERS = \
diff --git a/libpurple/plugins/Makefile.am b/libpurple/plugins/Makefile.am
--- a/libpurple/plugins/Makefile.am
+++ b/libpurple/plugins/Makefile.am
@@ -118,10 +118,12 @@ dbus_example_la_LIBADD = $(GLIB_LIB
.PHONY: always
$(top_builddir)/libpurple/dbus-types.h: always
- cd $(@D) && $(MAKE) $(AM_MAKEFLAGS) $(@F)
+ @echo -e " GEN\t$@"
+ @cd $(@D) && $(MAKE) $(AM_MAKEFLAGS) $(@F)
dbus-example-bindings.c: $(top_srcdir)/libpurple/dbus-analyze-functions.py $(dbus_example_la_SOURCES)
- cat $(srcdir)/$(dbus_example_la_SOURCES) | \
+ @echo -e " GEN\t$@"
+ @cat $(srcdir)/$(dbus_example_la_SOURCES) | \
$(PYTHON) $(top_srcdir)/libpurple/dbus-analyze-functions.py --export-only > $@
$(dbus_example_la_OBJECTS) dbus-example.so: dbus-example-bindings.c $(top_builddir)/libpurple/dbus-types.h
diff --git a/libpurple/plugins/perl/Makefile.am b/libpurple/plugins/perl/Makefile.am
--- a/libpurple/plugins/perl/Makefile.am
+++ b/libpurple/plugins/perl/Makefile.am
@@ -92,6 +92,7 @@ EXTRA_DIST = \
$(perl_scripts)
common/Makefile: common/Makefile.PL
+ @echo -e " GEN\t$@"
@if test "x${top_srcdir}" != "x${top_builddir}"; then \
for f in ${common_sources}; do \
srcloc=${srcdir}; \
@@ -99,7 +100,7 @@ common/Makefile: common/Makefile.PL
${LN_S} -f $$srcloc/$$f $$f; \
done; \
fi
- @cd common && $(perlpath) Makefile.PL
+ @cd common && $(perlpath) Makefile.PL > /dev/null
common/Makefile.PL: common/Makefile.PL.in $(top_builddir)/config.status
cd $(top_builddir) && $(SHELL) ./config.status $(subdir)/$@ $(am__depfiles_maybe)
@@ -110,8 +111,8 @@ all-local: common/Makefile
if [ ! -f Makefile ]; then \
$(perlpath) Makefile.PL; \
fi && \
- ($(MAKE) CC="@$(abs_top_srcdir)/libpurple/tag.sh CC $(CC)" PERLRUN="@$(abs_top_srcdir)/libpurple/tag.sh PERL $(PERL)" CCFLAGS="$(PERL_CFLAGS) $(CFLAGS)" $(PERL_EXTRA_OPTS) || \
- $(MAKE) CC="@$(abs_top_srcdir)/libpurple/tag.sh CC $(CC)" PERLRUN="@$(abs_top_srcdir)/libpurple/tag.sh PERL $(PERL)" CCFLAGS="$(PERL_CFLAGS) $(CFLAGS)" $(PERL_EXTRA_OPTS)) && \
+ ($(MAKE) CC="@$(abs_top_srcdir)/libpurple/tag.sh CC $(CC)" LD="@$(abs_top_srcdir)/libpurple/tag.sh LD $(CC)" PERLRUN="@$(abs_top_srcdir)/libpurple/tag.sh PERL $(PERL)" CCFLAGS="$(PERL_CFLAGS) $(CFLAGS)" CP="@cp" RM_F="@rm -f" CHMOD="@chmod" $(PERL_EXTRA_OPTS) || \
+ $(MAKE) CC="@$(abs_top_srcdir)/libpurple/tag.sh CC $(CC)" LD="@$(abs_top_srcdir)/libpurple/tag.sh LD $(CC)" PERLRUN="@$(abs_top_srcdir)/libpurple/tag.sh PERL $(PERL)" CCFLAGS="$(PERL_CFLAGS) $(CFLAGS)" CP="@cp" RM_F="@rm -f" CHMOD="@chmod" $(PERL_EXTRA_OPTS)) && \
cd ..; \
done
diff --git a/libpurple/tag.sh b/libpurple/tag.sh
--- a/libpurple/tag.sh
+++ b/libpurple/tag.sh
@@ -7,8 +7,9 @@
tag=$1
found=0
object=""
-c_file=""
-xs_file=""
+file_1=""
+file_2=""
+is_final=0
for arg in "$@"
do
if [ "$found" == 1 ]; then
@@ -21,16 +22,22 @@ do
if [ "$tag" == "auto" ] && [ "$arg" == "-shared" ]; then
tag="CCLD"
fi
- if [ "$tag" == "PERL" ] && [ "$arg" == "-e" ]; then
- found=1
+ if [ "$tag" == "PERL" ] && [ "${arg%(*}" == "Mkbootstrap" ]; then
+ object="${arg%;}"
+ is_final=1
+ break
fi
ext_1=${arg#${arg%??}}
if [ "${ext_1}" == ".c" ]; then
- c_file="$arg"
+ file_1="$arg"
fi
ext_2=${arg#${arg%???}}
if [ "${ext_2}" == ".xs" ]; then
- xs_file="$arg"
+ file_2="$arg"
+ fi
+ ext_3=${arg#${arg%????}}
+ if [ "${ext_3}" == ".3pm" ]; then
+ file_2="$arg"
fi
done
@@ -38,15 +45,15 @@ if [ "$tag" == "auto" ]; then
tag="CC"
fi
-if [ "$tag" == "PERL" ]; then
+if [ "$tag" == "PERL" ] && [ "$is_final" == 0 ]; then
object=`echo "$object" | sed -n 's|.*output *=> *"\([^"]*\)".*|\1|p'`
fi
-if [ "$object" == "" ] && [ "${c_file}" != "" ]; then
- object="${c_file}"
+if [ "$object" == "" ] && [ "${file_1}" != "" ]; then
+ object="${file_1}"
fi
-if [ "$object" == "" ] && [ "${xs_file}" != "" ]; then
- object="${xs_file}"
+if [ "$object" == "" ] && [ "${file_2}" != "" ]; then
+ object="${file_2}"
fi
shift 1
diff --git a/pidgin/gtkutils.c b/pidgin/gtkutils.c
--- a/pidgin/gtkutils.c
+++ b/pidgin/gtkutils.c
@@ -249,7 +249,7 @@ pidgin_create_webview(gboolean editable,
if (editable) {
gtk_webviewtoolbar_attach(GTK_WEBVIEWTOOLBAR(toolbar), webview);
gtk_webviewtoolbar_associate_smileys(GTK_WEBVIEWTOOLBAR(toolbar), "default");
- gtk_webview_set_toolbar(webview, toolbar);
+ gtk_webview_set_toolbar(GTK_WEBVIEW(webview), toolbar);
}
pidgin_setup_webview(webview);
diff --git a/pidgin/plugins/musicmessaging/Makefile.am b/pidgin/plugins/musicmessaging/Makefile.am
--- a/pidgin/plugins/musicmessaging/Makefile.am
+++ b/pidgin/plugins/musicmessaging/Makefile.am
@@ -23,10 +23,12 @@ CLEANFILES = music-messagin
.PHONY: always
$(top_builddir)/libpurple/dbus-types.h: always
- cd $(@D) && $(MAKE) $(AM_MAKEFLAGS) $(@F)
+ @echo -e " GEN\t$@"
+ @cd $(@D) && $(MAKE) $(AM_MAKEFLAGS) $(@F)
music-messaging-bindings.c: $(top_srcdir)/libpurple/dbus-analyze-functions.py $(musicmessaging_la_SOURCES)
- cat $(srcdir)/$(musicmessaging_la_SOURCES) | \
+ @echo -e " GEN\t$@"
+ @cat $(srcdir)/$(musicmessaging_la_SOURCES) | \
$(PYTHON) $(top_srcdir)/libpurple/dbus-analyze-functions.py --export-only > $@
$(musicmessaging_la_OBJECTS) musicmessaging.so: music-messaging-bindings.c $(top_builddir)/libpurple/dbus-types.h
diff --git a/pidgin/plugins/musicmessaging/musicmessaging.c b/pidgin/plugins/musicmessaging/musicmessaging.c
--- a/pidgin/plugins/musicmessaging/musicmessaging.c
+++ b/pidgin/plugins/musicmessaging/musicmessaging.c
@@ -586,7 +586,9 @@ static void conv_destroyed (PurpleConver
static void add_button (MMConversation *mmconv)
{
+#if 0
PurpleConversation *conv = mmconv->conv;
+#endif
GtkWidget *button, *image, *sep;
gchar *file_path;
diff --git a/pidgin/plugins/perl/Makefile.am b/pidgin/plugins/perl/Makefile.am
--- a/pidgin/plugins/perl/Makefile.am
+++ b/pidgin/plugins/perl/Makefile.am
@@ -37,6 +37,7 @@ EXTRA_DIST = \
$(common_sources)
common/Makefile: common/Makefile.PL
+ @echo -e " GEN\t$@"
@if test "x${top_srcdir}" != "x${top_builddir}"; then \
for f in ${common_sources}; do \
srcloc=${srcdir}; \
@@ -44,7 +45,7 @@ common/Makefile: common/Makefile.PL
${LN_S} -f $$srcloc/$$f $$f; \
done; \
fi
- @cd common && $(perlpath) Makefile.PL
+ @cd common && $(perlpath) Makefile.PL > /dev/null
common/Makefile.PL: common/Makefile.PL.in $(top_builddir)/config.status
cd $(top_builddir) && $(SHELL) ./config.status $(subdir)/$@ $(am__depfiles_maybe)
@@ -55,8 +56,8 @@ all-local: common/Makefile
if [ ! -f Makefile ]; then \
$(perlpath) Makefile.PL; \
fi && \
- ($(MAKE) CC="@$(abs_top_srcdir)/libpurple/tag.sh CC $(CC)" PERLRUN="@$(abs_top_srcdir)/libpurple/tag.sh PERL $(PERL)" CCFLAGS="$(PERL_CFLAGS) $(CFLAGS)" $(PERL_EXTRA_OPTS) || \
- $(MAKE) CC="@$(abs_top_srcdir)/libpurple/tag.sh CC $(CC)" PERLRUN="@$(abs_top_srcdir)/libpurple/tag.sh PERL $(PERL)" CCFLAGS="$(PERL_CFLAGS) $(CFLAGS)" $(PERL_EXTRA_OPTS)) && \
+ ($(MAKE) CC="@$(abs_top_srcdir)/libpurple/tag.sh CC $(CC)" LD="@$(abs_top_srcdir)/libpurple/tag.sh LD $(CC)" PERLRUN="@$(abs_top_srcdir)/libpurple/tag.sh PERL $(PERL)" CCFLAGS="$(PERL_CFLAGS) $(CFLAGS)" CP="@cp" RM_F="@rm -f" CHMOD="@chmod" $(PERL_EXTRA_OPTS) || \
+ $(MAKE) CC="@$(abs_top_srcdir)/libpurple/tag.sh CC $(CC)" LD="@$(abs_top_srcdir)/libpurple/tag.sh LD $(CC)" PERLRUN="@$(abs_top_srcdir)/libpurple/tag.sh PERL $(PERL)" CCFLAGS="$(PERL_CFLAGS) $(CFLAGS)" CP="@cp" RM_F="@rm -f" CHMOD="@chmod" $(PERL_EXTRA_OPTS)) && \
cd ..; \
done
More information about the Commits
mailing list