[SCM] ardour/master: Modernize cdbs: Do copyright-check in maintainer script (not during build).

[Jaromír Mikeš]

2017-08-21 11:35 GMT+02:00 Jonas Smedegaard <dr at jones.dk>:

Hi Jonas,

Quoting Jaromír Mikeš (2017-08-21 08:43:02)
> > 2017-07-31 8:25 GMT+02:00 <js at users.alioth.debian.org>:
> >
> > > The following commit has been merged in the master branch:
> > > commit c48460d12c633074a6aae6b572d0d805b3433667
> > > Author: Jonas Smedegaard <dr at jones.dk>
> > > Date:   Mon Jul 31 02:10:16 2017 -0400
> > >
> > >     Modernize cdbs: Do copyright-check in maintainer script (not during
> > > build).
> > >
> > > diff --git a/debian/README.source b/debian/README.source
> > > index 684ef50..2146866 100644
> > > --- a/debian/README.source
> > > +++ b/debian/README.source
> > > @@ -48,6 +48,9 @@ You are of course free to *not* run the script, if
> you
> > > prefer.
> > >
> > >  copyright_newhints
> > >  ------------------
> > > -run this command to generade copyright_newhints file:
> > > -licensecheck --check '.*' --recursive --copyright --deb-fmt --ignore
> > > '^(waf|(icons|gtk2_ardour)/icons/.*\.png|tools/osx_
> > > packaging/.*\.png|gtk_ardour/.*splash\.png|doc/ardour_meter_
> > > colors\.png|doc/layering/.*\.png|icons/made_with/ardour_
> > > made\.png|debian/(changelog|copyright(|_hints|_newhints)))' --lines 0
> * |
> > > /usr/lib/cdbs/licensecheck2dep5 > debian/copyright_newhints
> > >
> > >
> > This is not right way to generate copyright_newhints now?
> > If not which one?
>
> I won't label them as "right" and "wrong" ways, but the approach I use
> nowadays is the one expressed by the commit you quote: To do
> copyright-check in a maintainer script instead of during build.
>
> Not sure what more you are asking - I believe it is all part of the git
> commit.


I might be wrong, but d/copyright-check script doesn't generate
copyright_newhints
for me :(

mira
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.alioth.debian.org/pipermail/pkg-multimedia-maintainers/attachments/20170821/3fa04ae6/attachment.html>