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.

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

Error when pushing with submodule.recurse=true "Gitea: Internal error" #25188

Open
nvx opened this issue Jun 10, 2023 · 2 comments
Open

Error when pushing with submodule.recurse=true "Gitea: Internal error" #25188

nvx opened this issue Jun 10, 2023 · 2 comments
Labels
issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented type/bug

Comments

@nvx
Copy link

nvx commented Jun 10, 2023

Description

Steps to reproduce (can reproduce on try.gitea.io, substitute the remote url with an empty repo you have push access to)

$ git config --global submodule.recurse true
$ git clone https://github.com/flipperdevices/flipperzero-firmware.git
<snip>
$ git remote add try git@try.gitea.io:nvx/flipperzero-firmware.git
$ git push try dev
fatal: remote 'try' not configured
fatal: process for submodule 'assets/protobuf' failed
fatal: the remote end hung up unexpectedly
Gitea: Failed to execute git command

Pushing to another GitHub remote works fine. Nothing out of the ordinary appears in the logs, no errors/etc.

Unsetting the submodule.recurse option allows the repo to be pushed as expected.

My server shows "Gitea: Internal error" instead of "Gitea: Failed to execute git command" though. Not sure if this is a Gitea version or Git version difference (I'm on gitea/gitea:1.19.3 docker image with whatever version of Git/etc is bundled in that image).

Gitea Version

1.19.3, but also on try which is 1.20.0+rc0-20-gfeb38da25

Can you reproduce the bug on the Gitea demo site?

Yes

Log Gist

No response

Screenshots

No response

Git Version

No response

Operating System

No response

How are you running Gitea?

My server is Docker, but reproducible on the demo site too

Database

PostgreSQL

@nvx nvx added the type/bug label Jun 10, 2023
@wxiaoguang wxiaoguang added the issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented label Jun 12, 2023
@wxiaoguang
Copy link
Contributor

wxiaoguang commented Jun 12, 2023

Added "confirmed" label.

To answer:

My server shows "Gitea: Internal error" instead of "Gitea: Failed to execute git command" though. Not sure if this is a Gitea version or Git version difference

Because Gitea 1.20 optimized the feedback error message. It's more meaningful than "Internal Error" now. We can see that the problem is caused by git command error .... well, it needs some time do debug/fix (ps: I am not working on it now, only share some ideas, feel free to pick it)

@nvx
Copy link
Author

nvx commented Jun 13, 2023

My server shows "Gitea: Internal error" instead of "Gitea: Failed to execute git command" though. Not sure if this is a Gitea version or Git version difference

Because Gitea 1.20 optimized the feedback error message. It's more meaningful than "Internal Error" now. We can see that the problem is caused by git command error

I expected as much. I must admit it was rather confusing seeing just "Internal Error" and nothing in the logs on 1.19, at least 1.20 gives a hint where the problem is (maybe it also shows more in the server logs, I haven't tried running 1.20 yet locally).

Cheers for the confirmation. At least I figured out a workaround so it's not really a showstopper, but hopefully saves some confusion for the next guy at the very least. :D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented type/bug
Projects
None yet
Development

No branches or pull requests

2 participants