-
-
Notifications
You must be signed in to change notification settings - Fork 7k
Update liblistserial to 1.1.3 #4971
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
3290c14
to
0a24188
Compare
Thanks, but does not work for me (tm). Crash log from the Windows _debug.exe version: |
@HorstBaerbel , can I ask you to run |
The program crashes with an Access Violation. If you compile with debug information I can tell you where exactly. I have VS2013 installed for debugging. |
I can only compile it using |
Before unfolding the debugger, can you try running the same small executable after setting LIBSERIALPORT_DEBUG environment variable? |
COM5 is the builtin 3G modem in my Laptop. It does NOT show up in the device manager under "COM & LPT" like COM3, 4, 6, 7, but under "Modems". |
And I guess compiling would take me days when looking at the sources... No mention of VS whatsoever, lots of dependencies, no CMake... |
Here we go |
Oh, and the crashes came when upgrading from 1.6.7 or 8 to 1.6.9... |
Argh, the COM5 is listed as SP_TRANSPORT_NATIVE, so it shouldn't call the USB-related loop (lines 426 to 462). |
I'm not at home atm, but I'll gladly test more on Wednesday if it helps you find the problem... |
I could test some binaries today or tomorrow. Will be unable to test saturday+sunday again... |
Some fuel for the fire. IDE 1.6.5-r2 works fine. 1.6.8 crashed on startup, didn't pursue it, just went back. Just downloaded 1.6.9 and same problem as 1.6.8 so I decided to look into it and read the posts. In the end, I have a GPS using u-blox 6 driver on COM5. Unplug this and IDE starts no problem. Tried changing the GPS to COM20, IDE won't start. So there seems to be a problem when IDE starts and looks for available COM ports and doesn't play well with others. |
Same problem here with Arduino IDE 1.6.9 on Windows 8.1-64; crash dump: output of liblistSerials-1.1.1:
of course COM9 is the USB-GPS ....; output of liblistSerials-1.1.0 which works - no crash:
output of liblistSerials-1.0.7 which works - no crash:
|
d5da03d
to
2ecd801
Compare
Hi Martino, output of liblistSerials-1.1.2 (crashes):
|
Hi @gknauf , which GPS receiver are you using? I'm trying to get one to properly debug this issue |
Hi Martino, |
I have the exact same problem on 1.6.9 on Windows 7(64) with a Zebra DS457 USB barcode reader connected. |
Hi, I have started experiencing the same problem when I installed my ZTE WCDMA technologies internet modem. I tried the 570 build but it didn't help and Arduino IDE is crashing when I plug in ZTE into USB port. |
Hi Martino, just my 2ct ;-) |
I would advice against that. This will probably break like half of the Arduino IDE installations on the planet for no reason... |
The problem here is that Windows handles serial ports terribly. Coping a serial port name with a VID/PID/ISERIAL triplet take ~10 times the code we need on Linux/OSX, and the results differ a lot if the device is composite. The old code (which spawned |
Solves arduino#4696, arm binaries are now rPi1 compatible (armv6)
2ecd801
to
8f4ee6d
Compare
Hi everyone, I believe I've found the root cause of the crashes (it was a memory corruption due to charset conversions). |
✅ Build completed. Please test this code using one of the following: ⬇️ http://downloads.arduino.cc/javaide/pull_requests/arduino-PR-4971-BUILD-578-linux32.tar.xz ℹ️ The |
If you want to test it withoud downloading the full-blown IDE the link is http://downloads.arduino.cc/liblistSerials/liblistSerials-1.1.3.zip (replace the dll as usual) |
Just downloaded full IDE, replaced 1.6.5, fired up my navigation software with GPS on com16 using u-blox 6 driver, Uno on com15 and both seem to be co-existing just fine. One warning message on compile, David Watson
To: arduino/Arduino Arduino@noreply.github.com If you want to test it withoud downloading the full-blown IDE the link is http://downloads.arduino.cc/liblistSerials/liblistSerials-1.1.3.zip (replace the dll as usual)— |
Merging to give it some nightly test, revert if necessary |
Replaced listSerialsj.dll in the arduino lib folder, I can confirm it is working correctly now. Thank you very much for the quick response! |
Hi Martino, Also replaced listSerialsj.dll and now Arduino IDE works without any problem. Thanks! |
Just for completeness, I reported the other day that I did an install of the new IDE and everything worked fine. On another machine, running 1.6.9 that would not start with the GPS attached. I replaced the listSerials.dll and it too now works fine. Good work. Next time I see you, beer's on me.
To: arduino/Arduino Arduino@noreply.github.com Hi Martino,Also replaced listSerialsj.dll and now Arduino IDE works without any problem.Thanks!— |
Hi Martino, |
Sorry guys. After all that good work to get past the comm port issue, too many warnings/errors trying to compile code that works. The comm port problem allowed 1.6.9/10 to 'not crash' on startup. Since then, I've been trying it out with programs the work.
With 1.6.10 there was a page full of warnings and errors before aborting the compile. Mostly stating that TimerOne had no such methods. |
Hi David, the problem you are experiencing is a side effect of third party boards (the new avr core uses a feature only available in the new compiler, but the builder chooses the old compiler). |
Looking forward to trying this out in 1.6.10 :) |
Thank you very much! Tried again MCP2221 on Arduino IDE and now it is working great with version 1.1.3 ! |
Could solve #4696, includes arduino/libserialport@d24994c
PR for @ArduinoBot