On 4/1/20 5:38 PM, Sebastien Jodogne wrote: > Package: wnpp > Severity: wishlist > Owner: Sebastien Jodogne <jodogne@orthanc-labs> > > * Package name : orthanc-python > Version : 1.0 > Upstream Author : Sebastien Jodogne <s.jodo...@gmail.com> > * URL : https://www.orthanc-server.com/ > * License : AGPL > Programming Lang: C++, Python > Description : Develop plugins for Orthanc using the Python programming > language > > This plugin can be used to write Orthanc plugins using the Python programming > language instead of the more complex C/C++ programming languages. It can be > used to gain access to Python modules directly in Orthanc. > > This plugin can be of great help to anyone wishing to automate her imaging > workflow, to design/train new machine learning algorithms, or to deploy AI > systems directly in clinical setups. > > Full documentation is available in the Orthanc Book: > https://book.orthanc-server.com/plugins/python.html >
Hi Sebastien, if this is a python module then it should follow python guidelines. The name in this case should be python3-orthanc, see LibraryStyleGuide: https://wiki.debian.org/Python/LibraryStyleGuide After building and installing orthanc-python I can't import orthanc from python3 most likely because python doesn't know where to find it. By following the python packaging guidelines this problem will be solved. I'd remove Resources/Builders/ via files-excluded in d/copyright since these files are not relevant for Debian as far as I see. There is no need in empty d/patches/ d/watch, is there need for +dfsg suffix ? If I run uscan --force-download it downloads and repacks tarball without dfsg suffix. Some files have trailing white space, which is a bit annoying, but this is already nitpicking :) - d/control - d/copyright Best regards, Alex