[kde-freebsd] small area51 trunk/area51/PORTS/sysutils/k3b-kde4 build fix

[Raphael Kubo da Costa]

On Friday 12 February 2010 09:58:54 you wrote:
> On Friday 05 February 2010 21:06:53 you wrote:
> > On Friday 05 February 2010 18:02:26 Juergen Lock wrote:
> > > I was asked on #kde-freebsd to send this by mail so here goes :)
> > > 
> > >  The important part was the missing #else part so the constructor was
> > > 
> > > missing the last arg, but since the hal text applies to freebsd too I
> > > also changed the #ifdef.
> > > 
> > > --- src/k3bsystemproblemdialog.cpp.orig
> > > +++ src/k3bsystemproblemdialog.cpp
> > > @@ -175,8 +175,10 @@ void K3b::SystemProblemDialog::checkSyst
> > > 
> > >          problems.append( K3b::SystemProblem(
> > >          K3b::SystemProblem::CRITICAL,
> > >          
> > >                                             i18n("No optical drive
> > > 
> > > found."), i18n("K3b did not find any optical device in your system."),
> > > -#ifdef Q_OS_LINUX
> > > +#if defined(Q_OS_LINUX) || defined(Q_OS_FREEBSD)
> > > 
> > >                                             i18n("Make sure HAL daemon
> > >                                             is
> > > 
> > > running, it is used by K3b for finding devices.") +#else
> > > +                                           QString()
> > > 
> > >  #endif
> > >  
> > >                                             ) );
> > >      
> > >      }
> > >  
> > >  Cheers,
> > >  
> > > 	Juergen
> > 
> > FWIW, I've committed the patch to KDE trunk and asked Juergen to post it
> > here since it's still needed with the latest released version.
> 
> Sure it's committed? Still seems to be missing:
> 
> 
> /usr/ports/sysutils/k3b-
> kde4/work/k3b-1.70.0/src/k3bsystemproblemdialog.cpp:181: error: expected
> primary-expression before ')' token
> *** Error code 1
> 
> Stop in /usr/ports/sysutils/k3b-kde4/work/k3b-1.70.0.
> *** Error code 1
> 
> Or is this a different issue?

I have committed the fix to KDE trunk, not area51 ;) That means it will be 
fixed in the next official k3b release, meanwhile someone should add it to 
area51.