Skip Menu |

This queue is for tickets about the Devel-bt CPAN distribution.

Report information
The Basics
Id: 93585
Status: open
Priority: 0/
Queue: Devel-bt

People
Owner: Nobody in particular
Requestors: dom [...] cpan.org
Cc: gregoa [...] cpan.org
LEONT [...] cpan.org
AdminCc:

Bug Information
Severity: (no value)
Broken in: 0.06
Fixed in: (no value)



Subject: Test failures on armel, kfreebsd-amd64, mips
The following test failures on the above archs were found in Debian in https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=721421: t/release-no-tabs.t ..... skipped: these tests are for release candidate testing t/release-pod-syntax.t .. skipped: these tests are for release candidate testing Test Summary Report ------------------- t/basic.t (Wstat: 1792 Tests: 7 Failed: 7) Failed tests: 1-7 Non-zero exit status: 7 Files=4, Tests=7, 20 wallclock secs ( 0.48 usr 0.16 sys + 4.52 cusr 1.34 csys = 6.50 CPU) Result: FAIL Failed 1/4 test programs. 7/7 subtests failed. make[1]: *** [test_dynamic] Error 255 make[1]: Leaving directory `/«PKGBUILDDIR»' dh_auto_test: make -j1 test returned exit code 2 make: *** [build-arch] Error 2 Cheers, Dominic.
Subject: [rt.cpan.org #93585] Additional build failures
Date: Tue, 25 Mar 2014 15:42:47 +0000
To: bug-Devel-bt [...] rt.cpan.org
From: Daniel Lintott <daniel [...] serverb.co.uk>
In addition to the above architectures, Devel::bt also fails to build on hurd-i386. Attached is a log with the output from the build process. Following a discussion on debian-hurd@lists.debian.org the test is getting a stack trace of the signal handling thread... as opposed the thread which raised the signal [0]. [0] https://lists.debian.org/debian-hurd/2014/03/msg00061.html -- Daniel Lintott on behalf of the Debian Perl Group GPG Key: 4096R/5D73EC6E

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

Download signature.asc
application/pgp-signature 899b

Message body not shown because it is not plain text.

Dear Maintainer, The Debian perl group is reviewing packages with bugs which make them un-releasable; in particular when they are not heavily used by Debian users. We would like to remove such modules from Debian if we don't think they are likely to be fixed. Devel::BT is one such module, owing to this bug, and we would like to know whether you have any plans to look at the bug in the foreseeable future before we remove the package from Debian. If we don't hear anything we will remove the package from Debian on or around 19th September. This of course does not affect the standing of your module on CPAN. Thank you for maintaining this module so far!
On 2014-08-28 20:49:53, DOM wrote: Show quoted text
> Dear Maintainer, > > The Debian perl group is reviewing packages with bugs which make them > un-releasable; in particular when they are not heavily used by Debian > users. We would like to remove such modules from Debian if we don't > think they are likely to be fixed. > > Devel::BT is one such module, owing to this bug, and we would like to > know whether you have any plans to look at the bug in the foreseeable > future before we remove the package from Debian. > > If we don't hear anything we will remove the package from Debian on or > around 19th September. This of course does not affect the standing of > your module on CPAN. > > Thank you for maintaining this module so far!
11:49 <ether> did you see this? https://rt.cpan.org/Ticket/Display.html?id=93585 12:32 <rafl> i did just now for the first time, and i'm not entirely sure how i feel about it 12:33 * rafl ponders 12:36 <rafl> so, that module is really just a debugging aid for developers and end users 12:36 <rafl> the initial motivation to write it was people coming to me with thing segfaulting and me needing a backtrace to make any sense of it 12:37 <rafl> but repeatedly walking users through how to use gdb was a big hassle, so i automated it 12:38 <rafl> i have a hard time believing there are developers or endusers on mips, armel, and kfreebsd-amd64 12:38 <ether> heh 12:38 <rafl> that's probably more true for armel and mips than for kfreebsd, but still 12:38 <rafl> so i think this would be useful to keep for the architectures it works on 12:39 <rafl> cause realistically everyone uses amd64 linux 12:39 <rafl> i also think there's a lot of value to having this in debian 12:40 <rafl> cause saying "apt-get install libdevel-bt-perl" is a lot easier than walking a newcomer through the four dozen ways of installing modules 12:40 <rafl> that's kind of why i asked the perl group to package it in the first place 12:41 <rafl> i totally would like to get it working on all platforms, just for the sake of it, but i'm not sure if that's gonna be effort well spent 12:41 <rafl> and debian does have ways of providing packages only for certain architectures 12:41 <ether> sounds like they should only package it for those architectures where tests pass, then 12:42 <ether> or the Makefile.PL can do that check and exit otherwise? 12:42 <rafl> well.. 'make test' already bails out if things don't work. i think it'd be better to not hardcode any platform names in Makefile.PL 12:43 <rafl> but yeah. if debian could limit the availability of that package to the platforms it's known to work on, i think that'd be ideal 12:43 <rafl> though i haven't really been involved much in debian recently, so i'm not sure what the policy on doing that is 12:43 <rafl> might be that they only do it for things that are truly platform dependant 12:44 <rafl> stuff like grub or yaboot, for example 12:45 <rafl> i also know that Leon has been working on a similar module that's using a very different implementation technique 12:45 <rafl> rather than spawning gdb he's walking the c call stack and symbol tables and whatnot manually 12:45 <rafl> which may or may not be more portable 12:46 <rafl> it might be a good idea to investigate if his module provides something functionally equivalent that Devel::bt could be deprecated in favour of 12:46 <rafl> i really just want the functionality to be there and easily available to relatively unskilled users - i don't really care if that functionality is provided by Devel::bt or something else 12:47 <ether> should I paste this conversation into the ticket and show it to leont? 12:47 <rafl> that'd be really nice of you! 12:52 <ether> kk!
On Thu Mar 06 17:48:56 2014, DOM wrote: Show quoted text
> The following test failures on the above archs were found in Debian in > > https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=721421: > > t/release-no-tabs.t ..... skipped: these tests are for release > candidate testing > t/release-pod-syntax.t .. skipped: these tests are for release > candidate testing > > Test Summary Report > ------------------- > t/basic.t (Wstat: 1792 Tests: 7 Failed: 7) > Failed tests: 1-7 > Non-zero exit status: 7 > Files=4, Tests=7, 20 wallclock secs ( 0.48 usr 0.16 sys + 4.52 cusr > 1.34 csys = 6.50 CPU) > Result: FAIL > Failed 1/4 test programs. 7/7 subtests failed. > make[1]: *** [test_dynamic] Error 255 > make[1]: Leaving directory `/«PKGBUILDDIR»' > dh_auto_test: make -j1 test returned exit code 2 > make: *** [build-arch] Error 2 > > Cheers, > Dominic.
The attached patch might fix the issue on Hurd, I can't really say much about the issue on armel or kfreebsd-amd64 without having some build/test output from them though. As for the latter: there might be a kernel security setting interfering, I vaguely remember such an issue op OpenBSD at least. Leon
Subject: 0001-Raise-instead-of-kill-the-signal.patch
From 7243c7acfa7731697dfd75e930906817588c9c2f Mon Sep 17 00:00:00 2001 From: Leon Timmermans <fawaka@gmail.com> Date: Mon, 1 Sep 2014 11:53:23 +0200 Subject: [PATCH] Raise instead of kill the signal --- t/basic.t | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/t/basic.t b/t/basic.t index 4519941..95fbb90 100644 --- a/t/basic.t +++ b/t/basic.t @@ -18,7 +18,7 @@ local $ENV{PERL5LIB} = join $Config::Config{path_sep} => @INC; for my $signal (@signals) { next unless __PACKAGE__->can($signal); my $signum = __PACKAGE__->can($signal)->(); - my @cmd = ($^X, qw(-d:bt -e), "kill $signum, \$\$"); + my @cmd = ($^X, qw(-d:bt -MPOSIX=raise -e), "raise($signum)"); use Capture::Tiny 'capture'; my ($stdout) = capture { system @cmd }; -- 1.9.1
CC: 721421 [...] bugs.debian.org, debian-mips [...] lists.debian.org, debian-arm [...] lists.debian.org
Subject: Re: [rt.cpan.org #93585] Test failures on armel, kfreebsd-amd64, mips
Date: Sun, 21 Sep 2014 18:50:26 +0100
To: Karen Etheridge via RT <bug-Devel-bt [...] rt.cpan.org>
From: Dominic Hargreaves <dom [...] earth.li>
[Adding mips and armhf porters; please see <https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=721421> for context. On Fri, Aug 29, 2014 at 03:53:52PM -0400, Karen Etheridge via RT wrote: Show quoted text
> 11:49 <ether> did you see this? https://rt.cpan.org/Ticket/Display.html?id=93585 > 12:32 <rafl> i did just now for the first time, and i'm not entirely sure how i feel about it > 12:33 * rafl ponders > 12:36 <rafl> so, that module is really just a debugging aid for developers and end users > 12:36 <rafl> the initial motivation to write it was people coming to me with thing segfaulting and me needing a backtrace to make any sense of it > 12:37 <rafl> but repeatedly walking users through how to use gdb was a big hassle, so i automated it > 12:38 <rafl> i have a hard time believing there are developers or endusers on mips, armel, and kfreebsd-amd64 > 12:38 <ether> heh > 12:38 <rafl> that's probably more true for armel and mips than for kfreebsd, but still > 12:38 <rafl> so i think this would be useful to keep for the architectures it works on > 12:39 <rafl> cause realistically everyone uses amd64 linux > 12:39 <rafl> i also think there's a lot of value to having this in debian > 12:40 <rafl> cause saying "apt-get install libdevel-bt-perl" is a lot easier than walking a newcomer through the four dozen ways of installing modules > 12:40 <rafl> that's kind of why i asked the perl group to package it in the first place > 12:41 <rafl> i totally would like to get it working on all platforms, just for the sake of it, but i'm not sure if that's gonna be effort well spent > 12:41 <rafl> and debian does have ways of providing packages only for certain architectures > 12:41 <ether> sounds like they should only package it for those architectures where tests pass, then > 12:42 <ether> or the Makefile.PL can do that check and exit otherwise? > 12:42 <rafl> well.. 'make test' already bails out if things don't work. i think it'd be better to not hardcode any platform names in Makefile.PL > 12:43 <rafl> but yeah. if debian could limit the availability of that package to the platforms it's known to work on, i think that'd be ideal > 12:43 <rafl> though i haven't really been involved much in debian recently, so i'm not sure what the policy on doing that is > 12:43 <rafl> might be that they only do it for things that are truly platform dependant > 12:44 <rafl> stuff like grub or yaboot, for example > 12:45 <rafl> i also know that Leon has been working on a similar module that's using a very different implementation technique > 12:45 <rafl> rather than spawning gdb he's walking the c call stack and symbol tables and whatnot manually > 12:45 <rafl> which may or may not be more portable > 12:46 <rafl> it might be a good idea to investigate if his module provides something functionally equivalent that Devel::bt could be deprecated in favour of > 12:46 <rafl> i really just want the functionality to be there and easily available to relatively unskilled users - i don't really care if that functionality is provided by Devel::bt or something else > 12:47 <ether> should I paste this conversation into the ticket and show it to leont? > 12:47 <rafl> that'd be really nice of you! > 12:52 <ether> kk!
Hi Karen, Thanks for fowarding this! Some more details have been discussed on the Debian BTS, and I've just asked Leon if the new logs are any use for debugging. Debian policy has this to say on the matter: "Specifying a list of architectures or architecture wildcards other than any is for the minority of cases where a program is not portable or is not useful on some architectures. Where possible, the program should be made portable instead."[1] I don't think excluding mips and armhf (the two architectures where it used to work) would be an absolute no-go area but it would be frowned upon. I'm also CCing the Debian arm and mips porters, in case they can help. Cheers, Dominic. [1] <https://www.debian.org/doc/debian-policy/ch-controlfields.html#s-f-Architecture>