Skip to content

Build system fixes / improvements#33

Open
nebw wants to merge 4 commits into
joschu:masterfrom
nebw:master
Open

Build system fixes / improvements#33
nebw wants to merge 4 commits into
joschu:masterfrom
nebw:master

Conversation

@nebw

@nebw nebw commented Sep 26, 2015

Copy link
Copy Markdown

I wasn't able to build cgt on my system, so I had to make a couple of modifications to the CMake script:

  • I already have OpenBLAS installed. I such a case, cgt should not download and compile OpenBLAS. I added FindOpenBLAS.cmake from the caffe repository and modified the CMake script so that OpenBLAS is only built from scratch if FindOpenBLAS.cmake can't find OpenBLAS on the system
  • appending to CMAKE_C_FLAGS/CMAKE_CXX_FLAGS broke the build on my system because it adds a ';' before appending the string. According to the CMake docs, APPEND is meant for lists (not strings), which are sepearted by semicolons in CMake.

Comment thread CMakeLists.txt

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants