[riot-notifications] [RIOT-OS/RIOT] riotboot: quote PATH variable (#15389)
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: 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
You can view, comment on, or merge this pull request online at:
-- Commit Summary --
* riotboot: quote PATH variable
-- File Changes --
M makefiles/boot/riotboot.mk (4)
-- Patch Links --
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the notifications