Hello Peter, El 01/07/21 a las 08:41, Peter Collinson escribió: > I’ve revised the uploaded package because it failed to build on sbuild. > > It needed a dependency on netbase, and a change to a test script which just > happened to work on Python 3.7 but failed on Python3.9. > > > Regards > -------------------------------------------------- > Peter Collinson
Thanks for your work developing nftfw and packaging it for debian! Here you have some comments: * you have a master branch (that includes the debian/ dir). It is a common practice to have the debian packaging branch separated from the upstream code (including the tarball). See the candidate DEP-14. https://dep-team.pages.debian.net/deps/dep14/ * the watch file stores the downloaded upstream tarball as: nftfw_0.9.4.orig.tar.gz -> '<project>-0.9.4.tar.gz' '<project>-0.9.4.tar.gz' you can try with `uscan --verbose --download --download-current-version` Also, the .orig.tar.gz I can download with your .dsc is different from the .orig.tar.gz I download using uscan. It seems you have removed the debian/ dir. * do you have any reason for not using salsa? (it is not mandatory, I am just curious). Keep in mind that using Salsa would make it easier to make use of common Debian tools, such as Salsa CI: https://salsa.debian.org/salsa-ci-team/pipeline/ * building the package modifies debian/postinst (by the make_etc_lists). Could you please explain what is the goal of make_etc_lists? Why do you need it? Cheers, -- Santiago
signature.asc
Description: PGP signature