-
Notifications
You must be signed in to change notification settings - Fork 47
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
error while compiling with platformio 3.1.0 on arduino mega #27
Comments
Arduino IDE 1.6.6 too! |
umm, i think that this lib it's dead :( RIP :D |
Same with the Arduino IDE 1.6.5 |
This is the full error text:
|
I went back to this branch and it works for me on my Frankenstein DUE |
Same with the Arduino IDE 1.6.13 URLHandler *ledHandler = webServer.createURLHandler(F("/SimpleLED.html.json")); esp.resetCb = resetCb; Maybe you can upload the old and functional example for us to play with? |
I found out that my Windows Arduino IDE is a little picky with the void/char definition. I changed the librarys in this way and was able to run the MQTT example. Please notice that I have done this without any knowledge of the internal functionality - but it works for me. ELClientResponse.h ELClientResponse.cpp 55 *data = _arg_ptr; changed to ELClientWebServer.cpp 51 static void ELClientWebServer::webServerPacketHandler(void * response) changed to 270 uint32_t cb = &webServerCb; changed to 361 hnd->loadCb( hnd->URL.c_str() ); ELClientRest.h |
The blink LED example will run after the following change - but with this you kill the web page feedback... ELClientWebServer.h 38 void ledButtonPressCb(const char * btnId) changed to Additional I had to add in the Arduino |
@freechw and @ramarro123 |
@steamboating i am not using this library at all, esp-link it's not actively maintained (as you can see not even this project it's updated or receive attention from @tve that it's the owner) he said that he is open to PR but he won't work anymore on esp-link, probably the same apply to this lib as well. i am using esp arduino port, you have to write more code probably, but it's actively developed |
@ramarro123 I strongly disagree with your statement that esp-link is not actively maintained. Just because I'm not working on the pieces you care about doesn't mean it's abandon-ware! What have you contributed to the project? Why don't you fix the issues? How much are you paying for it that you have expectations? |
@tve i have not even try to flash my esp with that, so well... i have pay 0, and i will continue with that 0 forever :) also, about the state of the project, there are a lot of issue without reply (like this one, opened 16 days ago got a reply from you today after a ping) esp-link codebase last update (and i am speaking about commit that doesn't touch docs or sub minor feature like adding baudrate on a select box) are quite old. That's what i mean with "not actively maintained" ... something that it's not... actively maintained :) And, about contribution, well, i can't /wont contrib to all project that i use (not to mention the one that i don't use because don't fit my project... and for esp-link, you miss stk500v2 and the lib doesn't compile in patformio out of the box to make me want to invest time on that). Are you a committer of linux kernel (in case you use linux) or to any of the 1230210301230 GNU prog that you use? (including less or tail) Don't expect that people want to contrib to your work, nor conclude with a "do a pr" when someone signal an issue, it's your project not mine, if i will eventually do something of nice, i won't expect that other people help me (but i will be glad to, just... don't expect it) Having said so, i repeat, your project it's not actively maintained, this lib it's old of 1 month and doesn't compile out of the box. That's facts, i was just replying to the other user that i haven't time/will to test his fixes (that are btw correct, the problem it's just the new compiler, you can use platform io probably adding the permissive switch on compiler) |
@tve thank you Thorsten for sharing your work with the github world... |
@steamboating thanks & you're welcome. I'll try to get to this this week-end, but I'm already a bit over-committed and some of the code is not mine, so it'll take a bit more to double-check everything. Some of your changes don't look 100% correct but they should all work. |
Mhh, weird, I just tried to reproduce this issue. I downloaded the latest arduino software, which is now 1.6.13, and I ran the Makefile for the MQTT example and while it spat out some warnings it worked great. This is all on Linux. The compiler flags are:
Sounds like I'm gonna have to install the IDE on Windows to try to reproduce this. |
Hello Thorsten, thank you for looking over the code again - I have tested it on two different Windows PC's - win7 and 8 with the different available Arduino IDEs mentioned above with the out of the box settings. I thought already that you must have used an other setup for testing... Hopefully this is only a Windows problem and not an additional problem because of the installed boards? I am using the out of the box Arduino DUE settings. |
Looks like I need some help with this :-). I went to https://www.arduino.cc/en/Main/Software and downloaded the current IDE for Windows. I then downloaded the current release of ELClient from https://github.com/jeelabs/el-client/releases/tag/v3.0.alpha0 (I picked the source zip archive). I unpacked the zip and copied the ELClient directory into the arduino libraries directory (C:\Program Files (x86)\Arduino\libraries). I then started the IDE, used the File>Open menu and openened the mqtt example sketch (C:\Program Files (x86)\Arduino\libraries\ELClient\examples\mqtt\mqtt.ino), selected the Arduino Mega ADK board and clicked on the checkmark button ("verify"). That happily compiled and linked the sketch. I tried the same with the webserver_led sketch with the same result. |
I did another pass at fixing the various type cast warnings. This is pushed to master. |
Thank you Thorsten, i like this kind of support - sorry? that you could not find "our" bug - I will try to find also why I have the problem and you don't - will also happely test the new version soon and tell my experience here. Thanks for all your time! |
Hello Thorsten - you did a great job - all error messages are gone on my system with the update. I am using it on a Windows 8 System with Atmel Studio 7 and a self made Arduino DUE SAM3X8 PCB on which I have hard wired the Atmel with the ESP over Serial3 while doing all the debuging via SerialUSB. In the moment I am working with the TCP demo - now i am able to make a server connection. Thanks for all your time you are inversting in this project! |
An other hint for your el-web-led example:
|
Thanks for the update, I'll have to look into the LED issue. |
here it's the log of build for ELClientSocket, doesn't seems to compile on platformio 3.1.0
The text was updated successfully, but these errors were encountered: