Status

Add-on SDK
General
P3
normal
RESOLVED WONTFIX
6 years ago
3 years ago

People

(Reporter: irakli, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

If I understand it correctly on windows stdout of firefox process does not includes data written using `dump` function, this is also a reason why we employ a workaround https://github.com/mozilla/addon-sdk/blob/master/python-lib/cuddlefish/app-extension/components/harness.js#L533-549 (It actually spread along few other files, but this gives an overview) to overcome this limitation. Workaround adds complexity and causes code duplication. Would be nice if we could fix whatever is broken on platform!
Alex, can you please post a link to a bug, you've submitted patch for, to fix windows console related issues. I think it may be related.
Created attachment 564994 [details]
Jsctype hack

It is bug 673383. It may help but I can't say if it is really related.
It is about stdout handling on windows, so we may give it a chance.
Here is a jsctype equivalent of this platform patch. You may give it a try!

But regarding my jetpack runner experience, I was able to retrieve firefox stdout without such jsctype hack or platform fix. So there may be some tuning to do in python side to fix it!
One thing to try is running firefox though `cmd.exe` with /K or /C option.
You can retrieve the absolute path of cmd.exe by reading env.ComSpec.

Updated

6 years ago
Priority: -- → P3
I believe that the automated tests that run on tinderbox output using dump and that seems to be correctly captured by python, have we looked at how that works?
Irakli, can we wontfix this one?
Flags: needinfo?(rFobic)
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Flags: needinfo?(rFobic)
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.