Signed-off-by: Lays Rodrigues <lays.silva@tvglobo.com.br>
Details
Details
Diff Detail
Diff Detail
- Repository
- R232 AtCore
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
| testclient/CMakeLists.txt | ||
|---|---|---|
| 45 | the above else is only run if (APPLE) please put this in that else. (line 34). Will test on mac os as soon as I get a chance. | |
| testclient/CMakeLists.txt | ||
|---|---|---|
| 45 | why in the else? it seems right to me. | |
| testclient/CMakeLists.txt | ||
|---|---|---|
| 45 | We would have if done as commited. line 22 : if (NOT APPLE)
options for almost all builds..
line 34: else ()
Apple specific options.
line 38: end if ()
line 39: if (APPLE)
more apple specific stuff
line 46: endif()See the second if is not needed and its contens should go in the previous else since that is already equilvent to if(APPLE) | |
| testclient/CMakeLists.txt | ||
|---|---|---|
| 44 | This line is still here ? | |