-
Notifications
You must be signed in to change notification settings - Fork 839
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
correct cmake script to support Open Watcom toolchain #8167
Conversation
Can one of the admins verify this patch? |
Hi @jmalak , Can you tell us more about your project and interest in our library? I recently worked with a customer for Watcom-C compiler support, but I did not try it with CMake. We do require a contributor agreement to be in place to accept code changes. We could treat this as a bug report if that becomes an issue. Are you planning additional pull requests? Thanks, David Garske, wolfSSL |
The main interest is to compile WolfSSL by Open Watcom toolchain, because OpenSSL is too much non-standard and requires many patches to do it working with Open Watcom. I thing that WolfSSL is more suitable to port to Open Watcom with much smaller number of changes. It should enable Open Watcom users to use WolfSSL for theirs applications. Main interest is build for Linux/Windows/OS/2 targets.
Yes I am planning to do more pull requests because in WolfSSL are some incompatibilities with Open Watcom which will requires small changes by example for Windows thread support. Please If you need some agreement then give me some link to info how to process. |
Hi @jmalak , That's wonderful! To start the process of becoming an approved contributor please email [email protected] and refernce a link to this PR and include your country of residence. Thanks, |
6acec23
to
09f3718
Compare
Sorry, I put some WIP into this branch. Now it is corrected. |
ok to test |
retest this please |
Open Watcom doesn't support -Wall it uses -wx instead -wcd=202 suppress message about unreferenced functions
It's been sitting here without a review for over a month. |
Hi @jmalak , Thank you for the reminder on this. I checked and you are an approved contributor. I will personally review this on Monday. Thanks, |
Jenkins... retest this please: Failure was NOT related to this PR "FIPSv2-regression" |
Hi @jmalak , We should be able to get these PR's merged quickly, but I would like to run some tests. Can you please share the example cmake command line you used for building wolfSSL with open watcom? I've tried a few things.
Thanks, |
Hi @dgarske, I am not sure what is best method to check it. Create build subdirectory to have separated build objects from WolfSSL tree. Then you can use following cmake command to configure WolfSSL in CMake for Open Watcom cmake -B build -G "Watcom WMake" -D CMAKE_VERBOSE_MAKEFILE=TRUE -D CMAKE_SYSTEM_NAME=linux -D CMAKE_SYSTEM_PROCESSOR=x86 -D WOLFSSL_ASM=no -DCMAKE_BUILD_TYPE=Release Next you can run build. Regards Jiri |
Hi @jmalak , Again thank you so much for these PR's. I am getting further, but now getting a linker error:
Any ideas for how to solve that? The end result of this will be a new CI test for Watcom C. Thanks, |
This is only the first step that is necessary, but it does not mean that the compilation will be successful. This issue blocked start compilation by Open Watcom toolchain at all with CMake. I have a second PR #8175 with a thread launch fix that solves the issue with Open Watcom calling convention for thread start function and the problem with the function never returning. I try to proceed with smaller changes rather than making a big bang (all in one change). It is simpler for me to learn WolfSSL codebase gradually. Anyway I found out one mistake in Open Watcom compiler preprocessor which first need to fix. Regards Jiri |
Oh sorry, I primary works on Windows version now that Linux version can have different issue due to POSIX thread implementation. The best is test Windows build, you can do it on Linux by CMake command cmake -B build -G "Watcom WMake" -D CMAKE_VERBOSE_MAKEFILE=TRUE -D CMAKE_SYSTEM_NAME=Windows -D CMAKE_SYSTEM_PROCESSOR=x86 -D WOLFSSL_ASM=no -DCMAKE_BUILD_TYPE=Release Open Watcom cross-compilation is fully supported by CMake that you can build also on Windows for Linux etc. Jiri |
To the issue with linker is due to Open Watcom doesn't support shared libraries on Linux yet, it should be configured for linux to use static libraries. For Windows target it should not be a problem the DLL is fully supported. |
* Correct cmake script to support Open Watcom toolchain (wolfSSL#8167) * Fix thread start callback prototype for Open Watcom toolchain (wolfSSL#8175) * Added GitHub CI action
* Correct cmake script to support Open Watcom toolchain (wolfSSL#8167) * Fix thread start callback prototype for Open Watcom toolchain (wolfSSL#8175) * Added GitHub CI action
* Correct cmake script to support Open Watcom toolchain (wolfSSL#8167) * Fix thread start callback prototype for Open Watcom toolchain (wolfSSL#8175) * Added GitHub CI action
* Correct cmake script to support Open Watcom toolchain (wolfSSL#8167) * Fix thread start callback prototype for Open Watcom toolchain (wolfSSL#8175) * Added GitHub CI action
* Correct cmake script to support Open Watcom toolchain (wolfSSL#8167) * Fix thread start callback prototype for Open Watcom toolchain (wolfSSL#8175) * Added GitHub CI action
* Correct cmake script to support Open Watcom toolchain (wolfSSL#8167) * Fix thread start callback prototype for Open Watcom toolchain (wolfSSL#8175) * Added GitHub CI action
* Correct cmake script to support Open Watcom toolchain (wolfSSL#8167) * Fix thread start callback prototype for Open Watcom toolchain (wolfSSL#8175) * Added GitHub CI action
* Correct cmake script to support Open Watcom toolchain (wolfSSL#8167) * Fix thread start callback prototype for Open Watcom toolchain (wolfSSL#8175) * Added GitHub CI action
* Correct cmake script to support Open Watcom toolchain (wolfSSL#8167) * Fix thread start callback prototype for Open Watcom toolchain (wolfSSL#8175) * Added GitHub CI action
* Correct cmake script to support Open Watcom toolchain (wolfSSL#8167) * Fix thread start callback prototype for Open Watcom toolchain (wolfSSL#8175) * Added GitHub CI action
* Correct cmake script to support Open Watcom toolchain (wolfSSL#8167) * Fix thread start callback prototype for Open Watcom toolchain (wolfSSL#8175) * Added GitHub CI action
* Correct cmake script to support Open Watcom toolchain (wolfSSL#8167) * Fix thread start callback prototype for Open Watcom toolchain (wolfSSL#8175) * Added GitHub CI action
* Correct cmake script to support Open Watcom toolchain (wolfSSL#8167) * Fix thread start callback prototype for Open Watcom toolchain (wolfSSL#8175) * Added GitHub CI action
* Correct cmake script to support Open Watcom toolchain (wolfSSL#8167) * Fix thread start callback prototype for Open Watcom toolchain (wolfSSL#8175) * Added GitHub CI action
* Correct cmake script to support Open Watcom toolchain (wolfSSL#8167) * Fix thread start callback prototype for Open Watcom toolchain (wolfSSL#8175) * Added GitHub CI action
Closing in favor of #8391 |
* Correct cmake script to support Open Watcom toolchain (wolfSSL#8167) * Fix thread start callback prototype for Open Watcom toolchain (wolfSSL#8175) * Added GitHub CI action
* Correct cmake script to support Open Watcom toolchain (wolfSSL#8167) * Fix thread start callback prototype for Open Watcom toolchain (wolfSSL#8175) * Added GitHub CI action
Description
CMake build by Open Watcom toolchain failed due to -Wall option
Open Watcom doesn't support -Wall option it uses -wx instead
-wcd=202 option suppress message about unreferenced functions
Testing
Run WolfSSL CMake build with Open Watcom toolchain
It will not report incorrect -Wall option