Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docker: Default settings should provide a volume for git-cache #8267

Closed
jnohlgard opened this issue Dec 14, 2017 · 2 comments
Closed

docker: Default settings should provide a volume for git-cache #8267

jnohlgard opened this issue Dec 14, 2017 · 2 comments
Assignees
Labels
Area: build system Area: Build system Community: help wanted The contributors require help from other members of the community Type: new feature The issue requests / The PR implemements a new feature for RIOT

Comments

@jnohlgard
Copy link
Member

Description

Running the build in a Docker container with the BUILD_IN_DOCKER=1 make command should provide a persistent volume for storing the git-cache directory to reduce network download traffic.

Steps to reproduce the issue

in examples/ccn-lite-relay:

BUILD_IN_DOCKER=1 make

Expected results

git sources should be loaded from the git-cache

Actual results

git sources are re-downloaded whenever the bin directory is deleted

Versions

riot/riotbuild docker image b329486bb877 from 2017-12-01T14:36:49.484Z
up-to-date master branch from 2017-12-14

@jnohlgard jnohlgard added Feature Request Community: help wanted The contributors require help from other members of the community Area: build system Area: Build system labels Dec 14, 2017
@jnohlgard jnohlgard modified the milestone: Release 2018.01 Dec 14, 2017
@cladmi cladmi self-assigned this Jul 31, 2018
@cladmi
Copy link
Contributor

cladmi commented Jul 31, 2018

I will look at it.
I think it would require to move the default out of dist/tools/git/git-cache:

GIT_CACHE_DIR=${GIT_CACHE_DIR:-${HOME}/.gitcache}

To have access to the value in the build system.

@cladmi
Copy link
Contributor

cladmi commented Jul 31, 2018

I proposed something in #9651

The difference from this issue description is that you say "default settings" whereas I only did it if the ${GIT_CACHE_DIR} is defined and exists, I do not create one.

@miri64 miri64 added Type: new feature The issue requests / The PR implemements a new feature for RIOT and removed Feature Request labels Sep 30, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: build system Area: Build system Community: help wanted The contributors require help from other members of the community Type: new feature The issue requests / The PR implemements a new feature for RIOT
Projects
None yet
Development

No branches or pull requests

5 participants