-
Notifications
You must be signed in to change notification settings - Fork 446
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
Update libusb to version 1.0 #673
base: master
Are you sure you want to change the base?
Conversation
I tried it on Windows 11 with PN532 UART (built on Visual Studio 2022) |
@hank9999 thank you for testing. I don't think that is project is maintained anymore. Just one pull request was merged over the last two years and no real interactions from the lead developers. I will leave this pull request open for anyone who wants to create a libusb1.0 fork and in the hopes that one day, this project will become active again |
Thank you for replying. Yep, the project seems that it is not maintained, hope someone have time to merge the changes. |
You are right, this project is not actively maintained anymore, but still working as expected. @kenspeckle1 Could you rebase your work on top of main branch in order to ease rework then merge it? |
Hi @neomilium, nice to see, that the project is not completely abandoned. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Requested changes:
- The merge commit should not be as this PR should be rebased on top of master.
- «Restore NEWS.md» commit should be squashed with the commit that wrongly removes it.
- «Bugfixes» should be squashed with the commit that introduces the bugs.
- «Apply styles for changed files» should be integrated within development commits
As the switch between libusb 0.1 to 1.0 is a huge step and can potentially introduces bugs, we need to have a perfectly clean git history. BTW, the review will be more efficient and reliable if all commits are short, understandable and well described in git log. |
Also, I will try to fix the history next weekend |
1d729e3
to
a2d86ad
Compare
a2d86ad
to
23352f1
Compare
@neomilium Sadly it took me a bit longer than anticipated but I squashed all commits into one single commit |
With the commit squash we loose the authoring, and all well-written atomic commits. AFAIR, some commits were written by @doegox , maybe some by @smortex . From my point of view its not acceptable and not more reviewable than before, when we had a big and dirty merge commit. @doegox , @smortex, maybe @iwamatsu and any available authors, as I'm not comfortable with this PR due to the impact what are your position? |
historical branch of libusb10 has commits mostly by myself, one by @LudovicRousseau and one by Lucien Judert. |
Your code violates the coding style. Additionally, the mingw-cross-compile.sh script seems redundant after this PR, so I suggest deleting it. |
Fixes #191
I reimplemented the usb code for libusb1.0.
Currently it uses https://github.com/kenspeckle1/libnfc/tree/update_libusb10_branch as base which in turn updates the libusb10 branch to match the master branch. As such it would be best if PR #672 would be merged before this or the libusb10 branch would be updated in some other way.
A review and help with testing would be greatly appreciated. I only have access to an ACR122U device and a linux pc so I am unable to test other devices or OSes.