[hal patch] Fix building with g++ 4.3

Martin Pitt martin at piware.de
Mon Mar 3 07:27:20 PST 2008


Hi,

Michael Biebl [2008-03-03 16:19 +0100]:
> 2008/3/3, Martin Pitt <martin at piware.de>:
> > Hi,
> >
> >  attached is a trivial patch which fixes building hal with g++ 4.3.
> 
> Hm,  why do you want to include a c++ header in a c project?

Because the autotools stuff automatically uses g++ to build a *.cpp
file, and addon-dell-backlight.cpp actually uses dynamic_cast,
namespace, and other C++ features.

Maybe the more correct fix is to change the file to only use C, but
that's more invasive, and might not be desired in the first place.

Thanks,

Martin

-- 
Martin Pitt                        | http://www.piware.de
Ubuntu Developer (www.ubuntu.com)  | Debian Developer  (www.debian.org)

My 5 today: #195799 (smplayer-themes), #195389 (deluge-torrent), #195812
(hipo), #195177 (qlandkarte), #196100 (xserver-xorg-video-trident)
Do 5 a day - every day! https://wiki.ubuntu.com/5-A-Day


More information about the hal mailing list