[Debian-med-packaging] Bug#958840: kmer: autopkgtest regression: No module named 'localAlignerInterface'

[Antoni Villalonga]

Hi,

For some reason I din't included all my previous work on my merge request.

I've recovered most of it from a binary package I've found on my machine. And
after lot of hours I've found the python module naming issue (see d/rules
changes).

All changes are now included on:
 https://salsa.debian.org/med-team/kmer/-/merge_requests/4

Sorry for the long delay and the childish mistake.

Best regrads,

PS: d/changelog is unmodified in my last MR. Probably should include a
reference to close this bug.

On Sat, May 16, 2020 at 07:37:54AM +0200, Andreas Tille wrote:
> Hi Antoni,
> 
> since you once dived into this which was interrupted when Salsa was
> offline:  Would you be able to finish this?
> 
> That would be really helpful.
> 
> Kind regards
> 
>       Andreas.
> 
> On Tue, Apr 28, 2020 at 08:43:55PM +0200, Andreas Tille wrote:
> > On Tue, Apr 28, 2020 at 04:43:30PM +0000, Antoni Villalonga wrote:
> > > 
> > > I think I've faced that problem and fixed.
> > > Fix should be included into '2to3.patch'
> > > 
> > > I think the relevant part is:
> > > 
> > > --- a/atac-driver/chainer/localalign/localAlignerInterfacemodule.C
> > > +++ b/atac-driver/chainer/localalign/localAlignerInterfacemodule.C
> > > @@ -227,8 +227,17 @@
> > >  };
> > >  
> > >  
> > > +static struct PyModuleDef cModMethods =
> > > +{
> > > +    PyModuleDef_HEAD_INIT,
> > > +    "localAlignerInterface",  /* name of module */
> > > +    "",          /* module documentation, may be NULL */
> > > +    -1,          /* size of per-interpreter state of the module, or -1 if the module keeps state in global variables. */
> > > +    registration_table
> > > +};
> > > +
> > >  extern "C"
> > > -void initlocalAlignerInterface() {
> > > -  Py_InitModule("localAlignerInterface", registration_table);
> > > +void PyInit_localAlignerInterface() {
> > > +  PyModule_Create(&cModMethods);
> > >  }
> > 
> > Thanks.  Looks promising.
> >  
> > > Sorry I can't access salsa due to a maintenance, also can't test autopkgtest
> > > for testing for the same reason.
> > 
> > Salsa and other hosts seem to be back now.
> >  
> > > It worked fine for sid some days ago.
> > 
> > Kind regards and thanks a lot for your contribution
> > 
> >       Andreas. 
> > 
> > -- 
> > http://fam-tille.de
> > 
> > _______________________________________________
> > Debian-med-packaging mailing list
> > Debian-med-packaging at alioth-lists.debian.net
> > https://alioth-lists.debian.net/cgi-bin/mailman/listinfo/debian-med-packaging
> 
> -- 
> http://fam-tille.de

-- 
Antoni Villalonga
https://friki.cat/