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

Bug#788653: RFS: ktap/0.4-1 [ITP] -- lightweight script-based dynamic tracing tool for Linux



On Wed, Sep 09, 2015 at 06:42:29AM +0000, Gianfranco Costamagna wrote:
> >You are correct, there is no need in "linux-headers-amd64", however there
> 
> >is second option just "linux-headers", and I don't understand why it
> >doesn't work for you.
> 
> 
> it does work for *me*, it doesn't for buildd system.
> 
> IIRC they are configured to pick only the first version...

Okay, thanks for the information.

> >I've updated package at mentors.debian.net, can you take a look one more
> >time?
> 
> 
> sure :)
> 
> there is an issue I don't understand:
> help2man --no-discard-stderr -h-h --version-string $(VERSION) -n "$(DESCRIPTION)" ./ktap > debian/ktap.1
> echo "debian/ktap.1" >> debian/ktap.manpages
> 
> 
> this way if you rebuild the package twice
> $ cat debian/ktap.manpages 
> debian/ktap.1
> debian/ktap.1
> 
> 
> 
> 
> I guess this isn't what you want to achieve.
> 
> I'm fine with generating the documentation at build time, but I don't understand why to do the echo, instead
> of having the file there.
> debian/ktap.manpages doesn't change at build time, so why autogenerate it?

You are correct, I've put debian/ktap.manpages statically.

> also, if you create a "ktab.manpages" it is picked by dh_installmanpages, so the 

s/dh_installmanpages/dh_installman/

Otherwise there is an error about missing man page, since it is
generated at build time.

> dh_installdocs debian/doc/tutorial.html debian/ktap.1

Yep, thanks, I corrected. this.

> line is simply wrong (or duplicate)
> 
> I guess you want to override dh_installmanpages and dh_instaldocs separately
> 
> 
> 
> (the build was fine, so I guess the package is mostly ready now)

So, is there anything that I need/can to do to ship it to debian
repository?


Reply to: