Closed Bug 1198759 Opened 9 years ago Closed 4 years ago

Automatically load devtools from source when run from ./mach

Categories

(DevTools :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: ochameau, Unassigned)

References

(Blocks 2 open bugs)

Details

Attachments

(1 file, 1 obsolete file)

As suggested by Brian in bug 1188413 comment 7, it would be great to aumatically load devtools from sources instead of jar package when we launch it from ./mach. So that, we can immediately reload the tools from the sources without having to restart/rebuild. We may hide that behind the chrome pref to not enable this behavior for everyone. Also we would have to wait before doing that as the "loading devtools from source directory" is still broken in various cases like gcli/webIDE (at least).
Attached patch patch v1 (obsolete) (deleted) — Splinter Review
Tweak mach to pass the topsrcdir. I looked at existing env variable and haven't seen anything useful. There is PWD, which will often be topsrcdir, but not always I imagine... I haven't looked into Directory Service XPCOM, but I don't think we have the source dir here. Otherwise, regarding devtools patch, I kept this hidden behind chrome pref, but I tend to think we should load from source by default? (especially once we are confident about loading from source behavior).
Depends on: 1199201
Before proceeding with this patch, I would like to go through most features/panels and see if everything works. For example, WebIDE has always been completely broken. I'm wondering if browser toolbox works also. And what about all by-default-hidden tools?
Attached patch patch v2 (deleted) — Splinter Review
Rebased.
Attachment #8652876 - Attachment is obsolete: true
Attachment #8668530 - Flags: feedback?(jryans)
I'll try this out soon and see how it goes.
This is still in my queue to try and think about, sorry for the delay.
Comment on attachment 8668530 [details] [diff] [review] patch v2 Review of attachment 8668530 [details] [diff] [review]: ----------------------------------------------------------------- Okay, I finally had time to try this. I think this generally works well. What are you current thoughts on this approach? Do you prefer your add-on plan instead of this?
Attachment #8668530 - Flags: feedback?(jryans) → feedback+
Also, if we were to recommend something like this as the "default workflow", I think it might be best to eliminate as much preprocessing as we can first, to avoid surprises. Or at least very clearly state all known edge cases where it may fail.
We were talking in Orlando, and it sounds like people would like to have this. Alex, what do you think?
Flags: needinfo?(poirot.alex)
I should morph this patch into registering the devtools addon as it cover much more ressources. But we should first ensure more people in our team actually use that manually to highlight the possibly hidden breakages of it. The addon workflow is going to open new completely different workflow of contribution, may be we can start with that. It is more appealing than this loader trick. I plan to setup a group within devtools team to start playing with that.
mach could potentially auto-install the add-on for you, since the add-on is currently temporary.
I've been using the "text file inside extensions dir" hack with this, which keeps the reload add-on installed, pretty nice. However what did trip me up once was having multiple source checkouts, but only one profile, so the profile can end up pointing to the wrong checkout via the text file.
I would prefer to work on bug 1185460, which would benefit to more people and make the original experience better. I'm still anxious in making the addon codepath become the default one.
Flags: needinfo?(poirot.alex)
Product: Firefox → DevTools

We are no longer pursuing making DevTools be an addon, so we can't do that.
In the meantime, ./mach run experience has improved.

Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: