[riot-notifications] [RIOT-OS/RIOT] riotboot: quote PATH variable (#15389)

Bas Stottelaar notifications at github.com
Thu Nov 5 23:43:49 CET 2020


### Contribution description
This PR resolves an issue, if your `PATH` contains directories with spaces. At least on macOS, some applications add themselves to the system-wide `PATH`.

This is a small portion of the build log without the fix (notice the last line being the last part of the `PATH`):

```
PATH=/usr/local/opt/binutils/bin:/usr/local/opt/make/libexec/gnubin:/usr/local/opt/grep/libexec/gnubin:/Users/basilfx/bin:/opt/homebrew/bin:/opt/homebrew/sbin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/Applications/VMware Fusion.app/Contents/Public:/Library/Frameworks/Mono.framework/Versions/Current/Commands:/Applications/Wireshark.app/Contents/MacOS \
		make --no-print-directory -C /Users/basilfx/Desktop/RIOT/RIOT_knx/RIOT/dist/tools/riotboot_gen_hdr all
make[1]: Leaving directory '/Users/basilfx/Desktop/RIOT/RIOT_knx/RIOT/pkg/littlefs'
env: Fusion.app/Contents/Public:/Library/Frameworks/Mono.framework/Versions/Current/Commands:/Applications/Wireshark.app/Contents/MacOS: No such file or directory
```

### Testing procedure
Add something to your PATH with a space, then compile `tests/riotboot`.

### Issues/PRs references
None
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * riotboot: quote PATH variable

-- File Changes --

    M makefiles/boot/riotboot.mk (4)

-- Patch Links --

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


More information about the notifications mailing list