Bug 2188001 - git-review is broken with git 2.34+
Summary: git-review is broken with git 2.34+
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora EPEL
Classification: Fedora
Component: git-review
Version: epel8
Hardware: Unspecified
OS: Unspecified
unspecified
urgent
Target Milestone: ---
Assignee: Pete Zaitcev
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 2039224
Blocks: 2063790
TreeView+ depends on / blocked
 
Reported: 2023-04-19 12:33 UTC by Alfredo Moralejo
Modified: 2023-08-10 01:00 UTC (History)
6 users (show)

Fixed In Version: git-review-2.2.0-1.el8
Doc Type: If docs needed, set a value
Doc Text:
Clone Of: 2039224
Environment:
Last Closed: 2023-08-10 01:00:25 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Alfredo Moralejo 2023-04-19 12:33:42 UTC
After git update to git-2.39.0-1.el8.x86_64.rpm in CentOS Stream 8, we are hitting this issue in EPEL8. May we get it updated in epel8 too?

+++ This bug was initially created as a clone of Bug #2039224 +++

$ git review
Errors running git rebase -p -i remotes/gerrit/master
fatal: --preserve-merges was replaced by --rebase-merges
It is likely that your change has a merge conflict. You may resolve it
in the working tree now as described above and then run 'git review'
again, or if you do not want to resolve it yet (note that the change
can not merge until the conflict is resolved) you may run 'git rebase
--abort' then 'git review -R' to upload the change without rebasing.

See https://bugs.archlinux.org/task/73058 for an explanation. Needs updating to include https://opendev.org/opendev/git-review/commit/7182166ec00ad3645821435d72c5424b4629165f.

git-review-1.28.0-10.fc35.noarch
git-2.34.1-1.fc35.x86_64

--- Additional comment from Dmitry Tantsur on 2022-01-11 10:07:25 UTC ---

Note that git-review 2.2.0 actually includes the fix. 1.28.0 is from 2019, so an update is overdue.

--- Additional comment from Dmitry Tantsur on 2022-01-11 10:09:52 UTC ---

diff --git a/git_review/cmd.py b/git_review/cmd.py
index b248b46..c264eb7 100644
--- a/git_review/cmd.py
+++ b/git_review/cmd.py
@@ -852,9 +852,9 @@ def rebase_changes(branch, remote, interactive=True):
         sys.exit(1)
 
     if interactive:
-        cmd = "git rebase -p -i %s" % remote_branch
+        cmd = "git rebase --rebase-merges -i %s" % remote_branch
     else:
-        cmd = "git rebase -p %s" % remote_branch
+        cmd = "git rebase --rebase-merges %s" % remote_branch
 
     (status, output) = run_command_status(cmd, GIT_EDITOR='true')
     if status != 0:

--- Additional comment from Pete Zaitcev on 2022-01-11 19:35:15 UTC ---

Indeed, we're behind with packaging of 2.2.0. Worse, the whole shebang needs to be fast-tracked for the benefits of CentOS etc.

See this PR
https://src.fedoraproject.org/rpms/git-review/pull-request/3

--- Additional comment from Francisco Seruca Salgado on 2022-01-20 11:09:43 UTC ---

Hi Dmitry Tantsur,

I had the same issue, a temporary fix is to downgrade your git version.

--- Additional comment from Fedora Update System on 2022-02-11 16:12:54 UTC ---

FEDORA-2022-d807d9cd76 has been submitted as an update to Fedora 35. https://bodhi.fedoraproject.org/updates/FEDORA-2022-d807d9cd76

--- Additional comment from Fedora Update System on 2022-02-12 01:39:36 UTC ---

FEDORA-2022-d807d9cd76 has been pushed to the Fedora 35 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf upgrade --enablerepo=updates-testing --advisory=FEDORA-2022-d807d9cd76`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2022-d807d9cd76

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

--- Additional comment from Fedora Update System on 2022-02-19 01:31:42 UTC ---

FEDORA-2022-d807d9cd76 has been pushed to the Fedora 35 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 2 Fedora Update System 2023-08-01 14:52:24 UTC
FEDORA-EPEL-2023-a22af6f0ec has been submitted as an update to Fedora EPEL 8. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2023-a22af6f0ec

Comment 3 Fedora Update System 2023-08-02 01:39:03 UTC
FEDORA-EPEL-2023-a22af6f0ec has been pushed to the Fedora EPEL 8 testing repository.

You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2023-a22af6f0ec

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Comment 4 Fedora Update System 2023-08-10 01:00:25 UTC
FEDORA-EPEL-2023-a22af6f0ec has been pushed to the Fedora EPEL 8 stable repository.
If problem still persists, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.