windows compiling with native tools and clang, macos catalina compiling, added travis ci build tests #611
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
doegox
reviewed
Jul 7, 2020
doegox
reviewed
Jul 7, 2020
doegox
reviewed
Jul 7, 2020
|
very impressive, including Travis, thanks a lot ! |
This was referenced Jul 13, 2020
Closed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Compiles fine in windows with cmake, with clang, clang-cl and cl. Compiles fine in linux with autotools, cmake, clang and gcc. Compiles fine in macos catalina with cmake, autotools and clang. Added travis ci build test of these. I think windows mingw is broken, at least with 8.1 that travis ships. There is no 64bit usblib and i was unable to tell wingw to take 32bit libs when compiles, but i've not lost much time on this frankly, since msvc build just works. Maybe the next move will be to drop autotools and use only cmake, and migrate to libusb1 with libusb-compat wrapper. That's all for now.