Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

Improve CCache handling #13456

Merged
merged 16 commits into from
Dec 14, 2018
Merged

Improve CCache handling #13456

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
24d706c
Remove gitignore entries
marcoabreu Nov 29, 2018
0a2e773
Modify Makefile
marcoabreu Nov 29, 2018
84865ca
Modify user permissions
marcoabreu Nov 29, 2018
7649124
Add new ccache wrapper function
marcoabreu Nov 29, 2018
1d77c94
Change PATH rewrite to a different one to resolve CUDA issues
Nov 29, 2018
f6fd7ec
Add ccache to gpu cmake
Nov 29, 2018
2a03107
Enable ccache for every build
marcoabreu Nov 29, 2018
0ceba10
Set permissions for arm dockerfiles
marcoabreu Nov 29, 2018
ff92c48
Disable ccache for ASAN
marcoabreu Nov 29, 2018
329283c
Remove g++-8 ccache redirect
marcoabreu Nov 29, 2018
9eb9f6f
Update Android Dockerfiles for user permissions
marcoabreu Nov 29, 2018
4abbaca
Fix ASAN compiler typo
marcoabreu Nov 29, 2018
e8386a7
Remove sanity for speed
marcoabreu Nov 29, 2018
ed16144
Move build dir creation in android armv8
marcoabreu Nov 29, 2018
b61c17b
Revert "Remove sanity for speed"
marcoabreu Nov 29, 2018
194d17b
Add ccache for NVCC in Makefile
marcoabreu Nov 29, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Modify user permissions
  • Loading branch information
@marcoabreu
marcoabreu committed Nov 29, 2018
commit 84865cad7f61c6c1a890fdd96113dc99a7913351
5 changes: 5 additions & 0 deletions ci/docker/install/centos7_adduser.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -34,4 +34,9 @@ then
mkdir /work/mxnet
mkdir /work/build
chown -R jenkins_slave /work/

# Later on, we have to override the links because underlying build systems ignore our compiler settings. Thus,
# we have to give the process the proper permission to these files. This is hacky, but unfortunately
# there's no better way to do this without patching all our submodules.
chown -R jenkins_slave /usr/local/bin
Copy link
Contributor

Choose a reason for hiding this comment

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

Not clear, even with comment I don't understand why. Why can't we have some other folder added to the PATH?

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 do, that's why I created /tmp/ccache-redirects. The problem is that in some places (especially submodules), that paths have been hardcoded. That's why we have to do both :(

fi
5 changes: 5 additions & 0 deletions ci/docker/install/ubuntu_adduser.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -40,4 +40,9 @@ then
mkdir /work/mxnet
mkdir /work/build
chown -R jenkins_slave /work/

# Later on, we have to override the links because underlying build systems ignore our compiler settings. Thus,
# we have to give the process the proper permission to these files. This is hacky, but unfortunately
# there's no better way to do this without patching all our submodules.
chown -R jenkins_slave /usr/local/bin
fi