-
Notifications
You must be signed in to change notification settings - Fork 862
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
fixed compile issues and warnings for iOS platform #2852
Conversation
…rsion 3.0, CMake >= 3.28 prefer that this command never be called and the NEW behavior for this policy is to issue a FATAL_ERROR when the command is called.
…riables or keywords when unquoted
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## master #2852 +/- ##
==========================================
- Coverage 67.04% 66.48% -0.56%
==========================================
Files 102 102
Lines 20368 20368
==========================================
- Hits 13656 13542 -114
- Misses 6712 6826 +114 ☔ View full report in Codecov by Sentry. |
CMake 2.8.12 supports |
But |
I have no idea why Travis-CI build fails. I tried to get the job logs from travis-ci, but unable to access the job logs. It said:
If the travis-ci job log and CMake log are available, I would be happy to assist in identifying and resolving the issue. |
All I can see is that there's some initial cmake checks that finally gets "errors ocurred!" by completely no visible reason. Rest of the things are post-factum and some general error report from a command. |
Right, not much insight, except that obviously CMake fails somewhere.
|
See also "/home/travis/build/Haivision/srt/CMakeFiles/CMakeOutput.log". |
That's this problem with these machines. You can only run some script afterwards that will extract some information from the files, but you can't access files through any shell or something. |
cmake version 3.5.1-1ubuntu3.
I'm afraid it is not available. What can be done is to echo the contents to the terminal in this PR. |
The No documents available for cmake version older than v3.0. While I have tested this functionality on CMake version 3.28, I cannot confirm its compatibility with CMake version 2.8.12. |
There are likely also other |
I have downloaded CMake version 2.8.12 from CMake release v2.8.12.2. Upon inspecting the execute_process(COMMAND ${CMAKE_CXX_COMPILER}
${CMAKE_CXX_COMPILER_ARG1} -dumpversion
OUTPUT_VARIABLE _GXX_VERSION
) This suggests that the syntax remains consistent even in CMake version 2.8.12. |
So, likely you have to modify the Travis execution configuration by checking the error from cmake command and dump its log file on the screen. |
Right, the syntax seems to be the same. |
I suppose |
…ted on version less than 3.7
travis-ci build pass now. |
@kgbook Thanks a lot for this PR! |
fixed issue:
exec_program
is deprecated since version 3.0, CMake >= 3.28 prefer that this command never be called and theNEW
behavior for this policy is to issue aFATAL_ERROR
when the command is called.fixed warning: cmake policy CMP0054.
update .gitignore: ignore files generated by CLion and CMake.