Split up source files in groups

[omdathetkan]

Note: Do not merge

Split up source files in three groups to make projects more friendly to use.

If this is considered an improvement I will do a proper pull request to master.
Note that the KL25Z, LPC11U24 and LPC1347 did not have the --c99 setting for c files before, please verify if this is intended or not.

[0xc0170]

Hello jorisa,

can you share what is inside those folders , a snapshot?
Shouldn't "hal" be a folder inside mbed? Like is on github a root folder mbed - (api,common,hal, target). Same separation of those files.

Splitting will definitely improve user experience.

Regards,
0xc0170

[omdathetkan]

I'm grouping as follows:

+                if file.startswith("mbed\\common"):
+                    source_files['mbed'].append(f)
+                elif file.startswith("mbed\\targets"):
+                    source_files['hal'].append(f)
+                else:
+                    source_files['src'].append(f)

Right now it looks like this:

I figured this would be a good trade-off between a single flat list and completely reproducing the full folder structure. When I'm debugging something its usually either an app, the mbed api, or fixing something for a specific target.

[emilmont]

Hi Joris,
thanks for this "pull request".

The only thing that is blocking me from accepting it immediately is that we want to handle more gracefully the case of exporting a simple program that does not contain the mbed library sources (actually the most common case). My understanding, just reading the code, is that the project will get two empty groups ("hal" and "mbed").
Can you add a check for an empty list in Jinja2 syntax?

Cheers,
Emilio

[omdathetkan]

Will do! Could you comment on the fact whether all templates should contain the --c99 flag?
Right now some of them do, some of them don't and I don't know if this is on purpose.

[emilmont]

Yes, all templates should contain the "--c99" flag for compatibility with the mbed build system:
https://github.com/mbedmicro/mbed/blob/master/workspace_tools/toolchains/arm.py#L60