Skip to content
Snippets Groups Projects

WIP: plugins/sources/git.py: Cope with rename returning error EEXIST

Closed Matthew Yates requested to merge matthew-yates/buildstream:rename-errno-fix into bst-1.2

Description

POSIX rename can set errno to ENOTEMPTY or EEXIST when the destination dir already has content (http://pubs.opengroup.org/onlinepubs/009695399/functions/rename.html), however the git plugin only copes with ENOTEMPTY (in 1.2.4)

In our project we have many elements referencing the same git repo, so this has causes some issues when building for the first time.

I've noticed this has already been fixed on master in utils.move_atomic - this is just a backport of that functionality to git plugin in 1.2.

Changes proposed in this merge request:

  • Check for EEXIST on os.rename failure in the git plugin.

This merge request, when approved, will close:


Edited by Laurence Urhegyi

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading