Hi Andreas,

Thanks for the notice that pytz is already on Debian.

And thanks so much for the patch :) I will continue working on that and
trying to
fix the issue.

I will keep you updated

Cheers,
Arias Emmanuel
@eamanu
yaerobi.com


On Wed, Apr 15, 2020 at 12:24 PM Andreas Tille <andr...@an3as.eu> wrote:

> Hi Emmanuel,
>
> I've seen your ITP of idseq-dag.  You mentioned
>
>    idseq-dag depends on pytz. I will send the ITP for that package.
>
> I admit its a bit confusing but that package is python3-tz in Debian, is
> packaged and I've added it to Build-Depends (amongst other changes).
>
> The current packaging is suffering from failures inside the test suite
> looking like this:
>
>
> ======================================================================
> ERROR: test_step_single
> (tests.generate_alignment_viz.GenerateAlignmentVizTest)
> ----------------------------------------------------------------------
> Traceback (most recent call last):
>   File
> "/build/idseq-dag-4.2.3/.pybuild/cpython3_3.8_idseq-dag/build/tests/generate_alignment_viz.py",
> line 9, in test_step_single
>     run_step =
> IdseqStepSetup.get_step_object(PipelineStepGenerateAlignmentViz,
> "alignment_viz_out", paired=False)
>   File
> "/build/idseq-dag-4.2.3/.pybuild/cpython3_3.8_idseq-dag/build/tests/idseq_step_setup.py",
> line 47, in get_step_object
>     PipelineFlow.fetch_input_files_from_s3(
>   File
> "/build/idseq-dag-4.2.3/.pybuild/cpython3_3.8_idseq-dag/build/idseq_dag/engine/pipeline_flow.py",
> line 179, in fetch_input_files_from_s3
>     output_file = idseq_dag.util.s3.fetch_from_s3(s3_file, local_dir,
> allow_s3mi=True)
>   File
> "/build/idseq-dag-4.2.3/.pybuild/cpython3_3.8_idseq-dag/build/idseq_dag/util/s3.py",
> line 286, in fetch_from_s3
>     if is_reference or os.path.abspath(dst).startswith(config["REF_DIR"]):
> TypeError: startswith first arg must be str or a tuple of str, not NoneType
>
>
> The issue is that I had to replace some default dir in a patch[1].
> Unfortunately this did not propagated into the config dictionary and
> the key "REF_DIR" is not set.  For the moment I gave up here - may
> be you want to continue by either fixing my patch or finding a better
> way to convince the test suite to write to some place where this is
> permitted.
>
> Kind regards
>
>        Andreas.
>
> [1]
> https://salsa.debian.org/med-team/idseq-dag/-/blob/master/debian/patches/set_default_output_dir_var_tmp.patch
>
> On Sat, Apr 11, 2020 at 12:38:13AM -0300, Emmanuel Arias wrote:
> > Hello everybody,
> >
> > I am available to help. I will be working on the TODO list [0].
> >
> > [0]
> >
> https://salsa.debian.org/med-team/community/2020-covid19-hackathon/-/wikis/COVID-19-Hackathon-packages-needing-work
> >
> > Cheers,
> > Arias Emmanuel
> > @eamanu
> > yaerobi.com
> >
> >
> > On Fri, Apr 10, 2020 at 3:10 AM Olek Wojnar <o...@debian.org> wrote:
> >
> > > On 4/9/20 7:37 PM, fancycade wrote:
> > > > Thank you for the assistance with this!
> > > >
> > > > I have uploaded my package to the appropriate repository under the
> > > > Python Modules team and will reach out for a RFS.
> > >
> > > Great, thanks for your contribution!
> > >
> > > -Olek
> > >
> > >
> > >
> > >
>
> --
> http://fam-tille.de
>

Reply via email to