Bug 28459 – bugzilla-tool needs a new name

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Eric Seidel (no email)]

bugzilla-tool needs a new name

If we can come up with a name here, I'm happy to make it happen.  I've got a few out-standing patches against the current tool, so I'd like to move those/land those before we land the rename though.

webkit-tool
wkt
kitchen-sink?

[Adam Barth]

patch-tool

[Adam Barth]

*** Bug 33129 has been marked as a duplicate of this bug. ***

[Eric Seidel (no email)]

I thought about naming more this morning.  I like Maciej/Adam's proposal to rename bzt to webkit-patch.  But one remaining question is where do we stuff all the other commands which don't fit that model.  Also, implied in that rename is that commands should combine to form smarter verb-named commands.  Like "apply-patches" and "apply-attachment" combining to just "apply" with some flags to enable the less-common usages.

All of the bugzilla-tool commands currently marked "common" are patch related, except tree-status:
apply-attachment, apply-patches, land-attachment, land-diff, land-patches, post-commits, post-diff, rollout, tree-status

[Adam Barth]

I'm fine with demoting tree-status for the main list of commands.

[Eric Seidel (no email)]

Adam and I talked a bit offline about making a new "webkit-patch" tool and slowly moving commands over to there when they're "ready for primetime.  bugzilla-tool (renamed or otherwise) could stick around for the while as the kitchen-sink (the tool which includes all the commands).

[Eric Seidel (no email)]

Created attachment 45937 [details]
Adam's requested proof-of-concept of adding a clone tool with the same name.

[WebKit Review Bot]

style-queue ran check-webkit-style on attachment 45937 [details] without any errors.

[Eric Seidel (no email)]

Comment on attachment 45937 [details]
Adam's requested proof-of-concept of adding a clone tool with the same name.

Forgot to pass --no-review.

I think I'm OK with just renaming bugzilla-tool to webkit-patch.  Yes, it sucks that webkit-patch will have a bunch of unrelated crap in it.  Yes, people will get confused about what bugzilla-tool is called.  Yes, we'll have to re-start all of our scripts.  But I'm less and less sure that having separate tools buys us much.  We could continue down the path of this patch, and with only a little more code restrict the commands which are exposed under webkit-patch.  But I'm not sure it buys us much.

[Adam Barth]

Created attachment 46029 [details]
Patch

[Eric Seidel (no email)]

Comment on attachment 46029 [details]
Patch

-from-bug was a nice touch.

for another bug: I think we should kill the optional [BUGID] on commands like land-diff (now "land") because they shouldn't be needed now that we can parse bug ids from ChangeLogs.

We could rename "tree status" to "may-I-land" or something similar. :)

Is Upload not yet a strict super-set of post?  Would be nice to kill post eventually.

Seems prepare and post don't actually need to be in the main help now that we have the new hotness we call upload.

Looks good.  Seems we should send an email to webkit-dev mentioning the rename.  We can just point people to --help if it's too complicated to explain.