Skip to content
Snippets Groups Projects

Setting the working directory before git operations

Merged gitlab-qa-bot requested to merge github/fork/crazyscience/patch-3 into master

Created by: crazyscience

Set the initial working directory. An issue can arise if the user is logged in as root and the current working directory is /root/. The console will return an error:

fatal: Could not change back to '/root': Permission denied

Changing the working directory prior to the clone operation seems to fix this.

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
  • Created by: coveralls

    Coverage Status

    Coverage decreased (-12.13%) when pulling 9e4b7b03b5485987fd98ec82a18cb23383d5310a on crazyscience:patch-3 into 78ff99a7 on gitlabhq:master.

    By Administrator on 2013-05-07T02:35:32 (imported from GitLab project)

    By Administrator on 2013-05-07T02:35:32 (imported from GitLab)

  • Created by: senny

    Can you put the change from #3861 into this PR? No need to keep them in separate pull requests.

    By Administrator on 2013-05-07T07:58:35 (imported from GitLab project)

    By Administrator on 2013-05-07T07:58:35 (imported from GitLab)

  • Created by: senny

    Same with #3859. can you move those changes to this PR?

    By Administrator on 2013-05-07T07:59:43 (imported from GitLab project)

    By Administrator on 2013-05-07T07:59:43 (imported from GitLab)

  • Created by: crazyscience

    Alright. Sorry about that. I'm a rookie here in the open source world. You live and you learn.

    By Administrator on 2013-05-07T12:17:47 (imported from GitLab project)

    By Administrator on 2013-05-07T12:17:47 (imported from GitLab)

  • Created by: senny

    @crazyscience no worries and thanks for your work :yellow_heart:

    can you squash all the commits into a single one? This will help us to keep the graph more focused.

    You don't need to open a new PR after you squashed, simply issue a force push on the branch and GitHub will update the PR.

    By Administrator on 2013-05-07T12:22:55 (imported from GitLab project)

    By Administrator on 2013-05-07T12:22:55 (imported from GitLab)

  • Created by: coveralls

    Coverage Status

    Coverage decreased (-0.03%) when pulling 1a6165192e5eb61157744a630277473af60b1b21 on crazyscience:patch-3 into 78ff99a7 on gitlabhq:master.

    By Administrator on 2013-05-07T12:40:23 (imported from GitLab project)

    By Administrator on 2013-05-07T12:40:23 (imported from GitLab)

  • Created by: coveralls

    Coverage Status

    Coverage decreased (-12.04%) when pulling 1a6165192e5eb61157744a630277473af60b1b21 on crazyscience:patch-3 into 78ff99a7 on gitlabhq:master.

    By Administrator on 2013-05-07T12:49:12 (imported from GitLab project)

    By Administrator on 2013-05-07T12:49:12 (imported from GitLab)

  • Created by: crazyscience

    ok. taken care of.

    By Administrator on 2013-05-08T01:30:48 (imported from GitLab project)

    By Administrator on 2013-05-08T01:30:48 (imported from GitLab)

  • Created by: coveralls

    Coverage Status

    Coverage decreased (-0.03%) when pulling ac88d512 on crazyscience:patch-3 into 2e9599b7 on gitlabhq:master.

    By Administrator on 2013-05-08T02:03:51 (imported from GitLab project)

    By Administrator on 2013-05-08T02:03:51 (imported from GitLab)

  • Created by: senny

    good job.

    @randx can you take a look?

    By Administrator on 2013-05-08T05:22:43 (imported from GitLab project)

    By Administrator on 2013-05-08T05:22:43 (imported from GitLab)

Please register or sign in to reply
Loading