libguytoolsysinfo.so fails to build: sprintf not declared

Bug #509450 reported by Kamal Mostafa
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
guymager (Ubuntu)
Fix Released
Undecided
Unassigned
libguytools1 (Debian)
Fix Released
Unknown
libguytools1 (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

Binary package hint: libguytools1

libguytoolsysinfo.so (one of the libraries built by libguytools1) fails to build:

make[1]: Entering directory `/tmp/buildd/libguytools1-1.0.4/sysinfo'
g++ -c -pipe -O2 -O3 -D_REENTRANT -Wall -W -fmessage-length=0 -fPIC -DLIBGUYTOOLS_VERSION=\"1.0.4\" -DQT_NO_DEBUG -DQT_GUI_LIB -DQT_CORE_LIB -DQT_SHARED -I/usr/share/qt4/mkspecs/linux-g++ -I. -I/usr/include/qt4/QtCore -I/usr/include/qt4/QtGui -I/usr/include/qt4 -I../include -I. -o toolsysinfo.o toolsysinfo.cpp
toolsysinfo.cpp: In function 'APIRET ToolSysInfoGetMacAddr(t_ToolSysInfoMacAddr*)':
toolsysinfo.cpp:90: error: 'sprintf' was not declared in this scope
make[1]: *** [toolsysinfo.o] Error 1
make[1]: Leaving directory `/tmp/buildd/libguytools1-1.0.4/sysinfo'

However, this qmake-qt4-based build continues anyway and completes "successfully", just omitting the broken libguytoolsysinfo.so library. Packages which depend on libguytools1 for this library will therefore FTBFS, e.g. package 'guymager' http://launchpadlibrarian.net/35999162/buildlog_ubuntu-lucid-amd64.guymager_0.4.2-1_FAILEDTOBUILD.txt.gz :

   [guymager...FAILEDTOBUILD.txt.gz]: /usr/bin/ld: cannot find -lguytoolsysinfo

Related branches

Changed in libguytools1 (Ubuntu):
assignee: nobody → Kamal Mostafa (kamalmostafa)
status: New → In Progress
Changed in guymager (Ubuntu):
assignee: nobody → Kamal Mostafa (kamalmostafa)
status: New → In Progress
description: updated
Revision history for this message
Kamal Mostafa (kamalmostafa) wrote :

Added the missing #include <stdio.h> to fix libguytoolsysinfo.so in libguytools1 (attached branch). Verified that package 'guymager' no longer FTBFS with the fixed libguytools1. Reported and patch forwarded upstream to Debian.

Changed in libguytools1 (Ubuntu):
assignee: Kamal Mostafa (kamalmostafa) → nobody
status: In Progress → Confirmed
Changed in guymager (Ubuntu):
assignee: Kamal Mostafa (kamalmostafa) → nobody
status: In Progress → Confirmed
Changed in libguytools1 (Ubuntu):
assignee: nobody → Ilya Barygin (randomaction)
status: Confirmed → In Progress
Revision history for this message
Ilya Barygin (randomaction) wrote :

Uploaded libguytools1, thank you for your contribution. Will reschedule rebuilds for guymager when libguytools1 finishes to build.

Changed in libguytools1 (Ubuntu):
assignee: Ilya Barygin (randomaction) → nobody
status: In Progress → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package libguytools1 - 1.0.4-1ubuntu1

---------------
libguytools1 (1.0.4-1ubuntu1) lucid; urgency=low

  * sysinfo/toolsysinfo.cpp: added #include <stdio> for sprintf (LP: #509450).
 -- Kamal Mostafa <email address hidden> Mon, 18 Jan 2010 14:10:57 -0800

Changed in libguytools1 (Ubuntu):
status: Fix Committed → Fix Released
Changed in guymager (Ubuntu):
status: Confirmed → Fix Released
Changed in libguytools1 (Debian):
status: Unknown → New
Changed in libguytools1 (Debian):
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.