Implicitly document exactly where mach jumped the shark

RESOLVED WONTFIX

Status

Firefox Build System
Mach Core
RESOLVED WONTFIX
5 years ago
2 months ago

People

(Reporter: sfink, Assigned: sfink)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 1 obsolete attachment)

Just in case people look back and try to figure it out, this will save a little time.
Created attachment 729932 [details] [diff] [review]
Allow mach commands to have aliases
Created attachment 729933 [details] [diff] [review]
Implicit documentation of exactly where mach jumped the shark
Depends on: 847722
Created attachment 729936 [details] [diff] [review]
Implicit documentation of exactly where mach jumped the shark

reuben helpfully pointed out that I left in an additional js.cpp change in the patch. I'd hate to not get this perfect.
Attachment #729933 - Attachment is obsolete: true

Comment 4

5 years ago
r- until |mach jumpshark| opens https://www.youtube.com/watch?v=MpraJYnbVtE :)

Comment 5

5 years ago
Comment on attachment 729936 [details] [diff] [review]
Implicit documentation of exactly where mach jumped the shark

Being able to alias mach commands is great, but please, can we keep mach a serious tool?  I don't think jokes like this belong in a tool that our developers are supposed to use to get things done.  Thanks!
Don't worry, I had no intention of ever landing this patch. (And in retrospect, I can see some value in getting people more accustomed to using mach for things. Though the mach search command still feels like going overboard.)

Cleaning up old bugs.
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → WONTFIX

Updated

2 months ago
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.