Hi Yegor and Samuel,

On Saturday, 23 December 2017 21:45:11 CET Yegor Yefremov wrote:
> Forget about the STATICLIBS related patch, as libftdi1 creates both
> dynamic and static libs at once.

alright, I already noted down to check the git history of libftdi
why we did it that way in the first place. I would only break
backward compatibility if there's a good reason for it.
So I can skip this now - less work is always appreciated :)


Regarding the other patches:

0002-cmake-fix-FindUSB1.cmake.patch:

This seems to break the build of the unit test on my Fedora 26 box:

"
CMake Error: CMake can not determine linker language for target: test_libftdi1
CMake Error: Cannot determine link language for target "test_libftdi1".
-- Generating done
"



I tried the 0003-cmake-do-not-check-for-g-when-FTDIPP-is-disabled.patch
independently, but it breaks the build of the unit test, too.
Also it introduces a variable "FTDIPP" without a corresponding cmake option?


Please look into those patches again to ensure the unit test
is still buildable and the default behavior doesn't change. Thanks!

I'll soon be on a short vacation (lots of 34c3 presentations to watch...),
so there'll be a small delay until I can look into this.

Cheers,
Thomas


--
libftdi - see http://www.intra2net.com/en/developer/libftdi for details.
To unsubscribe send a mail to libftdi+unsubscr...@developer.intra2net.com   

Reply via email to