Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

gitserver: Fix CommitLog on some git versions#62828

Merged
eseliger merged 1 commit intomainfrom
es/05-21-gitserverfixcommitlogonsomegitversions
May 21, 2024
Merged

gitserver: Fix CommitLog on some git versions#62828
eseliger merged 1 commit intomainfrom
es/05-21-gitserverfixcommitlogonsomegitversions

Conversation

@eseliger
Copy link
Member

I don't have a clue yet why this happens, but I'll let the two outputs below speak for themselves:

On my mac:

➜  sourcegraph git:(main) git log --format='format:%x1e%H%x00%aN%x00%aE%x00%at%x00%cN%x00%cE%x00%ct%x00%B%x00%P%x00' --after=2024-02-21T18:26:09Z --topo-order HEAD --name-only
➜  sourcegraph git:(main) git version
git version 2.45.1

In the latest gitserver docker image from main:

/data/repos $ git log --format='format:%x1e%H%x00%aN%x00%aE%x00%at%x00%cN%x00%cE%x00%ct%x00%B%x00%P%x00' --after=2024-02-2
1T18:26:09Z --topo-order HEAD --name-only
fatal: option '--name-only' must come before non-option arguments
/data/repos $ git version
git version 2.45.1

Test plan:

Verified the generated command runs alright inside our docker image.

I don't have a clue yet why this happens, but I'll let the two outputs below speak for themselves:

On my mac:

```
➜  sourcegraph git:(main) git log --format='format:%x1e%H%x00%aN%x00%aE%x00%at%x00%cN%x00%cE%x00%ct%x00%B%x00%P%x00' --after=2024-02-21T18:26:09Z --topo-order HEAD --name-only
➜  sourcegraph git:(main) git version
git version 2.45.1
```

In the latest gitserver docker image from main:

```
/data/repos $ git log --format='format:%x1e%H%x00%aN%x00%aE%x00%at%x00%cN%x00%cE%x00%ct%x00%B%x00%P%x00' --after=2024-02-2
1T18:26:09Z --topo-order HEAD --name-only
fatal: option '--name-only' must come before non-option arguments
/data/repos $ git version
git version 2.45.1
```

Test plan:

Verified the generated command runs alright inside our docker image.
@cla-bot cla-bot bot added the cla-signed label May 21, 2024
Copy link
Member Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @eseliger and the rest of your teammates on Graphite Graphite

@github-actions github-actions bot added team/product-platform team/source Tickets under the purview of Source - the one Source to graph it all labels May 21, 2024
@eseliger eseliger marked this pull request as ready for review May 21, 2024 18:54
@eseliger eseliger requested a review from a team May 21, 2024 18:54
@eseliger eseliger merged commit 0af3ed0 into main May 21, 2024
Copy link
Member Author

Merge activity

@eseliger eseliger deleted the es/05-21-gitserverfixcommitlogonsomegitversions branch May 21, 2024 19:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla-signed team/product-platform team/source Tickets under the purview of Source - the one Source to graph it all

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants