531107 - [Windows] Enabling a extension results in "deep recursion on subroutine" warnings

Status: RESOLVED FIXED

(Bugzilla :: Extensions, defect)

Description

[Michael Thomas (Mockodin)]

Attachment: Perl Command Line Interpreter has stopped working

Also Applies to Windows 7 (and likely Windows Vista)

Code Base: CVS TIP 3.5.2+

Reproducible Always

To Reproduce:
1. delete extensions/Example/disabled
2. Run checksetup or if in a mod_perl environment apache -t

Results:
Deep recursion on subroutine "Bugzilla::Extension::__do_call" at Bugzilla/Extension.pm line 189.
Deep recursion on subroutine "Bugzilla::Extension::__do_call" at Bugzilla/Extension.pm line 159.
Deep recursion on anonymous subroutine at C:/usr/lib/File/Spec/Win32.pm line 92.
Deep recursion on subroutine "Bugzilla::Extension::__do_call" at Bugzilla/Extension.pm line 153.
Deep recursion on subroutine "Bugzilla::Extension::my_inc" at Bugzilla/Extension.pm line 210.
Deep recursion on subroutine "File::Spec::Win32::case_tolerant" at Bugzilla/Extension.pm line 166.

Followed by windows "Perl Command Line Interpreter has stopped working"

Notes:
If no extensions are enable, no errors occur.

Comment 1

[Frédéric Buclin]

I can reproduce this problem on Windows 7, but not on Linux.

Comment 2

[Max Kanat-Alexander]

Yeah, I can reproduce it too. It's probably somehow related to the path stuff that Extensions do. I'll look at it tonight or tomorrow if I get the chance.

Comment 3

[Max Kanat-Alexander]

Attachment: v1

Okay, this fixes it in my testing. mockodin, would you let me know if it fixes it for you? I think that since I'm the Extensions owner, that will be enough review to check in this fix.

Comment 4

[Michael Thomas (Mockodin)]

Looks good. Apache is now starting properly. Thanks MKanat.

Comment 5

[Max Kanat-Alexander]

Checking in Bugzilla/Extension.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/Extension.pm,v  <--  Extension.pm
new revision: 1.8; previous revision: 1.7
done

Comment 6

[Frédéric Buclin]

This bug has no approval+, for the record. ;)

Comment 7

[Max Kanat-Alexander]

Oh, thanks, LpSolit. :-)