Bug 151391 - kde-base/kdenetwork-3.5.5 fails with sys-kernel/linux-headers-2.6.18
Bug#: 151391 Product:  Gentoo Linux Version: 2006.1 Platform: All
OS/Version: Linux Status: RESOLVED Severity: normal Priority: P2
Resolution: FIXED Assigned To: kde@gentoo.org Reported By: portage@bigmichi1.dyndns.org
Component: KDE
URL: 
Summary: kde-base/kdenetwork-3.5.5 fails with sys-kernel/linux-headers-2.6.18
Keywords:  
Status Whiteboard: 
Opened: 2006-10-14 15:00 0000
Description:   Opened: 2006-10-14 15:00 0000 
/bin/sh ../../../libtool --silent --tag=CXX --mode=compile
i686-pc-linux-gnu-g++ -DHAVE_CONFIG_H -I. -I. -I../../..
-I../../../kopete/protocols/gadu/libgadu  -I/usr/kde/3.5/include
-I/usr/qt/3/include -I.  -DKDE_NO_COMPAT -DQT_NO_COMPAT -DQT_NO_CAST_ASCII
-DQT_NO_ASCII_CAST -I../../../kopete/libkopete -I../../../kopete/libkopete
-I../../../kopete/libkopete/avdevice -I../../../kopete/libkopete/ui
-I../../../kopete/libkopete/ui -I../../../kopete/libkopete/private
-I../../../kopete/libkopete/ui -I/usr/kde/3.5/include -I/usr/qt/3/include -I. 
-DQT_THREAD_SUPPORT  -D_REENTRANT -D_FILE_OFFSET_BITS=64  -Wno-long-long
-Wundef -ansi -D_XOPEN_SOURCE=500 -D_BSD_SOURCE -Wcast-align -Wconversion
-Wchar-subscripts -Wall -W -Wpointer-arith -DNDEBUG -DNO_DEBUG -O2 -O2
-march=pentium3 -pipe -fomit-frame-pointer -fforce-addr -Wformat-security
-Wmissing-format-attribute -Wno-non-virtual-dtor -fno-exceptions -fno-check-new
-fno-common -fvisibility=hidden -fvisibility-inlines-hidden
-DQT_CLEAN_NAMESPACE -DQT_NO_ASCII_CAST -DQT_NO_STL -DQT_NO_COMPAT
-DQT_NO_TRANSLATION  -c -o libkvideoio_la.all_cpp.lo libkvideoio_la.all_cpp.cpp
/usr/include/linux/videodev2.h:622: error: '__s64' does not name a type
kxv.cpp:118: warning: unused parameter 'd'
kxv.cpp:208: warning: unused parameter 'win'
kxv.cpp:208: warning: unused parameter 'data'
kxv.cpp:208: warning: unused parameter 'w'
kxv.cpp:208: warning: unused parameter 'h'
kxv.cpp:208: warning: unused parameter 'x'
kxv.cpp:208: warning: unused parameter 'y'
kxv.cpp:208: warning: unused parameter 'sw'
kxv.cpp:208: warning: unused parameter 'sh'
kxv.cpp:208: warning: unused parameter 'dw'
kxv.cpp:208: warning: unused parameter 'dh'
kxv.cpp:265: warning: unused parameter 'w'
kxv.cpp:265: warning: unused parameter 'dw'
kxv.cpp:265: warning: unused parameter 'dh'
kxv.cpp:466: warning: unused parameter 'w'
kxv.cpp:525: warning: unused parameter 'attribute'
kxv.cpp:525: warning: unused parameter 'val'
kxv.cpp:542: warning: unused parameter 'attribute'
kxv.cpp:542: warning: unused parameter 'val'
kxv.cpp:575: warning: unused parameter 'encoding'
kxv.cpp:594: warning: unused parameter 'e'
kxv.cpp:639: warning: unused parameter 'w'
kxv.cpp:639: warning: unused parameter 'h'
kxv.cpp:639: warning: unused parameter 'shm'
videodevice.cpp: In member function 'int
Kopete::AV::VideoDevice::selectInput(int)':
videodevice.cpp:752: warning: comparison between signed and unsigned integer
expressions
videodevice.cpp: In member function 'int
Kopete::AV::VideoDevice::setInputParameters()':
videodevice.cpp:798: warning: comparison between signed and unsigned integer
expressions
videodevice.cpp: In member function 'float
Kopete::AV::VideoDevice::getBrightness()':
videodevice.cpp:1294: warning: comparison between signed and unsigned integer
expressions
videodevice.cpp: In member function 'float
Kopete::AV::VideoDevice::setBrightness(float)':
videodevice.cpp:1317: warning: converting to '__u16' from 'float'
videodevice.cpp: In member function 'float
Kopete::AV::VideoDevice::getContrast()':
videodevice.cpp:1332: warning: comparison between signed and unsigned integer
expressions
videodevice.cpp: In member function 'float
Kopete::AV::VideoDevice::setContrast(float)':
videodevice.cpp:1355: warning: converting to '__u16' from 'float'
videodevice.cpp: In member function 'float
Kopete::AV::VideoDevice::getSaturation()':
videodevice.cpp:1370: warning: comparison between signed and unsigned integer
expressions
videodevice.cpp: In member function 'float
Kopete::AV::VideoDevice::setSaturation(float)':
videodevice.cpp:1393: warning: converting to '__u16' from 'float'
videodevice.cpp: In member function 'float
Kopete::AV::VideoDevice::getWhiteness()':
videodevice.cpp:1408: warning: comparison between signed and unsigned integer
expressions
videodevice.cpp: In member function 'float
Kopete::AV::VideoDevice::setWhiteness(float)':
videodevice.cpp:1431: warning: converting to '__u16' from 'float'
videodevice.cpp: In member function 'float Kopete::AV::VideoDevice::getHue()':
videodevice.cpp:1446: warning: comparison between signed and unsigned integer
expressions
videodevice.cpp: In member function 'float
Kopete::AV::VideoDevice::setHue(float)':
videodevice.cpp:1469: warning: converting to '__u16' from 'float'
videodevice.cpp: In member function 'bool
Kopete::AV::VideoDevice::getAutoBrightnessContrast()':
videodevice.cpp:1485: warning: comparison between signed and unsigned integer
expressions
videodevice.cpp: In member function 'bool
Kopete::AV::VideoDevice::setAutoBrightnessContrast(bool)':
videodevice.cpp:1494: warning: comparison between signed and unsigned integer
expressions
videodevice.cpp: In member function 'bool
Kopete::AV::VideoDevice::getAutoColorCorrection()':
videodevice.cpp:1506: warning: comparison between signed and unsigned integer
expressions
videodevice.cpp: In member function 'bool
Kopete::AV::VideoDevice::setAutoColorCorrection(bool)':
videodevice.cpp:1515: warning: comparison between signed and unsigned integer
expressions
videodevice.cpp: In member function 'bool
Kopete::AV::VideoDevice::getImageAsMirror()':
videodevice.cpp:1526: warning: comparison between signed and unsigned integer
expressions
videodevice.cpp: In member function 'bool
Kopete::AV::VideoDevice::setImageAsMirror(bool)':
videodevice.cpp:1535: warning: comparison between signed and unsigned integer
expressions
videodevicepool.cpp: In member function 'int
Kopete::AV::VideoDevicePool::getImage(QImage*)':
videodevicepool.cpp:471: warning: enumeration value 'PIXELFORMAT_YUYV' not
handled in switch
videodevicepool.cpp:471: warning: enumeration value 'PIXELFORMAT_UYVY' not
handled in switch
videodevicepool.cpp:471: warning: enumeration value 'PIXELFORMAT_YUV420P' not
handled in switch
videodevicepool.cpp:471: warning: enumeration value 'PIXELFORMAT_YUV422P' not
handled in switch
videodevicepool.cpp: In member function 'void
Kopete::AV::VideoDevicePool::loadConfig()':
videodevicepool.cpp:816: warning: unused variable 'disablemmap'
/bin/sh ../../../libtool --silent --tag=CXX --mode=link i686-pc-linux-gnu-g++ 
-Wno-long-long -Wundef -ansi -D_XOPEN_SOURCE=500 -D_BSD_SOURCE -Wcast-align
-Wconversion -Wchar-subscripts -Wall -W -Wpointer-arith -DNDEBUG -DNO_DEBUG -O2
-O2 -march=pentium3 -pipe -fomit-frame-pointer -fforce-addr -Wformat-security
-Wmissing-format-attribute -Wno-non-virtual-dtor -fno-exceptions -fno-check-new
-fno-common -fvisibility=hidden -fvisibility-inlines-hidden
-DQT_CLEAN_NAMESPACE -DQT_NO_ASCII_CAST -DQT_NO_STL -DQT_NO_COMPAT
-DQT_NO_TRANSLATION   -Wl,-O1 -Wl,--sort-common -Wl,--as-needed -o
libkvideoio.la  -no-undefined -Wl,--no-undefined -Wl,--allow-shlib-undefined
-L/usr/kde/3.5/lib -L/usr/qt/3/lib -L/usr/lib    -version-info 1:0:0
libkvideoio_la.all_cpp.lo  -lqt-mt  -lz -lpng -lz -lm -lXext -lX11  -lSM -lICE
-lpthread -lkdecore -lGLU -lGL -lX11
make[4]: *** [libkopete_videodevice_la.all_cpp.lo] Error 1
make[4]: *** Waiting for unfinished jobs....
libtool: link: warning: `-version-info/-version-number' is ignored for
convenience libraries
make[4]: Leaving directory
`/var/tmp/portage/kde-base/kdenetwork-3.5.5/work/kdenetwork-3.5.5/kopete/libkopete/avdevice'
make[3]: *** [all-recursive] Error 1
make[3]: Leaving directory
`/var/tmp/portage/kde-base/kdenetwork-3.5.5/work/kdenetwork-3.5.5/kopete/libkopete'
make[2]: *** [all-recursive] Error 1
make[2]: Leaving directory
`/var/tmp/portage/kde-base/kdenetwork-3.5.5/work/kdenetwork-3.5.5/kopete'
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory
`/var/tmp/portage/kde-base/kdenetwork-3.5.5/work/kdenetwork-3.5.5'
make: *** [all] Error 2

!!! ERROR: kde-base/kdenetwork-3.5.5 failed.

------- Comment #1 From Matteo Azzali 2006-10-15 04:20:55 0000 ------- 
'__s64' does not name a type

.... shouldn't the kernel headers define its own type?

------- Comment #2 From Diego E. 'Flameeyes' Pettenò 2006-10-15 04:37:38 0000 ------- 
The problem is with those headers, that are currently -* so it's not supported
by KDE team for sure. Kernel team can you look?

------- Comment #3 From Christian Heim (RETIRED) 2006-10-15 04:47:36 0000 ------- 
(In reply to comment #2)
> The problem is with those headers, that are currently -* so it's not supported
> by KDE team for sure. Kernel team can you look?

kernel team also isn't supporting those headers .. (there is a reason _why_
they are marked with -*).

------- Comment #4 From Jakub Moc (RETIRED) 2006-10-15 04:48:43 0000 ------- 
*** Bug 151454 has been marked as a duplicate of this bug. ***

------- Comment #5 From Daniel Drake 2006-10-15 12:12:58 0000 ------- 
toolchain: feel free just to close this if linux-headers-2.6.18 is known broken

------- Comment #6 From SpanKY 2006-10-15 17:37:27 0000 ------- 
i would say this is a KDE problem

__s64 requires non-strict ansi ... you're obviously building with -ansi so this
isnt going to work

previous headers we broke this behavior by always defining s64, but with 2.6.18
we're going for much less changes

------- Comment #7 From Matteo Azzali 2006-10-16 06:53:34 0000 ------- 
Sorry but I don't understand, please explain me, I do:

#equery b /usr/include/linux/videodev2.h
[ Searching for file(s) /usr/include/linux/videodev2.h in *... ]
sys-kernel/linux-headers-2.6.17-r1 (/usr/include/linux/videodev2.h)

the __s64 error is reported to be in that file, which belongs to linux
headers, are you meaning that kde team has for this time the right to modify
the linux-headers (toolchain herd) , that the error is reported to be in
that file because gcc got drunk but is in reality in a kde package or what?

------- Comment #8 From SpanKY 2006-10-16 12:36:59 0000 ------- 
i'm telling you that s64 is not supported when compiling with -ansi

------- Comment #9 From Diego E. 'Flameeyes' Pettenò 2006-10-16 14:52:00 0000 ------- 
*** Bug 151639 has been marked as a duplicate of this bug. ***

------- Comment #10 From Matteo Azzali 2006-10-17 05:14:53 0000 ------- 
Ok, but the issue is:

/usr/include/linux/videodev2.h:622: error: '__s64' does not name a type

and /usr/include/linux/videodev2.h is a linux-headers file.....
Don't this means that there's some "orphaned" non-ansi code in the videodev2.h
file (the s64 definition was removed from linux-headers but not all its
invocations)? If the s64 type was invoked in the kdenetwork package, 
shouldn't gcc report this error in a kdenetwork package file????

------- Comment #11 From Charlie Shepherd (RETIRED) 2006-10-17 05:17:57 0000 ------- 
__s64 is defined in linux/types.h in line 98:

#if defined(__GNUC__) && !defined(__STRICT_ANSI__)
typedef         __u64           uint64_t;
typedef         __u64           u_int64_t;
typedef         __s64           int64_t;
#endif

A similar error was reported here (
http://permalink.gmane.org/gmane.linux.lfs.beyond.support/35397 ) though its
for joystick.h

------- Comment #12 From Matteo Azzali 2006-10-17 05:19:29 0000 ------- 
Oh, maybe I got it, linux headers ARE non-ansi code, and that is fine for
linux headers, while it's not when compiling this package as strict ansi.
Right?

------- Comment #13 From Matteo Azzali 2006-10-17 05:42:41 0000 ------- 
Created an attachment (id=99873) [details]
kdenetwork-3.5.5-linux-headers-2.6.18.patch

I just susbstituted asm/types inlude with linux/types making sure that all
the linux headers include are where STRICT_ANSI is not defined.

Should work but please test (cause I can't)

------- Comment #14 From Michael Cramer 2006-10-17 10:00:10 0000 ------- 
with the patch, kopete will compile

------- Comment #15 From SpanKY 2006-10-17 11:24:09 0000 ------- 
> Oh, maybe I got it, linux headers ARE non-ansi code, and that is fine for
> linux headers, while it's not when compiling this package as strict ansi.

which is why this is KDE's problem

it relies on code from the kernel that is non-ansi, yet it knowingly tries to
use it anyways

your proposed patch looks "ok" to me

------- Comment #16 From Diego E. 'Flameeyes' Pettenò 2006-11-01 13:04:44 0000 ------- 
*** Bug 153741 has been marked as a duplicate of this bug. ***

------- Comment #17 From Diego E. 'Flameeyes' Pettenò 2006-12-29 11:34:12 0000 ------- 
Committed.