Closed
Bug 480968
Opened 15 years ago
Closed 14 years ago
Error message popup on Windows from Oracle module on every checksetup.pl run
Categories
(Bugzilla :: Installation & Upgrading, defect, P1)
Tracking
()
RESOLVED
FIXED
Bugzilla 3.4
People
(Reporter: Wurblzap, Assigned: mkanat)
References
()
Details
Attachments
(2 files)
1.76 KB,
patch
|
Wurblzap
:
review+
|
Details | Diff | Splinter Review |
2.77 KB,
patch
|
Wurblzap
:
review+
|
Details | Diff | Splinter Review |
Every checksetup.pl run yields a pop-up on Windows with ActiveState Perl 5.10.0, saying "This application has failed to start because OCI.dll was not found. Re-installing the application may fix this problem." Workaround is to simply click Ok. This happens even without any intention to install Oracle -- I never installed DBD-Oracle, I think it came in with the ActiveState Perl install (ppm tells me the module is in "perl", not in "site"). I'm aware this can probably be cured by deinstalling DBD-Oracle. I still think we should do something about it because it'll probably bite *lots* of people after our next release.
Flags: blocking3.4?
Comment 1•15 years ago
|
||
IMO, this is not a Bugzilla bug. I fixed the problem by typing (from memory): ppm --remove --area perl DBD::Oracle This problem should be reported to ActiveState, who installs it by default with no reason. IMO, this bug is INVALID (not a Bugzilla bug).
Flags: blocking3.4? → blocking3.4-
Assignee | ||
Comment 2•15 years ago
|
||
It would require extensive amounts of workaround code just to fix this, which is (I agree with LpSolit) a bug in the package that ActiveState ships for DBD::Oracle. FWIW, it already affects people, our next release won't make any difference.
Assignee | ||
Comment 3•15 years ago
|
||
I've reported this bug to ActiveState: http://bugs.activestate.com/show_bug.cgi?id=82183
Comment 4•15 years ago
|
||
Shouldn't we implement either of the workarounds suggested in the ActiveState bug report? Namely either 1) use CPAN.pm to determine the version number of installed modules, or 2) call SetErrorMode() to disable Windows error dialogs. There's even working code for the latter case in the ActiveState bug report. And former seems something checksetup.pl should probably always do if that's somehow "better" method to determine versions or fixes some other symptoms of module loading we might have had to workaround in the past. Or is there a known reason neither of those can be used in Bugzilla?
Assignee | ||
Comment 5•15 years ago
|
||
Yeah, we could do the SetErrorMode thing.
Assignee | ||
Updated•14 years ago
|
Severity: normal → major
Flags: blocking3.6+
Priority: -- → P1
Hardware: x86 → All
Assignee | ||
Comment 6•14 years ago
|
||
BTW, the reason that we don't want to use CPAN.pm to determine versions is that we want checksetup.pl to do *exactly* what Bugzilla is going to do when loading the module, so that we know that it will in fact load and work and that all its prerequisites are installed--not just that it itself is on the filesystem somewhere.
Assignee | ||
Updated•14 years ago
|
Target Milestone: --- → Bugzilla 3.4
Assignee | ||
Comment 7•14 years ago
|
||
This is the workaround recommended by ActiveState.
Assignee: installation → mkanat
Status: NEW → ASSIGNED
Assignee | ||
Updated•14 years ago
|
Attachment #420620 -
Flags: review?(LpSolit)
Assignee | ||
Comment 8•14 years ago
|
||
Comment on attachment 420620 [details] [diff] [review] v1 I can't test this, since apparently my installation doesn't have DBD-Oracle and I can't install it via PPM, but I do know that this compiles, and it's basically identical to the code from ActiveState.
Comment 9•14 years ago
|
||
(In reply to comment #8) > (From update of attachment 420620 [details] [diff] [review]) > I can't test this, since apparently my installation doesn't have DBD-Oracle and > I can't install it via PPM I have the same problem. Once it's removed, it seems impossible to reinstall it.
Assignee | ||
Comment 10•14 years ago
|
||
Comment on attachment 420620 [details] [diff] [review] v1 Hmmm. Byron, do you want to help us out with our dilemma here? :-)
Attachment #420620 -
Flags: review?(LpSolit) → review?(bugzilla)
Reporter | ||
Comment 11•14 years ago
|
||
Comment on attachment 420620 [details] [diff] [review] v1 This resolves the problem for me.
Attachment #420620 -
Flags: review+
Assignee | ||
Comment 12•14 years ago
|
||
Comment on attachment 420620 [details] [diff] [review] v1 Awesome; that's good enough for me.
Attachment #420620 -
Flags: review?(bugzilla)
Assignee | ||
Comment 13•14 years ago
|
||
3.4 requires a different patch because it doesn't have init_console().
Attachment #420748 -
Flags: review?(wurblzap)
Reporter | ||
Comment 14•14 years ago
|
||
Comment on attachment 420748 [details] [diff] [review] v1 (3.4) Yup, this works for me.
Attachment #420748 -
Flags: review?(wurblzap) → review+
Assignee | ||
Comment 16•14 years ago
|
||
tip: Checking in Bugzilla/Install/Util.pm; /cvsroot/mozilla/webtools/bugzilla/Bugzilla/Install/Util.pm,v <-- Util.pm new revision: 1.26; previous revision: 1.25 done 3.4: Checking in checksetup.pl; /cvsroot/mozilla/webtools/bugzilla/checksetup.pl,v <-- checksetup.pl new revision: 1.561.2.1; previous revision: 1.561 done Checking in Bugzilla/Install/Util.pm; /cvsroot/mozilla/webtools/bugzilla/Bugzilla/Install/Util.pm,v <-- Util.pm new revision: 1.17.2.2; previous revision: 1.17.2.1 done
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•