On Thu, 2013-04-04 at 14:05 +0200, Daniel Wagner wrote:
> Hi Jukka,
> 
> On 04/04/2013 01:46 PM, Jukka Rissanen wrote:
> > ---
> >   Makefile.am          |   6 +-
> >   unit/test-dnsproxy.c | 463 
> > +++++++++++++++++++++++++++++++++++++++++++++++++++
> >   2 files changed, 468 insertions(+), 1 deletion(-)
> >   create mode 100644 unit/test-dnsproxy.c
> >
> > diff --git a/Makefile.am b/Makefile.am
> > index 78b1b33..c4c72bd 100644
> > --- a/Makefile.am
> > +++ b/Makefile.am
> > @@ -240,7 +240,8 @@ client_connmanctl_SOURCES =  $(gdbus_sources) 
> > src/connman.h \
> >   client_connmanctl_LDADD = @DBUS_LIBS@ @GLIB_LIBS@ -lreadline -ldl
> >   endif
> >
> > -noinst_PROGRAMS += unit/test-pbkdf2-sha1 unit/test-prf-sha1 
> > unit/test-ippool
> > +noinst_PROGRAMS += unit/test-pbkdf2-sha1 unit/test-prf-sha1 
> > unit/test-ippool \
> > +           unit/test-dnsproxy
> 
> If I read this patch correct, test-dnsproxy expects that the ConnMan is 
> running in the background. I think in this case it should be part of tools.

Yep, this should go into /tools.

Cheers,

        Patrik

_______________________________________________
connman mailing list
[email protected]
http://lists.connman.net/listinfo/connman

Reply via email to