Closed Bug 679313 Opened 13 years ago Closed 13 years ago

cfx output is duplicated

Categories

(Add-on SDK Graveyard :: General, defect)

Product:
Add-on SDK Graveyard
Component:
General
Platform:
All
Windows 7
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: ochameau, Unassigned)

Details

(Whiteboard: [cherry-pick-1.1]) 

Bug 665250 introduced a regression where you end up with duplicate output for cfx:
####################################
Using binary at 'C:\Program Files (x86)\Mozilla Firefox\firefox.exe'.
Using profile at 'c:\users\coup2b~1\appdata\local\temp\tmp0k1m6n.mozrunner'.
Running tests on Firefox 5.0/Gecko 5.0 ({ec8030f7-c20a-464f-9b0e-13a3a9e97384})
under WINNT/x86-msvc.

0 of 0 tests passed.
No tests were run
Running tests on Firefox 5.0/Gecko 5.0 ({ec8030f7-c20a-464f-9b0e-13a3a9e97384})
under WINNT/x86-msvc.

0 of 0 tests passed.
No tests were run
FAIL
Total time: 2.285000 seconds
Program terminated unsuccessfully.
####################################

Just before cfx quits, the whole cfx output is dumped to the stdout again.
I feel like the benefit of bug 665250 is way lower than the regression it comes with.

My environnement is:
Window 7, native windows command line, python 2.7.
so this is a regression caused by bug 665250?  If so, I'll look into it.  I admit I thought this duplication of output for the tests was somehow intentional, but I confirm I see it too - although what I don't see duplicated are the 'dump' statements from content code.
Resolved by reversion of fix for bug 665250.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Cherry pick commit that reverted fix for bug 665250 on stabilization branch:

https://github.com/mozilla/addon-sdk/commit/ba1749ccb1a7d16efff7e45989eeb5698b63a7b9
Whiteboard: [cherry-pick-1.1]
You need to log in before you can comment on or make changes to this bug.