Skip to content
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

Include the "real" files instead of the Fakes #42

Merged
merged 1 commit into from
Mar 22, 2023

Conversation

wrong-kendall
Copy link
Contributor

Doing it this way allows PlatformIO to find the correct include files so that existing libraries still work.

This should address #39.

Doing it this way allows PlatformIO to find the correct include files so
that existing libraries still work.

This should address FabioBatSilva#39.
@FabioBatSilva
Copy link
Owner

Thanks @wrong-kendall .

Do you know why is this a problem on PlatformIO ?

@wrong-kendall
Copy link
Contributor Author

I don't really know why but I can tell what happens is that it executes:

g++ -o .pio/build/native/lib91a/protoWire/protoWire.o -c -DPLATFORMIO=60106 -DPIO_UNIT_TESTING -Ilib/protoWire -I.pio/libdeps/native/ArduinoFake/src lib/protoWire/protoWire.cpp

And that fails because the only Wire.h exists in arduino/Wire.h

[13:20:44 03/21/23]: <~/development/fake_includes>$ ls .pio/libdeps/native/ArduinoFake/src/
arduino          ArduinoFake.h  ClientFake.cpp  Client.h        fakeit            FunctionFake.h  PrintFake.h     SerialFake.h    StreamFake.h  WireFake.h
ArduinoFake.cpp  Arduino.h      ClientFake.h    CMakeLists.txt  FunctionFake.cpp  PrintFake.cpp   SerialFake.cpp  StreamFake.cpp  WireFake.cpp

@FabioBatSilva FabioBatSilva merged commit 8b400de into FabioBatSilva:master Mar 22, 2023
@FabioBatSilva
Copy link
Owner

Thanks @wrong-kendall 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants