Incomplete error message about arduino-builder HOT 8 CLOSED

I wonder if the full path to the platform.txt might be more useful, but I guess you don't have access to that information right there.

from arduino-builder.

I do but it would be quite a change in how error messages are reported. That one as well as others came straight from the IDE. Let's keep them this way for some time and see how much feedback comes

from arduino-builder.

Looking more closely at the check that causes this error message makes me wonder if it needs to be present at all? Why would arduino-builder enforce having a compiler.path variable? It seems this is an implementation detail of the specific platform. If the platform does not want a compiler.path, but wants to code the path directly int the recipes, or perhaps have c-compiler.path and cpp-compiler.path that should also be possible and this check can be entirely removed?

The reason the check is present, I think, is that the compiler used to define compiler.path, but now platforms should define it themselves. Instead of checking for compiler.path explicitely, wouldn't it be more useful to check for any undefined placeholders inside recipes? That might catch even more problems, and lead to better error messages (I've seen gcc complain with "invalid mcu specified: {board.mcu}" or something similar when for some reason board.mcu is not defined).

If the check stays, I think it should allow an empty string as a valid value, which it currently does not. This is what caused the error here: I have overridden compiler.path to the empty value, so the system-wide avr-gcc is used instead of the bundled one.

from arduino-builder.

Uhm yeah, I think it's a legacy from old IDE versions. @cmaglie what do you think? Can you get rid of that check?

from arduino-builder.

For me it can be removed.

This commit contains a bit of history:
arduino/Arduino@a89f5e6

from arduino-builder.

(edited the previous comment)

Previously the compiler.path was set by the IDE to the shipped avr-gcc or, if not found, to the system wide avr-gcc. This has been removed with the latest IDE from long time, so I guess the variable is no longer mandatory.

from arduino-builder.

Thanks @cmaglie, warning removed

from arduino-builder.

Right, so it was originally a warning and set a default compiler.path value, but it turned into an error with arduino-builder. Seems like a good thing to remove it now indeed. Also, yay for detailed commit messages ;-)

from arduino-builder.