mathertel / Radio

An Arduino library to control FM radio chips like SI4703, SI4705, RDA5807M, TEA5767.
http://mathertel.github.io/Radio
BSD 3-Clause "New" or "Revised" License
297 stars 89 forks source link

New Arduino IDE doesn't compile the Examples #25

Closed roboticboyer closed 5 years ago

roboticboyer commented 5 years ago

I'm trying to compile the examples of the Radio library without any success There are a lot of errors in the library probably because the Arduino IDE has been updated after revision 1.6 with changes not compatible with this library E.g. Arduino.h is not yet required, RADIO.h and radio.h are different, etc. @mathertel please could you update the library to be compatible with the new Arduino IDEs Thank you Gio

NuclearPhoenixx commented 5 years ago

Would do a pull request for the old library includes in the examples. But it looks like there is more stuff broken in some examples, like the analog pin numbers. Changing this rather randomly would maybe change what @mathertel has in mind about them.

roboticboyer commented 5 years ago

My request is that @mathertel will kindly update the library. I haven't the a so big knowledge to do the changes Are you @Phoenix1747 able to do the changes and propose the pull request or a fork? Thank you Gio

NuclearPhoenixx commented 5 years ago

I could do a pull request for most of the lib examples in like two weeks. No time right now, though.

roboticboyer commented 5 years ago

I made further investigation The compilation errors are related as minimum to:

The above errors are shared everywhere in the .cpp and .h files in the /src directory and in the examples .ino file

Probably there is also the #22 issue

Unfortunately I'm not yet able to do a pull request or fork the library with my fixes

mathertel commented 5 years ago

should be fixed with release 1.1.0.

roboticboyer commented 5 years ago

It's required changing version in library.properties so the Arduino Library Manager Bot will update the Arduino libraries repository

link

Thank you Gio