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

Fix vagrant rsync #464

Merged
merged 1 commit into from
Dec 4, 2014
Merged

Conversation

iNecas
Copy link
Contributor

@iNecas iNecas commented Nov 24, 2014

The git repository contains some blind symlink that cause issues when syncing
with default vagrant rsync setting.

E.g. Godeps/_workspace/src/github.com/docker/docker/pkg/symlink/testdata/fs/g
is pointing to non-existing path, causing vagrant to fail on:

rsync error: some files/attrs were not transferred

The git repository contains some blind symlink that cause issues when syncing
with default vagrant rsync setting.

E.g.
Godeps/_workspace/src/github.com/docker/docker/pkg/symlink/testdata/fs/g
is pointing to non-existing path, causing vagrant to fail on:

    rsync error: some files/attrs were not transferred
@smarterclayton
Copy link
Contributor

LGTM [merge]. Sorry for the delay, missed this in the queue. Thanks!

@openshift-bot
Copy link
Contributor

Origin Merge Results: SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_requests_openshift3/491/) (Image: devenv-fedora_458)

@openshift-bot
Copy link
Contributor

[Test]ing while waiting on the merge queue

@openshift-bot
Copy link
Contributor

@mfojtik
Copy link
Contributor

mfojtik commented Dec 4, 2014

[merge]

@openshift-bot
Copy link
Contributor

Evaluated for origin up to d17ddd1

openshift-bot pushed a commit that referenced this pull request Dec 4, 2014
@openshift-bot openshift-bot merged commit 73eee0e into openshift:master Dec 4, 2014
sjenning pushed a commit to sjenning/origin that referenced this pull request Nov 16, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants