[riot-notifications] [RIOT-OS/RIOT] makefiles/boards.inc.mk: list boards variables (#11662)

Gaƫtan Harter notifications at github.com
Fri Jun 7 16:45:43 CEST 2019


### Contribution description

Add a global definition of `BOARDS` not using `$(shell find)` but makefile `$(wildcard)` and a `info-boards` that list them all.
This can be used for scripts that want to repeat one thing on all boards.

The boards is by default the list of all `BOARDS`.

In an application, if a global target is used, it will still be overwritten to the supported boards. It was not changed by this PR.

This global definition has also been used for `make info-applications-supported-boards` to only evaluate the list of boards once.

This saved 46seconds of the execution time go from 2:45 to 2:00 with similar test conditions

* 2:45 (with find)
* 2:10 (with wildcard)
* 2:00 (with single evaluation)

### Further work

Evaluating `BOARDS` could be done only once in the murdock scripts too but not sure how it should be done. It would not save a lot of real time, but some cpu time.

### Testing procedure

List all boards in RIOT

```
make info-boards
# All boards space separated
```

The list of supported applications is the same as in master

```
make info-applications-supported-boards
# Same output
```

### Issues/PRs references

I needed a command to help checking refactoring on all boards.
You can view, comment on, or merge this pull request online at:

  https://github.com/RIOT-OS/RIOT/pull/11662

-- Commit Summary --

  * makefiles/boards.inc.mk: list boards variables
  * Makefile.include: use the variables from makefiles/boards.inc.mk
  * makefiles/app_dirs.inc.mk: Calculate 'BOARDS' value only once

-- File Changes --

    M Makefile (1)
    M Makefile.include (3)
    M makefiles/app_dirs.inc.mk (2)
    A makefiles/boards.inc.mk (14)
    M makefiles/info-global.inc.mk (1)

-- Patch Links --

https://github.com/RIOT-OS/RIOT/pull/11662.patch
https://github.com/RIOT-OS/RIOT/pull/11662.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/11662
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190607/449de265/attachment-0001.html>


More information about the notifications mailing list