Bug #104089 for Devel-OverrideGlobalRequire: Add original caller information to callback

Wed Apr 29 05:30:24 2015 NIERLEIN [...] cpan.org - Ticket created

Subject:

Add original caller information to callback

Hi, it would be great to include the original caller information in the callback. See the attached patch. Cheers, Sven

Subject:

0001-add-caller.patch

--- a/lib/Devel/OverrideGlobalRequire.pm 2015-04-29 11:28:01.000000000 +0200 +++ b/lib/Devel/OverrideGlobalRequire.pm 2015-04-29 11:21:00.000000000 +0200 @@ -129,7 +129,7 @@ return $res; - }, $arg_copy); + }, $arg_copy, [caller]); } }

Wed Apr 29 05:35:44 2015 ribasushi [...] leporine.io - Correspondence added

Subject:	Re: [rt.cpan.org #104089] Add original caller information to callback
Date:	Wed, 29 Apr 2015 11:35:23 +0200
To:	bug-Devel-OverrideGlobalRequire [...] rt.cpan.org
From:	Peter Rabbitson <ribasushi [...] cpan.org>

On 04/29/2015 11:30 AM, Sven Nierlein via RT wrote: Show quoted text

> Wed Apr 29 05:30:24 2015: Request 104089 was acted upon. > Transaction: Ticket created by NIERLEIN > Queue: Devel-OverrideGlobalRequire > Subject: Add original caller information to callback > Broken in: 0.001 > Severity: Wishlist > Owner: Nobody > Requestors: NIERLEIN@cpan.org > Status: new > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=104089 > > > > Hi, > > it would be great to include the original caller information in the callback. > See the attached patch.

Can you explain this need in more detail? You are already providing a coderef to the API, you can invoke caller(1) within this coderef yourself... What am I missing?

Wed Apr 29 05:35:44 2015 The RT System itself - Status changed from 'new' to 'open'

Wed Apr 29 05:38:43 2015 NIERLEIN [...] cpan.org - Correspondence added

You are right, didn't know thats possible.

Wed Apr 29 05:38:55 2015 NIERLEIN [...] cpan.org - Status changed from 'open' to 'resolved'