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

Re: [RFS] and peer review: mosdepth



On 03.11.20 23:23, Nilesh Patra wrote:
>
>
> On Wed, 4 Nov, 2020, 3:24 am Steffen Möller, <steffen_moeller@gmx.de
> <mailto:steffen_moeller@gmx.de>> wrote:
>
>     It took me a while but at some point I have understood that the
>     package
>     was already uploaded. I added the -d:release and added some bits to
>     auto_clean.
>
>
> Thanks!
>
>
>     Now,
>
>     dpkg-buildpackage && ./debian/rules clean
>
>     will bring this git status:
>
>     Changes not staged for commit:
>      (use "git add/rm <file>..." to update what will be committed)
>      (use "git restore <file>..." to discard changes in working directory)
>            deleted:    tests/missing.bed
>
>
> That's because of this line I added to d/rules[1]
> This is because missing.bed is an empty file and installing it isn't
> sensible - which a lintian warning signalled as well.
>
> [1]: https://salsa.debian.org/med-team/mosdepth/-/blob/master/debian/rules#L18
> <https://salsa.debian.org/med-team/mosdepth/-/blob/master/debian/rules#L18>
>
>
>     I added this to d/copyright's Files-Excluded - was that correct?
>
>
> As of now, no - since I didn't repack this, just removed it during
> testing - I thought that some tests are using this file but I was
> wrong to assume so.
>
> Repacking this again would mean switching to a +dfsg.1 version - maybe
> adding this to files-excluded with the next upstream release could be good

That is how it was meant. Routine-update should read this now and then
prepare the next version without that file.

I am not sure if that is the right thing, though. If there is a test
against an empty file then this is what it should be and we should
override the lintian warning.

I am more than happy with mosdepth for now and will next address the
missing packages for the hts-nim-tools. Many thanks!

Steffen






Reply to: