Closed Bug 545762 Opened 16 years ago Closed 16 years ago

cannot run tests when using PERL5LIB (e.g. when using local::lib)

Categories

(Bugzilla :: Testing Suite, defect)

defect
Not set
minor

Tracking

()

RESOLVED FIXED
Bugzilla 3.6

People

(Reporter: szabgab, Assigned: szabgab)

References

Details

Attachments

(1 file)

User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.7) Gecko/20100106 Ubuntu/9.10 (karmic) Firefox/3.5.7 Build Identifier: When trying to run ./runtests.pl the test t/001compile.t blows up as I am using local::lib and I have PERL5LIB configured to locate the privately installed modules. PERL5LIB is disregarded due to the use of -T in the tests script. Reproducible: Always Steps to Reproduce: 1. run ./runtests.pl Actual Results: Cannot find various CPAN modules
Assignee: testing → szabgab
Severity: normal → minor
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Linux → All
Hardware: x86_64 → All
passing values of PERL5LIB as -I on the command line
Attachment #426585 - Flags: review?
Attachment #426585 - Flags: review? → review+
Comment on attachment 426585 [details] [diff] [review] v1_allow_perl5lib.diff Looks reasonable. I don't think the "use Config" is necessary though, right?
Flags: approval3.6+
Flags: approval+
Target Milestone: --- → Bugzilla 3.6
As I could see it is necessary to make the code platform independent.
Oh, are there platforms where $ENV{PERL5LIB} won't exist if you don't use Config? That doesn't make sense, since it's an environment variable, right?
I have to split PERL5LIB on $Config{path_sep} and not just on double-colon (:) as I did in the patch I sent to the mailing list to make it work on other platforms as well. On Windows it is semi-colon (;).
Ahh, okay. That makes sense.
Thanks for the patch! :-) Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/trunk/ modified t/001compile.t Committed revision 6989. Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/3.6/ modified t/001compile.t Committed revision 6974.
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Blocks: 714074
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: