Skip to content
This repository was archived by the owner on Jul 1, 2023. It is now read-only.

build: bump to CMake 3.16, centralise artifacts #1063

Merged
merged 1 commit into from
Aug 19, 2020

Conversation

compnerd
Copy link
Contributor

Put all the libraries into lib, all the binaries into bin. This bumps
up the CMake requirement to 3.16.0 to enable the co-located artifacts.
It should make it easier for people to find the artifacts in the build
tree.

Copy link

@ematejska ematejska left a comment

Choose a reason for hiding this comment

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

Can you update the Readme.md with the new cmake requirement? Also, do you think that anything needs to be adjusted to make sure Rapid builds keep working?

@compnerd
Copy link
Contributor Author

Seems that we didn't document the requirements (I suppose because the intent was that people would be using s-p-m). I don't think that anything should be needed for that, the layout of the tree should be internal to the project.

Put all the libraries into lib, all the binaries into bin.  This bumps
up the CMake requirement to 3.16.0 to enable the co-located artifacts.
It should make it easier for people to find the artifacts in the build
tree.
@ematejska ematejska merged commit b88bbf9 into tensorflow:master Aug 19, 2020
@compnerd compnerd deleted the colocation branch August 20, 2020 15:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants