-
-
Notifications
You must be signed in to change notification settings - Fork 283
SerialFlash: fix compile errors #467
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
Conversation
Waiting for confirmation that crc16.h is OK for us to ship with current license-- please don't merge this until I've confirmed it with a comment |
Actually it is not a compile error as there is still no INO in the IDE examples and libraries folder. I got the latest 1.8.1 library as well as the latest-nightly built. You will see the error from the libnrary manager complaining about an invalid library. I will test the new PR to see if the behavior is still there |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is now removed from the corelibs and I do not see the invalid library warning on the IDE anymore
I will have to revoke my approval. Since we were not up to speed on the move to SerialFlash and renamed to CurieSerialFlash, it was not a part of a test suite. I just compile one of the CurieSerialFlash examples and I am getting this erroe
|
Hi @eriknyquist , |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
crc16.h is not needed, the problem is that Paul still didn't tag the updated SerialFlash.
CopyFromSerial can be safely removed, instead
@facchinm Thanks. I also created another PR 468 to temporary fix the comments on the examples comments which currently tell you to use library manager to get the Serial Flash Library. Currently the library manager does no have the correct version that will work and they have to get the latest from Github. |
4c22fd2
to
cf870a4
Compare
Old SerialFlash directory was still there-- delete it
cf870a4
to
8b3aa9d
Compare
@facchinm thanks-- I'm not sure why it wasn't compiling for me without crc16.h. I may have had an old version of the library hanging around. Anyway, working now, and I've removed crc16.h |
Old SerialFlash directory was still there-- delete it
@facchinm this was partially my fault.... I didn't test your change properly. Please take a look, and verify I wasn't misunderstanding your intentions....