Opened on 11/10/2017 at 09:09:57 AM

Closed on 11/20/2017 at 11:36:32 AM

#6021 closed change (fixed)

Refactor to use modern argparse

Reported by: tlucas Assignee: tlucas
Priority: P3 Milestone:
Module: Automation Keywords:
Cc: trev, Sebastian Blocked By:
Blocking: #5996 Platform: Unknown / Cross platform
Ready: yes Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

Description (last modified by tlucas)


The buildtools' was implemented before a standard means of parsing command line arguments was introduced into the Python Standard Library. This has changed

What to change

  • Get rid of the custom argument parser in
  • Keep all help texts and present them appropriately for every available command
  • Group commands by command, rather than by (previously) extension type

Integration Notes

The release checklist templates for Chrome/Opera/Safari and Firefox need to be adjusted to contain the new correct commands.

Other repositories must adhere to the new interface of

old:, sys.argv)


Hints for testers

This should have no impact on the builds themselves at all.

Attachments (0)

Change History (10)

comment:1 Changed on 11/13/2017 at 02:46:20 PM by tlucas

  • Description modified (diff)

comment:2 Changed on 11/13/2017 at 02:48:38 PM by tlucas

  • Cc trev Sebastian added

comment:3 Changed on 11/13/2017 at 02:49:02 PM by tlucas

I decided to actually create an issue for this, since their will be some integration work.

comment:4 Changed on 11/13/2017 at 02:50:38 PM by tlucas

  • Blocking 5996 added

comment:5 Changed on 11/14/2017 at 05:38:40 PM by tlucas

  • Priority changed from Unknown to P3
  • Ready set

comment:6 Changed on 11/14/2017 at 05:38:47 PM by tlucas

  • Owner set to tlucas

comment:7 Changed on 11/15/2017 at 10:26:43 AM by tlucas

  • Review URL(s) modified (diff)
  • Status changed from new to reviewing

comment:8 Changed on 11/17/2017 at 03:21:07 PM by tlucas

  • Description modified (diff)

comment:9 Changed on 11/20/2017 at 11:36:13 AM by abpbot

A commit referencing this issue has landed:
Issue 6021 - Refactoring

comment:10 Changed on 11/20/2017 at 11:36:32 AM by tlucas

  • Resolution set to fixed
  • Status changed from reviewing to closed

Add Comment

Modify Ticket

Change Properties
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from tlucas.
Note: See TracTickets for help on using tickets.