-
Notifications
You must be signed in to change notification settings - Fork 80
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
Trivial sketch won't compile using Arduino 1.8.13 #56
Comments
The problem appears to be that if you have other IDEs installed (e.g. Visual Studio), the include search paths find the platforms |
I think there's another reason: new versions of the AVR core have a header file 'new' that conflicts with the one from ArduinoSTL (arduino/ArduinoCore-avr@d1ae194). If anyone opens the Arduino IDE, goes to Tools -> Board -> Board manager and updates the Arduino AVR Boards to version 1.8.3, he/she will likely have the same problem. The result is the same, so nbolthaus' pull request should solve it all the same. Workaround in the meantime: downgrade Arduino AVR Boards to 1.8.2. |
Hi. I'll take a look at fixing this. I haven't used Arduino in a while. |
I can confirm I am experiencing the same problems as @roelschroeven using IDE 1.8.3 for AVR devices |
This is indeed a problem triggered by Arduino adding a I believe the proper solution on the Arduino end is to make sure that it implements the |
I created a PR for the Arduino AVR core, testing is welcome: arduino/ArduinoCore-avr#361 That almost solves this issue, the one remaining problem I saw is that I don't think this can be fixed on the Arduino side, but I can see two ways to fix this in ArduinoSTL:
I'm not sure which would be the best solution, though. |
The PR in the AVR core is merged, so that leaves just the second issue I noted above. |
Thank you for your efforts at making ArduinoSTL work with AVR boards 1.8.3 |
Hm, I just noticed a new problem:
I'm not sure why this didn't happen to me before, since it quite makes sense that it happens: Both Arduino (new.cpp, since merging arduino/ArduinoCore-avr#361) and ArduinoSTL (new_handler.cpp) declare the I'm not sure how to solve this, maybe it works to make it |
Instead of replacing every occurrence of (On MacOS the file is /Users/user/Library/Arduino15/packages/arduino/hardware/avr/1.8.3/cores/arduino/new) |
Hello i run into the same problem, just found this lib to use vectors... i use Visual Studio with Visual Micro and the Arduino IDE V1.8.13 in the background. i have tried include with <> and with "" doesnt matter... Thanks for any Help!! Schweregrad Code Beschreibung Projekt Datei Zeile Unterdrückungszustand |
Thank you for you work. I think ArduinoSTL may need some work too. Even changing the include in new_handler.cpp to use "new" I'm getting a linker error: Error linking for board ATmega328P (5V, 16 MHz) (Arduino Pro or Pro Mini) collect2.exe*: error: ld returned 1 exit status |
thanks man after down grading my problum is solved |
The fix suggested here will likely only work for the AVR targets. If you are using one of the ARM-based Arduinos such as MKR 1010 WiFi, then that uses the ARM toolchain, which has the Standard Template Library built in. In that case you must use |
Hello! I've updated the library and it now compiles using the latest Arduino IDE. Sorry to make you all wait! |
I believe this issue is not entirely fixed yet, in particular #56 (comment) is unresolved. See also arduino-libraries/Arduino_AVRSTL#2 (comment) for some more background, and arduino-libraries/Arduino_AVRSTL#8 for a possible fix (hardly tested, so feedback welcome). |
I confirm with @matthijskooijman -- this is not fixed. Only fix so far is downgrading Arduino AVR board to 1.8.2. |
Have the same error with Arduino IDE 2.0.0. Downgrading board to 1.8.2 works |
@mike-matera I would also like to follow up, this is not fixed. |
I've submitted a fix (last year already) for this here: #82 |
This library worked mostly fine for me in Arduino 1.8.12, but in 1.8.13 the following trivial sketch won't compile. My intuition is that it's an include-order dependency issue.
with the following errors:
The text was updated successfully, but these errors were encountered: