Closed
Bug 503510
Opened 15 years ago
Closed 15 years ago
re-enable esc building on tamarin-redux
Categories
(Tamarin Graveyard :: Build Config, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: brbaker, Assigned: brbaker)
Details
Attachments
(3 files, 1 obsolete file)
14.40 KB,
patch
|
dschaffe
:
review+
|
Details | Diff | Splinter Review |
2.25 KB,
patch
|
Details | Diff | Splinter Review | |
820 bytes,
patch
|
brbaker
:
review+
|
Details | Diff | Splinter Review |
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+
Assignee | ||
Comment 1•15 years ago
|
||
Attachment #387878 -
Flags: review?(dschaffe)
Assignee | ||
Comment 2•15 years ago
|
||
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.
Assignee | ||
Comment 3•15 years ago
|
||
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 4•15 years ago
|
||
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+
Assignee | ||
Comment 5•15 years ago
|
||
Pushed 2122:c89b4ae23e44
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 6•15 years ago
|
||
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)
Assignee | ||
Updated•15 years ago
|
Attachment #388510 -
Flags: review?(dschaffe)
Assignee | ||
Comment 7•15 years ago
|
||
patch v3 pushed as changeset 2134:d1fffa9d6662
Comment 8•15 years ago
|
||
allows esc to run on non-windows platforms. use ${shell_extension} instead of hard coding .exe.
Attachment #388686 -
Flags: review?(brbaker)
Updated•15 years ago
|
Attachment #388686 -
Attachment description: patch → make run-release-esc.sh run cross platform
Assignee | ||
Comment 9•15 years ago
|
||
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+
Assignee | ||
Comment 10•15 years ago
|
||
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.
Description
•