Bug#633749: [PyHealthcare] ANN: python-hl7 0.2.0
Andreas,
Putting this under the Debian Med team sounds good to me. I have only
worked with the Python Modules Team before, so I will review the
policies you referenced o make sure I package it properly. I will
send an RFS to the debian-med list when the package is ready for
review.
The git / unit test / license issues are something we need to resolve
upstream (another developer made some significant changes awhile back,
which we will try to merge back in). For now, I will package the
"mainline" version (http://pypi.python.org/pypi/hl7).
Thanks,
John
On Wed, Jul 13, 2011 at 7:53 AM, Andreas Tille <andreas@an3as.eu> wrote:
> Hi,
>
> thanks to Karsten for the forewarding!
>
> On Wed, Jul 13, 2011 at 02:09:08PM +0200, Karsten Hilbert wrote:
>> On Wed, Jul 13, 2011 at 06:49:27AM -0500, John Paulett wrote:
>>
>> > I just submitted an ITP to package python-hl7 for Debian.
>>
>> Thanks !
>>
>> I CCed the Debian Med mailing list so the relevant people
>> learn of this right away.
>
> In fact this was a good hint. John, I have no idea whether you know the
> Debian Med project[1] but if you feel good with team maintaining your
> ITPed package the Debian Med team would be happy to welcome you. In
> case you like this idea please read the Debian Med policy[2] which explains
> what this group maintenance means.
>
> For the moment I just added a paragraph about python-hl7 at the "Medical
> data" section of our tasks pages[3].
>
>> > A few things would need to happen to merge https://github.com/lkcl/hl7:
>> > - That repo is not a typical git/hg fork, but rather is a fresh git
>> > init based off of the 0.1.1 tar file. The changes would need to be
>> > rebased upon the mainline.
>
> I'm not sure whether this is a burning issue if you just want to package
> the current version. However, if this is a call for help we have Git
> experts lurking on this list (not me personally).
>
>> > - There are no unit tests that I can find (the ones that existed in
>> > the 0.1.1 code base are gone). I would ideally like to seem pretty
>> > high test coverage
>
> Hmmm, Unit tests would be a nice feature. In the ITP you are listed as
> maintainer *and* upstream - so what about reimplementing these tests
> into the new version? While this is also not urgent for packaging it
> makes prefectly sense to profit from tests.
>
>> > - It appears that 2k+ XML files from Mirth are included in the
>> > references/ folder. Mirth being an MPL license, I believe we need to
>> > discuss (at least document better) the impact of adding this to a
>> > BSD-licensed project.
>>
>> I see. Obviously I don't know enough about that code.
>
> As far as I know BSD and MPL license are compatible. So just mentioning
> this fact in the debian/copyright file should be sufficient.
>
> Kind regards and be assured that we will support your effort of
> packaging python-hl7
>
> Andreas.
>
>
> [1] http://www.debian.org/devel/debian-med
> [2] http://debian-med.alioth.debian.org/docs/policy.html
> [3] http://debian-med.alioth.debian.org/tasks/data
>
> --
> http://fam-tille.de
>
>
>
Reply to: