Re: [PATCH] Re: [PATCH] OPW 2013: Build System Gtest Addition

2013-02-12 Thread Branko Čibej
On 12.02.2013 18:36, Stefan Sperling wrote: > On Wed, Feb 06, 2013 at 08:37:04AM +, Gabriela Gibson wrote: >> On 05/02/13 17:00, Branko Čibej wrote: >>> On 05.02.2013 14:28, Gabriela Gibson wrote: >>> I'm almost sure you meant, "path = gtestlib". >>> >>> -- Brane >>> >> Nope, meant libgtest, en

Re: [PATCH] Re: [PATCH] OPW 2013: Build System Gtest Addition

2013-02-12 Thread Stefan Sperling
On Wed, Feb 06, 2013 at 08:37:04AM +, Gabriela Gibson wrote: > On 05/02/13 17:00, Branko Čibej wrote: > >On 05.02.2013 14:28, Gabriela Gibson wrote: > >I'm almost sure you meant, "path = gtestlib". > > > >-- Brane > > > Nope, meant libgtest, ended up with both and the mistake compiled %-) > >

Re: [PATCH] Re: [PATCH] OPW 2013: Build System Gtest Addition

2013-02-06 Thread Gabriela Gibson
On 05/02/13 17:00, Branko Čibej wrote: On 05.02.2013 14:28, Gabriela Gibson wrote: I'm almost sure you meant, "path = gtestlib". -- Brane Nope, meant libgtest, ended up with both and the mistake compiled %-) I think it's working now, see attached patch. Index: configure.ac =

Re: [PATCH] Re: [PATCH] OPW 2013: Build System Gtest Addition

2013-02-05 Thread Branko Čibej
On 05.02.2013 14:28, Gabriela Gibson wrote: > On 02/02/13 10:31, Branko Čibej wrote: >> You could try adding ${abs_srcdir}/gtest/include to the include path. :) >> Also, you'll get rid of those warnings by adding appropriate paths to >> the private-includes list in build.conf. >> >> -- Brane >> > T

Re: [PATCH] Re: [PATCH] OPW 2013: Build System Gtest Addition

2013-02-05 Thread Branko Čibej
On 05.02.2013 14:28, Gabriela Gibson wrote: > On 02/02/13 10:31, Branko Čibej wrote: >> You could try adding ${abs_srcdir}/gtest/include to the include path. :) >> Also, you'll get rid of those warnings by adding appropriate paths to >> the private-includes list in build.conf. >> >> -- Brane >> > T