Skip to content

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

Merged
merged 1 commit into from
Mar 10, 2017

Conversation

eriknyquist
Copy link
Contributor

@eriknyquist eriknyquist commented Mar 8, 2017

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....

@eriknyquist eriknyquist added the bug label Mar 8, 2017
@eriknyquist eriknyquist added this to the Deneb milestone Mar 8, 2017
@eriknyquist eriknyquist self-assigned this Mar 8, 2017
@eriknyquist eriknyquist requested review from facchinm and noelpaz March 8, 2017 22:03
@eriknyquist
Copy link
Contributor Author

eriknyquist commented Mar 8, 2017

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

@noelpaz
Copy link
Contributor

noelpaz commented Mar 8, 2017

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

Copy link
Contributor

@noelpaz noelpaz left a 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

@noelpaz
Copy link
Contributor

noelpaz commented Mar 9, 2017

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

In file included from /Users/<path snipped>/libraries/CurieSerialFlash/examples/CopyFromSerial/CopyFromSerial.ino:64:0: /Users/nfpaz/Library/Arduino15/packages/Intel-Test/hardware/arc32/0.0.214/libraries/CurieSerialFlash/CurieSerialFlash.h:4:25: fatal error: SerialFlash.h: No such file or directory #include "SerialFlash.h" ^

@facchinm
Copy link
Member

facchinm commented Mar 9, 2017

Hi @eriknyquist ,
my PR was prepared before the merge of #288, hence the stray file in SerialFlash folder; Github resolved the merge conflict in a quite funny way, so it is safe to remove that file/folder.
@noelpaz you need to install SerialFlash by PaulS in order to compile CurieSerialFlash. The intention was to free you from the burden of maintaining a library with very few differences (a couple of defines) from the mainline. We did the same for Adafruit's Neopixel and others.

Copy link
Member

@facchinm facchinm left a 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

@noelpaz
Copy link
Contributor

noelpaz commented Mar 9, 2017

@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.

Old SerialFlash directory was still there-- delete it
@eriknyquist
Copy link
Contributor Author

@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

@eriknyquist eriknyquist merged commit 9c079d1 into arduino:master Mar 10, 2017
@eriknyquist eriknyquist deleted the serialflash_fix branch March 10, 2017 20:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants