Move config system into it's own folder and refactor header generation

[theotherjimmy]

Descritpion

The config system is a huge file, and seems to be growing to support
things such as bootloader. This implies that the Config class may grow
to beyond the size of this one file.

Furthermore, I saw an error generated by the online build system recently.
This was the traceback:

ValueError: max() arg is an empty sequence
  File "mbed/build_tasks/tasks.py", line 33, in perform_build
    result = _perform_build(build_params)
  File "mbed/build_tasks/tasks.py", line 139, in _perform_build
    result = build(build_params=build_params)
  File "mbed_compiler/build.py", line 223, in build
    mbed_compiler.build_api.build_program(build_params=build_params, notify=notify)
  File "mbed_compiler/build_api.py", line 227, in build_program
    resources.objects = toolchain.compile_sources(resources, build_params['chrooted_build']) + resources.objects
  File "tools/toolchains/__init__.py", line 802, in compile_sources
    self.get_config_header()
  File "tools/toolchains/__init__.py", line 1134, in get_config_header
    crt_data = Config.config_to_header(self.config_data) if self.config_data else None
  File "tools/config.py", line 823, in config_to_header
    if params else 0)

How to reproduce

Simply create an mbed_app.json that removes all macros. This will
cause the macros array to be empty without causing a ConfigException.
When the macros array is empty, max() becomes very confused, as it
is asked to take the maximum of an empty array.

New behavior

We just add a [0] to the end to prevent that silly traceback.

[theotherjimmy]

/morph test

[mbed-bot]

Result: FAILURE

Your command has finished executing! Here's what you wrote!

/morph test

Output

mbed Build Number: 1874

Build failed!

[0xc0170]

morph test - known failure, will be fixed soon (patch already proposed, waiting for CI to complete)

[0xc0170]

Looking good, just 2 small things in the template. Tools are slowly becoming more organized 👍

[0xc0170]

Should this file contain license header? I would say yes.

[theotherjimmy]

I also say yes. now it has one.

[0xc0170]

should have a new line at the end of file

[theotherjimmy]

👍

[theotherjimmy]

@0xc0170 The gig's up! you noticed that I was organizing the tools.

[theotherjimmy]

Thanks for the review. I addressed all of the comments, so I'm moving this to needs Review and needs CI

[theotherjimmy]

/morph test

[mbed-bot]

Result: FAILURE

Your command has finished executing! Here's what you wrote!

/morph test

Output

mbed Build Number: 1889

Example Build failed!

[tommikas]

Something wrong with mbed_config.h generation?

armcc failing:

Compile [  0.3%]: FlashIAP.cpp
[Warning] mbed_config.h@66,0:  #2775-D: white space is required between the macro name "MEM_ALLOC" and its replacement text
[Warning] mbed_config.h@67,0:  #2775-D: white space is required between the macro name "MEM_FREE" and its replacement text
[Warning] mbed_config.h@70,0:  #2775-D: white space is required between the macro name "MBED_HEAP_STATS_ENABLED" and its replacement text
[Warning] mbed_config.h@72,0:  #1-D: last line of file ends without a newline
Compile [  0.7%]: ns_cmdline.c
[Warning] mbed_config.h@66,0:  #2775-D: white space is required between the macro name "MEM_ALLOC" and its replacement text
[Warning] mbed_config.h@67,0:  #2775-D: white space is required between the macro name "MEM_FREE" and its replacement text
[Warning] mbed_config.h@70,0:  #2775-D: white space is required between the macro name "MBED_HEAP_STATS_ENABLED" and its replacement text
[Warning] mbed_config.h@72,0:  #1-D: last line of file ends without a newline
[Error] ns_cmdline.c@342,0:  #29: expected an expression
[Error] ns_cmdline.c@344,0:  #29: expected an expression
[Error] ns_cmdline.c@437,0:  #29: expected an expression

gcc:

Compile [  0.3%]: FlashIAP.cpp
[Warning] mbed_config.h@66,18: missing whitespace after the macro name
[Warning] mbed_config.h@67,17: missing whitespace after the macro name
[Warning] mbed_config.h@70,32: missing whitespace after the macro name
Compile [  0.7%]: Ethernet.cpp
[Warning] mbed_config.h@66,18: missing whitespace after the macro name
[Warning] mbed_config.h@67,17: missing whitespace after the macro name
[Warning] mbed_config.h@70,32: missing whitespace after the macro name
Compile [  1.0%]: ns_cmdline.c
[Warning] mbed_config.h@66,18: ISO C99 requires whitespace after the macro name
[Warning] mbed_config.h@67,17: ISO C99 requires whitespace after the macro name
[Warning] mbed_config.h@70,32: ISO C99 requires whitespace after the macro name
[Error] mbed_config.h@67,17: expected expression before '=' token
[Error] mbed_config.h@67,17: expected expression before '=' token
[Error] mbed_config.h@67,17: expected expression before '=' token
[Error] mbed_config.h@67,17: expected expression before '=' token

[theotherjimmy]

@tommikas Thanks for the info. I'll check into the failure.

[theotherjimmy]

That should do it. I was using ConfigMacro.name instead of ConfigMacro.macro_name.

name included the value...

[theotherjimmy]

/morph test

[theotherjimmy]

/morph test

[mbed-bot]

Result: SUCCESS

Your command has finished executing! Here's what you wrote!

/morph test

Output

mbed Build Number: 31

All builds and test passed!