glapi: Avoid argparse type argument for API XML input files.
authorJose Fonseca <jfonseca@vmware.com>
Tue, 26 May 2015 10:01:57 +0000 (11:01 +0100)
committerJose Fonseca <jfonseca@vmware.com>
Tue, 26 May 2015 14:26:03 +0000 (15:26 +0100)
commitb787f48ed2a7e1855100afd943ae6b407abb401f
treee039bdbce98690ebe65960f5acb2443bbdef56b5
parent224a77cc60cc0e7f8a14e35ebca6e42544af39b1
glapi: Avoid argparse type argument for API XML input files.

argparse type is a nice type saver for simple data types, but it doesn't
look a good fit for the input XML file:

- Certain implementations of argparse (particularly python 2.7.3's)
  invoke the type constructor for the default argument even when an
  option is passed in the command line.  Causing `No such file or
  directory: 'gl_API.xml'` when the current dir is not
  src/mapi/glapi/gen.

- The parser takes multiple arguments.  This is currently worked around
  using lambdas, but that unnecessarily complex and hard to read.
  Furthermore it's odd to have a side-effect as heavy as parsing XML
  happening deep inside the argument parsing.

https://bugs.freedesktop.org/show_bug.cgi?id=90600

Reviewed-by: Brian Paul <brianp@vmware.com>
src/mapi/glapi/gen/gl_procs.py
src/mapi/glapi/gen/gl_table.py
src/mapi/glapi/gen/remap_helper.py