Closed
Bug 885544
Opened 11 years ago
Closed 8 years ago
Stop setting Firefox compatibility when using DEBUG=1 mode
Categories
(Firefox OS Graveyard :: Gaia, defect)
Firefox OS Graveyard
Gaia
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: ochameau, Assigned: ochameau)
Details
Attachments
(1 file)
(deleted),
text/html
|
Details |
In bug 866702, we choosed to automagically enable gaia's Firefox compatiblity mode when you set DEBUG=1. This mode ends up being incompatible with b2g-desktop.
I tend to think that it ends up being more misleading than helpfull.
If we keep the current behavior, you will have to do:
DEBUG=1 DESKTOP=0 make
in order to run on b2g-desktop.
I'd prefert to keep:
DEBUG=1 make
and educate developers to do the following in order to run on firefox:
DEBUG=1 DESKTOP=1 make
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → poirot.alex
Assignee | ||
Comment 1•11 years ago
|
||
Kevin, I know you prefer the current behavior, but when I see the issues it can create in bug 882760, I'd prefer modifying it :/
Attachment #765627 -
Flags: review?(kgrandon)
Comment 2•11 years ago
|
||
If this is the case then I will be happy if we can make DESKTOP=1 make set the DEBUG=1 flag as well. Basically instead of DEBUG=1 make, developers could simply type: DESKTOP=1 make, and they would be ready to work in a browser.
My real preference would be to actually implement a new flag instead of DEBUG so developer workflow is not changed, but this change shouldn't be too bad.
Comment 3•11 years ago
|
||
Comment on attachment 765627 [details]
Pull request 10515
I will be totally happy if migrate to only having to type DESKTOP=1 make, or create a `different` DEBUG flag for tools/b2g desktop.
Attachment #765627 -
Flags: review?(kgrandon)
Comment 4•11 years ago
|
||
I think we need to separate the technical things we do and the functional things we want.
We want :
* run unit tests (in the browser and in b2g desktop) => UNIT=1
* debug apps in the browser => DESKTOP=1
* simulator => SIMULATOR=1
* other stuff we could have later
We do :
* change settings and/or prefs => needed for tests, desktop, simulator (maybe not the same set for all of them)
* add httpd extension => needed for tests and desktop
* add helper extensions => needed for desktop and simulator
Therefore, we need to uncouple the functional things from the technical things. This is more like a matricial array of functionalities.
external env variable UNIT=1 would trigger internal DEBUG=1 (change settings/prefs), HTTPD=1 (install httpd extension)
external env variable DESKTOP=1 would trigger internal DEBUG=1 (change settings/prefs), HTTPD=1 (install httpd extension), HELPERS=1 (install helper extensions)
external env variable SIMULATOR=1 would trigger internal DEBUG=1 (change settings/prefs), HELPERS=1 (install helper extensions)
Incidentally, this would need different profile folders of course (because all of this install stuff in the profile) so we can build on the work done in Bug 864751 for this.
I may have missed something in what we need, but you get the point. The point is to _not_ change an existing environment for the sake of having another environment.
Assignee | ||
Updated•8 years ago
|
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•