You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
PR #706 greatly improved the docker images. However, we need a second pass to clean up / reorganize some elements. See the list below:
the build dir is often volatile in projects that use cmake. I suggest that it is turned into docker as before the PR as it hosts the Docker build documentation as well.
the *.Dockerfile should not be at the root of the repository, let's have them into the docker directory above.
if the build.sh script is generic enough that it can be used as an alternative to the building steps we outline in the documentation, then it could/should go at the root of the repository. If this script eases all builds, then we could re-write the build documentation accordingly in README.md.
The text was updated successfully, but these errors were encountered:
PR #706 greatly improved the docker images. However, we need a second pass to clean up / reorganize some elements. See the list below:
build
dir is often volatile in projects that usecmake
. I suggest that it is turned intodocker
as before the PR as it hosts the Docker build documentation as well.*.Dockerfile
should not be at the root of the repository, let's have them into thedocker
directory above.build.sh
script is generic enough that it can be used as an alternative to the building steps we outline in the documentation, then it could/should go at the root of the repository. If this script eases all builds, then we could re-write the build documentation accordingly inREADME.md
.The text was updated successfully, but these errors were encountered: