pico-extras icon indicating copy to clipboard operation
pico-extras copied to clipboard

nonesense message in pico_extras_import.cmake

Open smithps opened this issue 3 years ago • 4 comments

Line 53 has the message;

"PICO EXTRAS location was not specified. Please set PICO_EXTRAS_PATH or set PICO_EXTRAS_FETCH_FROM_GIT to on to fetch from git." should it really be;

... PICO_EXTRAS_FETCH_FROM_GIT to fetch from git.

smithps avatar Jul 19 '22 08:07 smithps

It defaults to being 'off', so you have to set it 'on' to make use of that feature? :shrug: https://github.com/raspberrypi/pico-extras/blob/master/external/pico_extras_import.cmake#L52

lurch avatar Jul 19 '22 08:07 lurch

It defaults to being 'off', so you have to set it 'on' to make use of that feature? 🤷 https://github.com/raspberrypi/pico-extras/blob/master/external/pico_extras_import.cmake#L52

Ahhhh !!! now the message makes sense. I would suggest phrasing the message as such though; changing the 'on' in the message to 'ON'

PICO EXTRAS location was not specified. Please set PICO_EXTRAS_PATH or set PICO_EXTRAS_FETCH_FROM_GIT to ON to fetch from git.

Perhaps autocorrect somewhere along the line made it lowercase - because of that I didn't read it as intended.

I also misquoted the line number, but I meant the message as quoted above.

smithps avatar Jul 19 '22 08:07 smithps

I agree with @smithps, changing to "ON" would make that a lot clearer.

JamesH65 avatar Jul 19 '22 08:07 JamesH65

Feel free to submit a PR @smithps

lurch avatar Jul 19 '22 09:07 lurch