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

Re: [RFS] and peer review: mosdepth



Hi Andreas,

On Tue, 3 Nov 2020 at 16:32, Andreas Tille <andreas@an3as.eu> wrote:
Hi Étienne and Steffen,

On Tue, Nov 03, 2020 at 11:48:41AM +0100, Étienne Mollier wrote:
> > Exciting times, nim-docopt and nim-hts made it through new within the
> > same day, so we now have some first important fruit from these nim
> > dependencies: mosdepth!
> >
> > https://salsa.debian.org/med-team/mosdepth
> >
> > This is a major show stopper on the way to get bcbio complete. There are
> > some more nim modules missing for the hts-nim-tools that are also a dep
> > of bcbio - but .. .we are getting closer. Much closer :)
>
> To answer to the "peer review" aspect, running lintian against
> mosdepth_0.3.1-1.dsc got me at least:
>
>       I: mosdepth source: unused-file-paragraph-in-dep5-copyright paragraph at line 9
>       I: mosdepth source: wildcard-matches-nothing-in-dep5-copyright intToStr.nim (line 9)

Wow, how did you build the package?  I just pushed fixed Build-Depends
and needed to set HOME to get a pbuilder build.  I'm now running int

...
help2man mosdepth > debian/mosdepth.1
help2man: can't get `--help' info from mosdepth
Try `--no-discard-stderr' if option outputs to stderr
make[1]: *** [debian/rules:14: override_dh_auto_build] Error 127

Looks like we ended up with race conditions here - Fixed these. Please consider to $gbp pull :)

Best,
Nilesh

Reply to: