Convert ragel-gen.py to use new options and expunge config from BinaryUtil.

Review Request #1970 - Created March 21, 2015 and submitted

Information
Eric Ayers
pants
zundel/ragel-gen-options
1308
8435ddf...
Reviewers
pants-reviews
benjyw, jinfeng, jsirois

- Register global options --pants-support-baseurls, --max-subprocess-args,
and --pants-support-fetch-timeout-secs.
- Remove 'config' parameter and other references from BinaryUtil.
- Remove pants.ini values that have good compiled in defaults.
- Updated help for the common --support and --version flags by making BinaryUtil a mixin

See Bug link for latest CI status.

Issues

  • 0
  • 1
  • 0
  • 1
Description From Last Updated
Eric Ayers
Eric Ayers
Stu Hood
John Sirois
Eric Ayers
John Sirois
Eric Ayers
Eric Ayers
Review request changed

Status: Closed (submitted)

Change Summary:

Thanks for the reviews Stu and John. Commit 5b230fd

Benjy Weinberger

   

I just noticed that these are curly braces, not parens, so the key here is a set literal, not a pair.

  1. Oops, see https://rbcommons.com/s/twitter/r/1995/

Loading...