[riot-notifications] [RIOT-OS/RIOT] makefiles/info-global: Remove dependency resolution cache (#14132)

Leandro Lanzieri notifications at github.com
Mon May 25 11:19:23 CEST 2020


### Contribution description
This removes a preliminar dependency resolution that is performed without including features, to skip a full dependency resolution when boards can be proven to be unsupported on an earlier stage.

This was introducing issues on some boards since the blacklisting of some features depends on board information which is not available at the time of performing this resolution.

#### Some time measurements

Running `make info-boards-supported` 50 times on my computer results in:

| Application | Time with cache | Time without cache |
| -------------- | ---------------------- | -------------------------- |
| `hello_world` | `873.0 ms ±  20.5 ms` |`851.7 ms ±  26.3 ms` |
| `tests/driver_ws281x` (proposed [here](https://github.com/RIOT-OS/RIOT/pull/13349#issuecomment-605950720)) | `266.4 ms ±  11.6 ms` | ` 263.0 ms ±  18.6 ms` |

### Testing procedure

- `make info-boards-supported` should work properly now (it was pointed out in the mailing list that the sodaq boards are not showing up in the list in current master, for instance)


### Issues/PRs references
Spotted by @fjmolinas [here](https://github.com/RIOT-OS/RIOT/pull/13738#issuecomment-633432857)

You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * makefiles/info-global: Remove dependency resolution cache

-- File Changes --

    M makefiles/info-global.inc.mk (10)

-- Patch Links --

https://github.com/RIOT-OS/RIOT/pull/14132.patch
https://github.com/RIOT-OS/RIOT/pull/14132.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/14132
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20200525/a0bdb159/attachment.htm>


More information about the notifications mailing list