[riot-notifications] [RIOT-OS/RIOT] Makefile.include: allow overwriting flash-recipe (#11433)

Gaƫtan Harter notifications at github.com
Tue Apr 23 15:12:07 CEST 2019


### Contribution description

This allows changing the flashing commands from the outside or in a BSP.

### Implementation

It cannot be set as `define flash-recipe ?=` as the feature is only introduced only in `gnumake 4`.
Also this allows re-using `default-flash-recipe` if wanted.

### Testing procedure

Flashing still works for a board that flashes. (CI will test this)

It can be overwritten:

```
make --no-print-directory -C examples/hello-world/ flash-only flash-recipe="echo changed recipe"
echo changed recipe
changed recipe
```

### Issues/PRs references

Could be used for #11093 without introducing a one flasher feature for the moment.

I need to overwrite `flash-recipe` to handle flashing on another machine in my setup and currently need to rely on `override` for setting the variable.

This would allow refactoring the `esp` flashing command to use commands on different lines
https://github.com/RIOT-OS/RIOT/blob/master/cpu/esp32/Makefile.include
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * Makefile.include: allow overwriting flash-recipe

-- File Changes --

    M Makefile.include (4)

-- Patch Links --

https://github.com/RIOT-OS/RIOT/pull/11433.patch
https://github.com/RIOT-OS/RIOT/pull/11433.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/11433
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190423/9ca85a9b/attachment.html>


More information about the notifications mailing list