Closed Bug 503510 Opened 15 years ago Closed 15 years ago

re-enable esc building on tamarin-redux

Categories

(Tamarin Graveyard :: Build Config, defect)

x86
Windows XP
defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: brbaker, Assigned: brbaker)

Details

Attachments

(3 files, 1 obsolete file)

Work item #224

In the last TC merge (4/24) the esc build failed. We should be building esc in redux so that we could have tracked when this failure occurred. Why it was removed from tr is unclear.
Flags: wanted-flashplayer10+
Flags: in-testsuite+
Flags: flashplayer-triage+
Flags: flashplayer-qrb+
Attachment #387878 - Flags: review?(dschaffe)
Thinking about this a little more maybe it makes more sense to place the esc into the deep testing since we are not actually recording the compilation time, we just want to make sure that we are able to recompile esc multiple times. 

We could add it to the windows platform now and then add it to the other platforms as they come online.
When we add more platforms to deep testing we can add this buildstep to them
Attachment #387878 - Attachment is obsolete: true
Attachment #387883 - Flags: review?(dschaffe)
Attachment #387878 - Flags: review?(dschaffe)
Comment on attachment 387883 [details] [diff] [review]
Run esc in windows deep testing

the /usr/bin/time -v -o flags are only supported on gnu time (would require some special setup on mac/linux).  as we discussed we could just remove the timing code.
Attachment #387883 - Flags: review?(dschaffe) → review+
Pushed 2122:c89b4ae23e44
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Remove the tracking of how long esc takes to compile. This is a test to make sure that we can compile esc and is NOT a performance test.
Attachment #388510 - Flags: review?(dschaffe)
Attachment #388510 - Flags: review?(dschaffe)
patch v3 pushed as changeset 2134:d1fffa9d6662
allows esc to run on non-windows platforms. use ${shell_extension} instead of hard coding .exe.
Attachment #388686 - Flags: review?(brbaker)
Attachment #388686 - Attachment description: patch → make run-release-esc.sh run cross platform
Comment on attachment 388686 [details] [diff] [review]
make run-release-esc.sh run cross platform

Can remove the second "chmod +x" since the shell has already been made executable on line 70
Attachment #388686 - Flags: review?(brbaker) → review+
esc is self compiling in the windows-deep, windows64-deep and mac-ppc-deep builders
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: