[gstreamer-bugs] [Bug 151419] New: - [PATCH] Fix CVS jhbuild build failure of progressreport.c

[bugzilla-daemon at bugzilla.gnome.org]

http://bugzilla.gnome.org/show_bug.cgi?id=151419
GStreamer | gst-plugins | Ver: HEAD CVS

           Summary: [PATCH] Fix CVS jhbuild build failure of
                    progressreport.c
           Product: GStreamer
           Version: HEAD CVS
          Platform: Other
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gst-plugins
        AssignedTo: gstreamer-bugs at lists.sourceforge.net
        ReportedBy: nathanr at nathanr.net
         QAContact: gstreamer-bugs at lists.sourceforge.net


jhbuild has -Werror on by default. This causes:

 gcc -DHAVE_CONFIG_H -I. -I. -I../.. -I../../gst-libs -I../../gst-libs -pthread
-D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64
-I/home/nathanr/bin/gnome2-cvs/include/glib-2.0
-I/home/nathanr/bin/gnome2-cvs/lib64/glib-2.0/include
-I/home/nathanr/bin/gnome2-cvs/include/libxml2
-I/home/nathanr/bin/gnome2-cvs/include/gstreamer-0.8 -DGST_DISABLE_DEPRECATED
-Wall -Werror -g -O2 -MT libgstdebug_la-progressreport.lo -MD -MP -MF
.deps/libgstdebug_la-progressreport.Tpo -c progressreport.c  -fPIC -DPIC -o
.libs/libgstdebug_la-progressreport.o
progressreport.c: In function `gst_progressreport_report':
progressreport.c:206: warning: long long int format, gint64 arg (arg 6)
progressreport.c:206: warning: long long int format, gint64 arg (arg 7)
progressreport.c:210: warning: long long int format, gint64 arg (arg 6)
make[3]: *** [libgstdebug_la-progressreport.lo] Error 1
make[3]: Leaving directory `/home/nathanr/download/gnome-cvs/gst-plugins/gst/debug'
make[2]: *** [all-recursive] Error 1
make[2]: Leaving directory `/home/nathanr/download/gnome-cvs/gst-plugins/gst'
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory `/home/nathanr/download/gnome-cvs/gst-plugins'
make: *** [all] Error 2
*** error during stage build of gst-plugins: could not build module *** [70/91]

I'm on AMD64, and have a feeling this has something to do with this warning.
Explicitly casting to what printf() expects fixes this. Patch below:

--- progressreport.c.~1.2.~     2004-05-21 23:28:24.000000000 +1000
+++ progressreport.c    2004-08-30 22:36:42.504853840 +1000
@@ -202,12 +202,12 @@
     if (got_total == TRUE) {
       g_print ("%s (%2d:%2d:%2d): %lld / %lld %s (%3.2g %%)\n",
           gst_object_get_name (GST_OBJECT (progressreport)), hh, mm, ss,
-          cur_progress, total_progress, format_name,
+          ((long long int) cur_progress), ((long long int) total_progress),
format_name,
           ((gdouble) (cur_progress)) / total_progress * 100);
     } else {
       g_print ("%s (%2d:%2d:%2d): %lld %s\n",
           gst_object_get_name (GST_OBJECT (progressreport)), hh, mm, ss,
-          cur_progress, format_name);
+          ((long long int) cur_progress), format_name);
     }
   } else {
     g_print ("%s (%2d:%2d:%2d): Could not query current position.\n",

------- You are receiving this mail because: -------
You are the assignee for the bug.
You are the QA contact for the bug.