Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add some type checking of configs #4000

Merged
merged 1 commit into from
Mar 29, 2017

Commits on Mar 22, 2017

  1. Add some type checking of configs

    Before, the types were not checked and just expected. The old behavior
    would cause lots of tracebacks, or, much worse, convert things like:
    ```
    {
         "target_overrides": {
            "*": {
    		"target.macros_add": "CONFIG_GPIO_AS_PINRESET"
            }
         }
    }
    ```
    into a definition of each of the letters as macros that expand to
    nothing, causing massive compilation problems.
    
    I resolved this by adding some type checking to the config data. Now
    there is a type check for most of the fields within a library and
    application configurations.
    theotherjimmy committed Mar 22, 2017
    Configuration menu
    Copy the full SHA
    d5637bb View commit details
    Browse the repository at this point in the history