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

Atomically move git located cookbook to cache #1598

Merged
merged 1 commit into from
Aug 30, 2016

Conversation

kamaradclimber
Copy link
Contributor

This will decrease race conditions when running multiple berks
installation.

Fix #1597

Change-Id: Ie7c19f1e74fb4ed21f401b51b83ffe8297e4ce1f

This will decrease race conditions when running multiple berks
installation.

Fix berkshelf#1597

Change-Id: Ie7c19f1e74fb4ed21f401b51b83ffe8297e4ce1f
@thommay thommay merged commit 04978e6 into berkshelf:master Aug 30, 2016
@thommay
Copy link
Contributor

thommay commented Aug 30, 2016

Disappointingly, this actually won't change the situation, due to https://github.com/berkshelf/berkshelf/blob/master/lib/berkshelf/core_ext/file_utils.rb ; but I'm just about to ship a PR for that.

@kamaradclimber
Copy link
Contributor Author

thanks!

On Tue, Aug 30, 2016 at 4:20 PM, Thom May notifications@github.com wrote:

Disappointingly, this actually won't change the situation, due to
https://github.com/berkshelf/berkshelf/blob/master/lib/
berkshelf/core_ext/file_utils.rb ; but I'm just about to ship a PR for
that.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#1598 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAeu8bKblXj0KypFn18nD5QqzuqXh5AUks5qlDwxgaJpZM4JvcJ4
.

Grégoire

@berkshelf berkshelf locked and limited conversation to collaborators Jun 16, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants