[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-edu-devel
Subject:    Re: [kde-edu]: Step: Bug in buildsystem?
From:       Alex Merry <huntedhacker () tiscali ! co ! uk>
Date:       2008-04-06 16:12:51
Message-ID: 200804061712.59159.huntedhacker () tiscali ! co ! uk
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


On Sunday 06 April 2008 03:03:59 Tomaz Canabrava wrote:
> so it should be set outside the whole if scope, and not after the if,
> and befor the else,
> that`s why i put it inside of the if, I thougth that if it`s found,
> the message will be prompted, and the variable will be set.

I'm not sure what you mean.  The else that appears in the patch context is not 
related to the if..endif block you put the set() inside.

Philippe's patch was correct.  Although it seems irrelevant, judging from 
Albert's email.

Alex

>
> 2008/4/5 Alex Merry <huntedhacker@tiscali.co.uk>:
> > On Saturday 05 April 2008 20:38:20 Philippe Widmer wrote:
> >  > On Saturday 05 April 2008 21.17:37 Tomaz Canabrava wrote:
> > > > the set should be inside of the if.
> >  >
> >  > but this would set GMM_FOUND only if GMM_FIND_QUIETLY is set and I
> >  > think the latter defines wether such status messages should be output
> >  > or not (Though I don't know if that's true). If so, I'd assume we want
> >  > to set GMM_FOUND always, no matter wether we output status message or
> >  > not, don't we?
> >
> >  That's correct - the set should be outside the if.
> >
> >  Alex
> >
> >
> >
> >  --
> >  KDE: http://www.kde.org
> >  Ubuntu/Kubuntu: http://www.ubuntu.org http://www.kubuntu.org
> >
> >
> > _______________________________________________
> >  kde-edu mailing list
> >  kde-edu@mail.kde.org
> >  https://mail.kde.org/mailman/listinfo/kde-edu


-- 
KDE: http://www.kde.org
Ubuntu/Kubuntu: http://www.ubuntu.org http://www.kubuntu.org


["signature.asc" (application/pgp-signature)]

_______________________________________________
kde-edu mailing list
kde-edu@mail.kde.org
https://mail.kde.org/mailman/listinfo/kde-edu


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic