Page MenuHomeSoftware Heritage
Differential D807

Add .dockerignore
ClosedPublic
Actions

Authored by ardumont on Dec 13 2018, 9:44 AM.

Details

Reviewers
douardda
olasd
vlorentz
Group Reviewers
Reviewers
Commits
rDENV089665e5433b: Add .dockerignore
Summary

To avoid copying too much information during the COPY steps.

Related T1443

Diff Detail

Repository
rDENV Development environment
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

ardumont created this revision.Dec 13 2018, 9:44 AM
Harbormaster failed remote builds in B2993: Diff 2566!Dec 13 2018, 9:44 AM
vlorentz added a comment.Dec 13 2018, 9:48 AM

You may also add */.eggs and */*.egg-info

vlorentz accepted this revision.Dec 13 2018, 9:48 AM
This revision is now accepted and ready to land.Dec 13 2018, 9:48 AM
ardumont added a comment.Dec 13 2018, 9:48 AM

I've initially committed this directly. Pushed.
The build failed. So i reverted. Pushed.
Now i'm opening the diff, realizing that the swh-env has been red for a while now T.T

Still, that's better to show what i proposed ;)

ardumont added a comment.Dec 13 2018, 9:49 AM

You may also add */.eggs and */*.egg-info

Right!

ardumont updated this revision to Diff 2567.Dec 13 2018, 9:50 AM

Add other suggested patterns

Harbormaster failed remote builds in B2994: Diff 2567!Dec 13 2018, 9:50 AM
ardumont mentioned this in D805: docker-compose: Add the service dependencies.Dec 13 2018, 10:03 AM
Closed by commit rDENV089665e5433b: Add .dockerignore (authored by ardumont). · Explain WhyDec 13 2018, 10:58 AM
This revision was automatically updated to reflect the committed changes.
ardumont edited the summary of this revision. (Show Details)Dec 13 2018, 11:04 PM

Revision Contents
Changeset List

PathSize
10 lines

Diff 2569

View Options

.dockerignore

Loading...
About · Developer information · Legal