[SCM] ardour3/master: Depend on libcwiid-dev and libcppunit-dev >= 1.12.0

[Kaj Ailomaa]

On Sat, Jun 15, 2013, at 09:44 PM, Adrian Knoth wrote:
> On 06/14/2013 06:08 PM, zequence-guest at users.alioth.debian.org wrote:
> 
> Hi!
> 
> (temporary online while travelling)
> 
> > diff --git a/debian/control b/debian/control
> > index feb428f..7bc692b 100644
> > --- a/debian/control
> > +++ b/debian/control
> > @@ -12,7 +12,9 @@ Build-Depends: cdbs,
> >   gettext,
> >   intltool,
> >   libboost-dev (>= 1.49.0),
> > + libcppunit-dev (>= 1.12.0),
> >   libcurl4-gnutls-dev (>= 7.25.0),
> > + libcwiid-dev,
> >   libfftw3-dev (>= 3.3.1),
> >   libraptor2-dev (>= 2.0.9),
> >   librdf0-dev (>= 1.0.15),
> 
> 
> debian/control is autogenerated[0] by CDBS. Please specify the
> dependencies in debian/rules.

Thanks. I was wondering how that was done.

> 
> Also, only change a single dependency per commit[1] and ideally provide
> a rationale why it's needed.
> 
> Why do we need libcppunit-dev? We don't run unittests on the build
> bots.
> 

I simply added them to enable the build, as it was failing before. 
I realize now that it will build passing the -d option to
dpkg-buildpackage.
So, an override needed in debian/rules for those?

I have no opinion on the two dependencies one way or another, and sorry
to be messing up the package if I were.

> 
> 
> Cheers
> 
> [0] I have:
> alias gencontrol='DEB_AUTO_UPDATE_DEBIAN_CONTROL=yes fakeroot
> debian/rules clean'

Thanks. I was wondering how that worked.

> [1] As mentioned in
> http://wiki.debian.org/DebianMultimedia/DevelopPackaging

I will continue to read that and many other online resources :).