[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: [MoM] r-cran-nnls: please fix copyright



Hi Alba,

On Thu, May 21, 2015 at 02:12:59PM +0100, Alba Crespi wrote:
> > That's a quite successful MoM project with currently two uploads.
> > Thanks for your contribution and lets hope that new processing speed
> > will increase a bit
> 
> Now 3. I've just pushed r-cran-phangorn.

Cool.

> I hope I'm not giving you too much work ;-P

No, you are not. ;-)

There are some missing dependencies:

...
ERROR: dependencies 'ape', 'quadprog', 'igraph', 'Matrix', 'nnls' are not available for package 'phangorn'

which you stumble upon when building in a chroot.  We know that ipraph
and nnls are in the new queue but please adjust the Build-Depends and
Depends of the package to be ready once the packages are available.

Since we can not yet upload this package I used "UNRELEASED" as target
distribution and pushed this change.  This is to make sure we will not
assume that this package was really uploaded when just looking at the
Git repository.

Moreover there is a related publication for this package which is
mentioned in inst/CITATION.  You even mention this in the long
description but this is not the correct place to do this since it is not
machine parseable.  Please instead create a file

   debian/upstream/metadata

You can easily find a template for this in

   svn export svn://anonscm.debian.org/svn/debian-med/trunk/package_template debian

The documentation for this file is here:

   https://wiki.debian.org/UpstreamMetadata

I personally insist only on the Reference field - all else can
be left out.

Finally I removed debian/files since it is an auto generated
file and should not be in Git.  Please pull.

Thanks for your work so far and do not hesitate to commit more. ;-)

BTW, if you (or somebody else) feel(s) the obligation on updating some
of our R packages - everybody in the team is more than welcome to push
updates to Git and I'll simply upload them.  :-)

Kind regards

        Andreas.

-- 
http://fam-tille.de


Reply to: