Bug #125587 for PDL-CCS: reproducibility

Fri Jun 15 03:42:48 2018 laurent.baillet [...] gmail.com - Ticket created

Subject:	reproducibility
Date:	Fri, 15 Jun 2018 09:42:03 +0200
To:	bug-PDL-CCS [...] rt.cpan.org
From:	Laurent Baillet <laurent.baillet [...] gmail.com>

hello Bryan Here are some patches - enhancing reproducibility (reproduce.patch) - correcting warnings raised by the debian autopkgtest suite ( autopkgtest-(use|syntax).patch ) can you please have a look on them regards

Message body is not shown because sender requested not to inline it.

Fri Jun 15 04:21:00 2018 moocow [...] cpan.org - Taken

Fri Jun 15 09:17:42 2018 moocow [...] cpan.org - Correspondence added

hi Laurent, patches gratefully received & applied, uploaded to CPAN as v1.23.8. I understand the "reproducible" patch ($Data::Dumper::Sortkeys=1), but I'm curious about the 'syntax' and 'use' patches: are the missing ()s on (imported) function calls really a problem for any perl version in any currently active debian distribution? And why qualify PDL::Lite::barf() rather than importing it? (since I would like to maintain use of my carpal tunnels, I'd prefer not to have to hit the shift-key all too often). Please let me know if the new version works for you. marmosets, Bryan

Fri Jun 15 09:17:42 2018 The RT System itself - Status changed from 'new' to 'open'

Fri Jun 15 09:17:42 2018 moocow [...] cpan.org - Status changed from 'open' to 'patched'

Mon Jun 18 16:05:34 2018 laurent.baillet [...] gmail.com - Correspondence added

Subject:	Re: [rt.cpan.org #125587] reproducibility
Date:	Mon, 18 Jun 2018 22:04:38 +0200
To:	bug-PDL-CCS [...] rt.cpan.org
From:	Laurent Baillet <laurent.baillet [...] gmail.com>

hello and sorry for the late reply the "syntax" and "use" patches are here to resolve "problems" raised by autopkgtest (https://packages.debian.org/sid/pkg-perl-autopkgtest) . On https://perl-team.pages.debian.net/autopkgtest.html you can find the 3 types of tests run on perl packages (and so, use.t and syntax.t) : their aim and reasons better than I could explain :) Personaly (and professionaly) I'm not really in to this, but I like to use a combo of Perl::Critic, Modern::Perl and Test::Strict to raise such things. In fact, It seems (I am a newbie packager) that reproducibility is the new major goal of the Debian team. I forgot ! Are you a Debian user ? I'll have a look on the rest ! Again, sorry for bothering you with that Regards On Fri, Jun 15, 2018 at 3:18 PM Bryan Jurish via RT <bug-PDL-CCS@rt.cpan.org> wrote: Show quoted text

> <URL: https://rt.cpan.org/Ticket/Display.html?id=125587 > > > hi Laurent, > > patches gratefully received & applied, uploaded to CPAN as v1.23.8. > > I understand the "reproducible" patch ($Data::Dumper::Sortkeys=1), but I'm > curious about the 'syntax' and 'use' patches: are the missing ()s on > (imported) function calls really a problem for any perl version in any > currently active debian distribution? And why qualify PDL::Lite::barf() > rather than importing it? (since I would like to maintain use of my carpal > tunnels, I'd prefer not to have to hit the shift-key all too often). > > Please let me know if the new version works for you. > > marmosets, > Bryan >

Tue Jun 19 03:19:35 2018 moocow [...] cpan.org - Correspondence added

morning Laurent, On 2018-06-18 16:05:34, laurent.baillet@gmail.com wrote: Show quoted text

> the "syntax" and "use" patches are here to resolve "problems" raised > by autopkgtest

thanks for pointing me to the docs! Show quoted text

> I forgot ! Are you a Debian user ?

yup, since "woody"; many of our servers at work also run various releases of debian, which is why I was somewhat surprised e.g. by the 'use' patch: I don't see any warnings on STDERR from `use PDL::CCS` on any of our machines here with or without the patch. Show quoted text

> I'll have a look on the rest !

If you could point at a particular version (debian release + perl version) where the patch is required, I'd be grateful - thanks! Show quoted text

> Again, sorry for bothering you with that

no worries, thanks again for your efforts. marmosets, Bryan

Wed Feb 13 07:33:37 2019 moocow [...] cpan.org - Correspondence added

marking this as "resolved"

Wed Feb 13 07:33:38 2019 moocow [...] cpan.org - Status changed from 'patched' to 'resolved'

Wed Feb 13 07:33:38 2019 moocow [...] cpan.org - Fixed in 1.23.8 added