avrdudes / avrdude

AVRDUDE is a utility to program AVR microcontrollers
GNU General Public License v2.0
730 stars 136 forks source link

CH341A futher improvement -- to support MSVC build #1339

Closed mcuee closed 1 year ago

mcuee commented 1 year ago

Other than #1336, the other main issue with https://github.com/avrdudes/avrdude/pull/1215 is that it only supports libusb-1.0 API. Therefore the default Windows MSVC build will not work since it uses avrdude-libusb which only supports libusb-0.1 API.

It will be good to add support for MSVC build via avrdude-libusb.

stefanrueger commented 1 year ago

It is unclear to me what a path to solving this issue is. None of the current maintainers have given this priority, so it is unlikely to be fixed anytime soon. There are (literally) 15 open issues mentioning libusb. This particular one sounds to me that time might solve it, eg, once MSVC supports libusb-1.0. In which case doing nothing might be OK? I suggest closing as won't fix.

mcuee commented 1 year ago

@stefanrueger

I think it is okay for an issue (especially enhancement request, or low impact bug) to exist for long. In avrdude we have very old issues which no one is working on. Netheverless they are still valid issues.

As for MSVC build, right now it is the default Windows binary provided when releasing avrdude. And it is using avrdude specific avrdude-libusb which was very useful when libusb-1.0 library had major problems with libusb0.sys. With libusb-1.0.26 release (on 11-April-2022), I see less issues (but there are still some issues) with libusb.sys now.

We can swtich msvc build to use official libusb-1.0 project and then this issue and #968 can be closed. However, we should hear @mariusgreuel's opinion on this topic. Take note that #968 is not a critical issue and this #1339 is just an enhancement,

mcuee commented 1 year ago

Issues related to msvc build using avrdude-libusb.

mcuee commented 1 year ago

There are (literally) 15 open issues mentioning libusb.

https://github.com/avrdudes/avrdude/issues?q=is%3Aissue+is%3Aopen+libusb

I will explain one by one.

So in the end, only two issues are really related to the use of avrdude-libusb for MSVC build under Windows.

mcuee commented 1 year ago

There are two enhancement which may benefit avrdude project to bridge the difference between different platforms.

stefanrueger commented 1 year ago

@mcuee Thanks for analysing and closing some of the issues that no longer need attention.