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)
Bugzilla
Testing Suite
Tracking
()
RESOLVED
FIXED
Bugzilla 3.6
People
(Reporter: szabgab, Assigned: szabgab)
References
Details
Attachments
(1 file)
|
689 bytes,
patch
|
mkanat
:
review+
|
Details | Diff | Splinter Review |
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
Updated•16 years ago
|
Assignee: testing → szabgab
Severity: normal → minor
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Linux → All
Hardware: x86_64 → All
| Assignee | ||
Comment 1•16 years ago
|
||
passing values of PERL5LIB as -I on the command line
Attachment #426585 -
Flags: review?
Updated•16 years ago
|
Attachment #426585 -
Flags: review? → review+
Comment 2•16 years ago
|
||
Comment on attachment 426585 [details] [diff] [review]
v1_allow_perl5lib.diff
Looks reasonable. I don't think the "use Config" is necessary though, right?
Updated•16 years ago
|
Flags: approval3.6+
Flags: approval+
Target Milestone: --- → Bugzilla 3.6
| Assignee | ||
Comment 3•16 years ago
|
||
As I could see it is necessary to make the code platform independent.
Comment 4•16 years ago
|
||
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?
| Assignee | ||
Comment 5•16 years ago
|
||
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 (;).
Comment 6•16 years ago
|
||
Ahh, okay. That makes sense.
Comment 7•16 years ago
|
||
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
You need to log in
before you can comment on or make changes to this bug.
Description
•