-
-
Notifications
You must be signed in to change notification settings - Fork 388
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ARDUIFINE: global compiler options extracted from ctags parsing #1117
Conversation
I would have preferred to propose something like
or
but that would have required parsing the whole |
About security, the compiler command line is affected but user is warned in the message console. This PR, apart from adding more noise to an old feature request, is another attempt to finally find an answer to arduino/Arduino#421. I'm aware that I updated in the I will be happy to close this PR if any other way is found to close arduino/Arduino#421 which has been opened for 8 years. Attempts or explorations are quite numerous, like this commit(from @facchinm in #421), or one from the link list compiled by @per1234 in #846's OP. I'll also be glad to update this PR until it fits with maintainers idea of what are arduino-cli / arduino-builder. |
@per1234, this change proposal even as a PoC is orthogonal to your above statement. #846's title "Provide dedicated sets of "extra flags" properties for platform developers and users" did make me think that it is "legal" to provide such facility to users. I have indeed two use-cases on widely used arduino libraries:
IMHO It could be a normal use for an Arduino library to need a global define to get some extended behaviour, at no cost when I would really like clarification on how Arduino plans to address such issues. edit 24 days later:
|
closed per #1517 (comment) |
Example: these lines in the
.ino
file:will globally enable the following options in the compiler command line:
That, including during compilation of the core and libraries.
If they are not otherwise used by the sketch, the symbols
ARDUIFINExxx
andARDUINOGLOBALyyy
are excluded from the final binary at link time.This PR aims at giving a solution to arduino/Arduino#421
and is maybe related to #846.
(the missing
M
is fixed)