Closed Bug 863092 Opened 8 years ago Closed 8 years ago

Make mach run and mach debug run firefox -no-remote

Categories

(Firefox Build System :: Mach Core, enhancement)

x86
macOS
enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla23

People

(Reporter: joe, Assigned: ehsan)

References

Details

Attachments

(1 file)

Ehsan promised he'd do this if I filed the bug. Well, here it is!
Flags: needinfo?(ehsan)
Note that you can currently do this with "mach run -no-remote" and "mach debug -no-remote".

I think it's a reasonable request to include -no-remote by default, but maybe there should be a way to override it in case you actually *do* want to pass a new command line to the existing process.

How useful would this be as a default option?  Are there many cases where you need -no-remote but don't also need other options like -P or -profile?
Depends on: 841445, 857600
See Also: → 858629
(fwiw, i have no idea what use case passing -profile serves without -no-remote or if that is just a bug)
There's really very little reason to not use -no-remote for local builds.  The only reason that I can think of is if you're trying to debug our remoting code, and I would definitely not object if somebody writes a patch to add an exception about that when they need it.  ;-)
Flags: needinfo?(ehsan)
Attached patch Patch (v1)Splinter Review
Attachment #738868 - Flags: review?(gps)
Comment on attachment 738868 [details] [diff] [review]
Patch (v1)

Review of attachment 738868 [details] [diff] [review]:
-----------------------------------------------------------------

Makes sense to me! I *always* run with -no-remote. Although I would feel better if we addressed the case where people don't want -no-remote. Follow-up I suppose.
Attachment #738868 - Flags: review?(gps) → review+
(In reply to :Ehsan Akhgari (needinfo? me!) from comment #4)
> There's really very little reason to not use -no-remote for local builds. 
> The only reason that I can think of is if you're trying to debug our
> remoting code, and I would definitely not object if somebody writes a patch
> to add an exception about that when they need it.  ;-)

Or if you live on the metal and want bleed-of-the-edge firefox for your everyday browser (and no, I don't do that ;)
http://hg.mozilla.org/mozilla-central/rev/dcc71e2b291c
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla23
Blocks: 863709
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.