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.

Sign up for GitHub

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

Jump to bottom

Git grep search error #30785

Closed
ymefg opened this issue Apr 30, 2024 · 6 comments · Fixed by #30843
Closed

Git grep search error #30785

ymefg opened this issue Apr 30, 2024 · 6 comments · Fixed by #30843
Assignees
@wxiaoguang
Labels
issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented type/bug

Comments

@ymefg
Copy link

ymefg commented Apr 30, 2024

Description

I'm encountering a failure when attempting to perform code searches in one specific repository of mine. This issue is exclusive to this particular repository; searching works fine in other repositories. Even when I try to create a new repository similar to the problematic one, I can't reproduce the issue.

image

Here's the additional information about the repository:

  • It has only one branch named "main".
  • All files are .md files.
  • There is a file named "readme.md".
  • The rest of the files have Chinese names, such as "通信协议.md".

Gitea Version

1.22.0+rc1

Can you reproduce the bug on the Gitea demo site?

No

Log Gist

No response

Screenshots

No response

Git Version

No response

Operating System

CentOS

How are you running Gitea?

docker compose

Database

SQLite
@ymefg ymefg added the type/bug label Apr 30, 2024
@wxiaoguang
Copy link
Contributor

Could you login to the Gitea server and find the "repo.git", and "cat config" to see its content?

@wxiaoguang
Copy link
Contributor

One more question: what's you git version?

@ymefg
Copy link
Author

ymefg commented Apr 30, 2024

Could you login to the Gitea server and find the "repo.git", and "cat config" to see its content?

Sorry, Gitea is deployed on the servers of the company I work for, and I'm off duty now. Also, due to the International Labor Day holiday, it'll be at least five days before I can send you the config and git version. I'll send the them once I'm back at work.

@wxiaoguang

This comment was marked as outdated.

Sign in to view
@wxiaoguang
Copy link
Contributor

Reproduced, will propose a fix.

@wxiaoguang wxiaoguang self-assigned this May 3, 2024
@wxiaoguang wxiaoguang added the issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented label May 3, 2024
@wxiaoguang wxiaoguang mentioned this issue May 3, 2024
Merged
@wxiaoguang
Copy link
Contributor

-> Fix grep search #30843

wxiaoguang added a commit that referenced this issue May 3, 2024
@wxiaoguang
Improve grep search (#30843)
0f3e717 
Reduce the context line number to 1, make "git grep" search respect the
include/exclude patter, and fix #30785
GiteaBot pushed a commit to GiteaBot/gitea that referenced this issue May 3, 2024
@wxiaoguang @GiteaBot
Improve grep search (go-gitea#30843)
a004f1a 
Reduce the context line number to 1, make "git grep" search respect the
include/exclude patter, and fix go-gitea#30785
@GiteaBot GiteaBot mentioned this issue May 3, 2024
Merged
wxiaoguang added a commit that referenced this issue May 3, 2024
@GiteaBot @wxiaoguang
Improve grep search (#30843) (#30850)
1f9a9fa 
Backport #30843 by wxiaoguang

Reduce the context line number to 1, make "git grep" search respect the
include/exclude patter, and fix #30785

Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>
@go-gitea go-gitea locked as resolved and limited conversation to collaborators Aug 1, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@wxiaoguang wxiaoguang

Labels
issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented type/bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Improve grep search wxiaoguang/gitea
2 participants
@wxiaoguang @ymefg