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

Incombatilibity whit nextion dispaly and chipkit (2.0.3) sd library #402

Open
pgunzelmann opened this issue Apr 8, 2018 · 4 comments
Open

Comments

@pgunzelmann
Copy link

Hello,

inside of sd Library of chipkit are following defintions:
boolean exists(char *filepath);
boolean mkdir(char *filepath);
boolean remove(char *filepath);
boolean rmdir(char *filepath);
if I compare with the adruino definitons (for adruino bards) i find
// Methods to determine if the requested file path exists.
boolean exists(const char *filepath);
boolean exists(const String &filepath) { return exists(filepath.c_str()); }

// Create the requested directory heirarchy--if intermediate directories
// do not exist they will be created.
boolean mkdir(const char *filepath);
boolean mkdir(const String &filepath) { return mkdir(filepath.c_str()); }

// Delete the file.
boolean remove(const char *filepath);
boolean remove(const String &filepath) { return remove(filepath.c_str()); }

boolean rmdir(const char *filepath);
boolean rmdir(const String &filepath) { return rmdir(filepath.c_str()); }
May you will change.

I run into this issue trying out to use nextion library in combination whit chip kit.

@majenkotech
Copy link
Member

The SD library is horrendously out of date and very nasty anyway. Where possible I recommend to use DFATFS instead, but I guess in this case you can't.

Since it's just missing const attributes it should be simple enough to fix though.

@majenkotech
Copy link
Member

Possible fix in #404

@pgunzelmann
Copy link
Author

Followup:
By using Boards Manager and installed chipKIT by chipKIT Community Version 2.0.3 the issue still exists.
How do i know at which time will the change be included at package_chipkit_index.json ?
Thanks for supportig so far, a manual copy of the chaged sd files run excellent ,;)

@majenkotech
Copy link
Member

This change will be in the next released version. It takes time for that to happen. There's other fixes we need to get out there too.

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

2 participants