[Debian-med-packaging] Bug#922861: samtools: Outdated and missing documentation files

[Andreas Tille]

Hi John,

On Thu, Feb 21, 2019 at 04:12:18PM +0000, John Marshall wrote:
> On 21 Feb 2019, at 15:08, Andreas Tille <andreas at an3as.eu> wrote:
> > On Thu, Feb 21, 2019 at 01:17:54PM +0000, John Marshall wrote:
> >> NEWS.Debian.gz contains an ancient "samtools (1.3-1) unstable; urgency=medium" Debian changelog item.
> > 
> > Yes.  This is intended as well.  The file NEWS.Debian.gz is evaluated by
> > apt-listchanges[1] to inform local admins about changes that need to be
> > respected in a certain package version.
> 
> I see, and this is clearer now that I've read the text there more carefully. Thanks.
> 
> Scripts and users can still use "samtools bamshuf ..." btw -- the old name has not been removed, for obvious compatibility reasons.

Fixed in Git[1].  While this might not be visible any more for users who
upgrade from Debian 9 to the soon to be released Debian 10 this can be
relevant for those upgrading from Debian 8.  This seems to be a good
reason to push out the package now.
 
> >> The package is missing the upstream release's README file. It would be good to install this file, even though it doesn't currently contain a great deal of salient information. (The upstream release's INSTALL file is also missing, but that is of course appropriate for a binary package.)
> > 
> > Done in Git[5]
> 
> Thanks. Similar comments to the other bug about README* and README vs. README.md. However you should do what you see fit, and it may be immaterial if starting from the upstream release tarball as there will be only one file matching README*.

Yes, that's the idea.

Kind regards

         Andreas.

[1] https://salsa.debian.org/med-team/samtools/commit/d1997a0246ab6153410a142663b408e8fec3e0cc

-- 
http://fam-tille.de