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

Compile error #1

Closed
MarcFinns opened this issue Mar 8, 2017 · 5 comments
Closed

Compile error #1

MarcFinns opened this issue Mar 8, 2017 · 5 comments

Comments

@MarcFinns
Copy link

MarcFinns commented Mar 8, 2017

Hi Bodmer,
Not sure if I made a mistake, but after installing the lib the example does not compile (on Mac) - see below.
I did edit the User_Setup.h (as I did in you older library) but i get the error below. It appears as the zip from GitHub has a patch problem. Is there any additional configuration step required? thanks for your time!

/Users/Marc/Documents/Arduino/libraries/TFT_eSPI-master/TFT_eSPI.h:27:46: fatal error: TFT_Drivers\ILI9341_Defines.h: No such file or directory
#include <TFT_Drivers\ILI9341_Defines.h>
^
compilation terminated.
exit status 1
Error compiling for board NodeMCU 1.0 (ESP-12E Module).

@pipi61
Copy link

pipi61 commented Mar 9, 2017

Hi!
in win81x64 Arduino169_espver230, I try several examples(Wemos D1 mini + ILI9341 2,8" 320x240), and works, really fast
Great works!

@MarcFinns
Copy link
Author

Now that I look at it again with a fresher mind, I suspect the issue is the backslash in the path. This fails on unix

@MarcFinns
Copy link
Author

Confirmed. The lib uses hardcoded paths with backslash () all over. This works on windows but fails on Mac and Linux, that require forward slash (/).
Any chances to conditionally define them? Otherwise at every upgrade a tedious selective search & replace is necessary...
Other than that, a great piece of work!

@Bodmer
Copy link
Owner

Bodmer commented Mar 9, 2017

@MarcFinns
Thanks for pointing this out, have changed \ to / in the file paths so it should work for both Windows and Linux now!

@Bodmer Bodmer closed this as completed Mar 9, 2017
@MarcFinns
Copy link
Author

Thanks! Great lib by the way!

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

No branches or pull requests

3 participants