Skip to content

Upgrade to CMake 3.11#825

Merged
klecki merged 2 commits intoNVIDIA:masterfrom
klecki:cmake-upgrade
Apr 29, 2019
Merged

Upgrade to CMake 3.11#825
klecki merged 2 commits intoNVIDIA:masterfrom
klecki:cmake-upgrade

Conversation

@klecki
Copy link
Contributor

@klecki klecki commented Apr 26, 2019

CMake 3.11 was released on March 28th, 2018.

Signed-off-by: Krzysztof Lecki klecki@nvidia.com

CMake 3.11 was released on March 28th, 2018.

Signed-off-by: Krzysztof Lecki <klecki@nvidia.com>
@klecki
Copy link
Contributor Author

klecki commented Apr 26, 2019

!build

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [717822]: BUILD STARTED

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [717822]: BUILD FAILED

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [717828]: BUILD STARTED

# limitations under the License.

cmake_minimum_required(VERSION 3.5)
cmake_minimum_required(VERSION 3.11)
Copy link
Contributor

Choose a reason for hiding this comment

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

any reason behind this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We have some workarounds for older CMakes, that could be cleaned up along the way, and I want to use newer FindCuda that has better support for cross-compiling.

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [717828]: BUILD FAILED

@JanuszL
Copy link
Contributor

JanuszL commented Apr 26, 2019

Please update the readme requirements for DALI build, add information how to update CMake to given version and we can remove FindProtobuf.cmake.

@klecki
Copy link
Contributor Author

klecki commented Apr 26, 2019

Please update the readme requirements for DALI build, add information how to update CMake to given version and we can remove FindProtobuf.cmake.

Will do, but I intend to leave FindProtobuf.cmake for now, as it does not understand cross-compilation AFAIK.

Signed-off-by: Krzysztof Lecki <klecki@nvidia.com>
@dali-automaton
Copy link
Collaborator

CI MESSAGE: [717828]: BUILD PASSED

@NVIDIA NVIDIA deleted a comment from dali-automaton Apr 28, 2019
@NVIDIA NVIDIA deleted a comment from dali-automaton Apr 28, 2019
@dali-automaton
Copy link
Collaborator

CI MESSAGE: [719793]: BUILD STARTED

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [719793]: BUILD PASSED

@klecki klecki merged commit e02e992 into NVIDIA:master Apr 29, 2019
kychennv pushed a commit to kychennv/DALI that referenced this pull request May 14, 2019
CMake 3.11 was released on March 28th, 2018.

Signed-off-by: Krzysztof Lecki <klecki@nvidia.com>
Signed-off-by: kychennv <kychen@nvidia.com>
haoxintong pushed a commit to haoxintong/DALI that referenced this pull request Jul 16, 2019
CMake 3.11 was released on March 28th, 2018.

Signed-off-by: Krzysztof Lecki <klecki@nvidia.com>
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.

5 participants