855160 - Implicitly document exactly where mach jumped the shark

Status: RESOLVED WONTFIX

(Firefox Build System :: Mach Core, enhancement)

Description

[Steve Fink [:sfink] [:s:]]

Just in case people look back and try to figure it out, this will save a little time.

Comment 1

[Steve Fink [:sfink] [:s:]]

Attachment: Allow mach commands to have aliases

Comment 2

[Steve Fink [:sfink] [:s:]]

Attachment: Implicit documentation of exactly where mach jumped the shark

Comment 3

[Steve Fink [:sfink] [:s:]]

Attachment: 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.

Comment 4

[Gregory Szorc [:gps]]

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

Comment 5

[(no longer active)]

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!

Comment 6

[Steve Fink [:sfink] [:s:]]

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.