[PATCH weston v5 5/6] Converted the config parser test to the new framework.

[Jon A. Cruz]

On 06/25/2015 07:18 AM, Pekka Paalanen wrote:
> On Sat, 20 Jun 2015 15:47:47 -0700
> "Jon A. Cruz" <jonc at osg.samsung.com> wrote:
> 
>> > Signed-off-by: Jon A. Cruz <jonc at osg.samsung.com>
>> > ---
>> >  Makefile.am                |   9 +-
>> >  tests/config-parser-test.c | 368 +++++++++++++++++++++++++++++++++++++--------
>> >  2 files changed, 310 insertions(+), 67 deletions(-)
> Hi Jon,
> 
> this is a nice example on how your framework is used.
> 
> 
>> > diff --git a/Makefile.am b/Makefile.am
>> > index f0ed4d2..8234d2a 100644
>> > --- a/Makefile.am
>> > +++ b/Makefile.am
>> > @@ -1144,7 +1144,14 @@ libtest_runner_la_SOURCES =			\
>> >  libtest_runner_la_CFLAGS = $(GCC_CFLAGS) $(COMPOSITOR_CFLAGS)
>> >  
>> >  config_parser_test_SOURCES = tests/config-parser-test.c
>> > -config_parser_test_LDADD = libshared.la libtest-runner.la $(COMPOSITOR_LIBS)
> Hmm, I'm not sure why COMPOSITOR_LIBS were there in the first place...
> 

A quick note on that point.

COMPOSITOR_LIBS is there in order to bring in the implementation of wl_list.


-- 
Jon A. Cruz - Senior Open Source Developer
Samsung Open Source Group
jonc at osg.samsung.com