-
Notifications
You must be signed in to change notification settings - Fork 279
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
multiple definition of void aREST::addToBuffer in PlatformIO #278
Comments
There is one big bug in this library. Definition and declaration is in one file. So, when you you use aREST.h in more file... linker won't work. |
@marcoschwartz do you agree? would you agree on splitting the lib into .h and .cpp file? |
Yes, it will be nice.
Thnx
D.
---------- Původní e-mail ----------
Od: Daniele Palumbo ***@***.***>
Komu: marcoschwartz/aREST ***@***.***>
Datum: 26. 5. 2021 18:20:44
Předmět: Re: [marcoschwartz/aREST] multiple definition of void aREST::
addToBuffer in PlatformIO (#278)
"
@marcoschwartz(https://github.com/marcoschwartz) do you agree? would you
agree on splitting the lib into .h and .cpp file?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
(#278 (comment)),
or unsubscribe
(https://github.com/notifications/unsubscribe-auth/AAHACVK2HMNUOHSYR2DZXRLTPUNVNANCNFSM4KPTAENQ)
.
"
|
I just spent several hours trying to figure this out. This makes it impossible to have multiple source files in a platformio project. Has anyone figured out a work around? |
Standard structure of h and cpp file allows for using this library in more complex projects where several libraries need to link this file. aims at fixing marcoschwartz#278. Note: the code has not been changed as to my best knowledge, however, I needed to add anodher include ethernet.h in line 75 of the header file.
I've stumbled upon the same problem and went ahead to split up the header into several files. Result is in the abovementioned pull request that seems to work well for me so far. |
Hi I can't work with this library in PlatformIO.
When building, I encounter the following Errors. What should I do?
The text was updated successfully, but these errors were encountered: