Closed Bug 758732 Opened 8 years ago Closed 8 years ago

configure sometimes fails with "rm: cannot lstat `conftest.exe': Permission denied" followed by "C++ compiler cannot create executables"

Categories

(Firefox Build System :: General, defect)

x86
Windows 7
defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mayhemer, Unassigned)

References

Details

Attachments

(1 file)

Attached patch v1Splinter Review
+++ This bug was initially created as a clone of Bug #545015 +++

The current delay of 1 is not enough.  As Ehsan has suggested, I'm using timeout of 2 a long time as a local path.  When I sometimes forget to apply it, I get the conftest.exe permission denied error.

Proposing to raise the timeout.
Attachment #627311 - Flags: review?(ehsan)
I suggest we just assume that VC++ can create executables and avoid doing this test altogether on Windows, instead of making configure two seconds slower than necessary--especially considering that configure is already ungodly slow on Windows without any sleeping.
Comment on attachment 627311 [details] [diff] [review]
v1

What could *possibly* go wrong?  :-)
Attachment #627311 - Flags: review?(ehsan) → review+
(In reply to Brian Smith (:bsmith) from comment #1)
> I suggest we just assume that VC++ can create executables and avoid doing
> this test altogether on Windows, instead of making configure two seconds
> slower than necessary--especially considering that configure is already
> ungodly slow on Windows without any sleeping.

This check is done by one of the autoconfigure macros (AC_LANG_C or whatever they call it) so we can't skip that particular check quite easily.
And by autoconfigure, I obviously mean autoconf!
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
It is possible to override bits of the built-in autoconf macros, m4 is pretty flexible.
(In reply to Ted Mielczarek [:ted] from comment #6)
> It is possible to override bits of the built-in autoconf macros, m4 is
> pretty flexible.

Filed bug 758752 for this.  Can you please fill in the details?
Blocks: 788241
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.