Uploaded image for project: 'Qt Quality Assurance Infrastructure'
  1. Qt Quality Assurance Infrastructure
  2. QTQAINFRA-5010

Inexplicable conflicts from cherry-pick bot compared to local cherry-pick -x

    XMLWordPrintable

Details

    • Bug
    • Resolution: Unresolved
    • P2: Important
    • None
    • 5.15, 6.2, 6.3
    • Cherry-pick bot
    • None

    Description

      The following cherry-picks had conflicts when cherry-pick bot created them:

      but a local git cherry-pick -x 8df415d5bcf23462bedb4cb7601b909851ee15dd succeeded without conflicts, and it seems it didn't even perform inexact rename detection:

      ~/Qt/qt5/qtmultimedia[tqtc/lts-6.2]$ git cherry-pick -x  8df415d5bcf23462bedb4cb7601b909851ee15dd
      Auto-merging src/multimedia/platform/pulseaudio/qpulseaudiosource.cpp
      [detached HEAD ef1e13c72] QPulseAudioSource: fix UB (memcpy() called with nullptr dest) in read()
       Date: Thu May 19 12:02:04 2022 +0200
       1 file changed, 6 insertions(+), 2 deletions(-)
      

      vs.

      ~/Qt/qt5/qtmultimedia[tqtc/lts-5.15]$ git cherry-pick -x  8df415d5bcf23462bedb4cb7601b909851ee15dd
      Performing inexact rename detection: 100% (2060345/2060345), done.
      Auto-merging src/plugins/pulseaudio/qaudioinput_pulse.cpp
      [detached HEAD 84be719a2] QPulseAudioSource: fix UB (memcpy() called with nullptr dest) in read()
       Date: Thu May 19 12:02:04 2022 +0200
       1 file changed, 6 insertions(+), 2 deletions(-)
      

      for https://codereview.qt-project.org/c/qt/tqtc-qtmultimedia/+/412278/1

      Git v2.25.1 on Ubuntu 20.04, no rerere, but merge.renamelimit=40000.

      Q: Why doesn't the bot manage to create these picks w/o conflicts if they just go though locally?

      Attachments

        Issue Links

          No reviews matched the request. Check your Options in the drop-down menu of this sections header.

          Activity

            People

              jujokini Jukka Jokiniva
              mmutz Marc Mutz
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:

                Gerrit Reviews

                  There are no open Gerrit changes