Skip to content
Snippets Groups Projects

Enable GIT_TRACE_PERFORMANCE through a config variable.

Merged Paco Guzman requested to merge 59-git-tracing into master
1 unresolved thread

The value of the variable must an absolute path needs to exist so we’re able to check if we can write in that file. Because in the case we cannot write we’ll throw a warning to the output of the users.

~/dev/gitlab/local/pacoguzman/gitlab-ce (master=)$ git push origin master
warning: could not open '/wadus' for tracing: Permission denied
Everything up-to-date

Closes #59 (closed)

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
  • Paco Guzman Added 1 commit:

    Added 1 commit:

    • 918401c0 - Enable GIT_TRACE_PERFORMANCE through a config variable git_trace_log_file
  • @pacoguzman I have said all I wanted to say here, I think you can pass this to the next reviewer.

  • Paco Guzman Added 1 commit:

    Added 1 commit:

    • 41c4ad8d - Enable GIT_TRACE_PERFORMANCE through a config variable git_trace_log_file
  • Reassigned to @rymai

  • Paco Guzman Milestone changed to %8.13

    Milestone changed to %8.13

  • Paco Guzman Added 11 commits:

    Added 11 commits:

    • 41c4ad8d...63c74eda - 10 commits from branch master
    • 9003ec61 - Enable GIT_TRACE_PERFORMANCE through a config variable git_trace_log_file
  • Author Maintainer

    I've just rebased against master

  • Paco Guzman Added 2 commits:

    Added 2 commits:

    • b71ca5da - 1 commit from branch master
    • 192e2bd3 - Enable GIT_TRACE/GIT_TRACE_PACKET/GIT_TRACE_PERFORMANCE by providing the git_tra…
  • Author Maintainer

    @rymai I've just changed the changelog and the commit message based on your suggestion! Thanks!

  • Mentioned in commit 0b4fd0af

  • username-removed-128633 Status changed to merged

    Status changed to merged

  • Mentioned in issue #59 (closed)

  • Mentioned in merge request !93 (closed)

  • Please register or sign in to reply
    Loading