{"thread":{"id":"27232","subject":"Strange cherry-picking issue with 1.7.5.msysgit.1.1.g05d0e","startedAt":"2011-05-01T08:28:44Z","lastAt":"2011-05-01T10:02:03Z","messageCount":3,"participants":["Sebastian Schuberth","Johannes Schindelin"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"166811","messageId":"ipj5k5$j04$1@dough.gmane.org","threadId":"27232","inReplyTo":null,"subject":"Strange cherry-picking issue with 1.7.5.msysgit.1.1.g05d0e","fromName":"Sebastian Schuberth","fromEmail":"sschuberth@gmail.com","sentAt":"2011-05-01T08:28:44Z","receivedAt":"2011-05-01T08:28:44Z","isPatch":false,"sender":{"key":"sschuberth@gmail.com","avatar":"https://avatars.githubusercontent.com/u/349154?v=4"},"body":"Hi,\n\nwhile preparing for the msysGit 1.7.5 release, I came across a strange issue with cherry-picking when using my own local build of 1.7.5.msysgit.1.1.g05d0e (built from the msysgit.git / 4msysgit.git devel branches; note that 4msysgit.git's devel is based on git.git's aka Junio's next).\n\nTo test the recent \"BindImage\" additions to the Windows installer, I did:\n\n# Create a topic branch based on msysgit.git devel:\nsebastian@PASSAU / (devel)\n$ git checkout -b ss/bindimage\nM       git\nSwitched to a new branch 'ss/bindimage'\n\n# Cherry-pick the \"BindImage\" commit from msysgit.git's mob:\nsebastian@PASSAU / (ss/bindimage)\n$ git cherry-pick 59467863efd336ba3a443a88ce140ab1e59a29f0\nerror: could not apply 5946786... Call BindImage on installer for all the exe and dll files\nhint: after resolving the conflicts, mark the corrected paths\nhint: with 'git add <paths>' or 'git rm <paths>'\nhint: and commit the result with 'git commit'\n\n# Inspect the working tree:\nsebastian@PASSAU / (ss/bindimage|CHERRY-PICKING)\n$ git status\n# On branch ss/bindimage\n# Changes to be committed:\n#\n#       modified:   share/WinGit/copy-files.sh\n#       modified:   share/WinGit/install.iss\n#\n# Changes not staged for commit:\n#   (use \"git add <file>...\" to update what will be committed)\n#   (use \"git checkout -- <file>...\" to discard changes in working directory)\n#\n#       modified:   git (new commits)\n#\n# Untracked files:\n#   (use \"git add <file>...\" to include in what will be committed)\n#\n#       share/InnoSetup/ISCmplr.dll~5946786... Call BindImage on installer for all the exe and dll files\n\nSo the cherry-pick fails although it should apply cleanly here. If I run the same command on the same working tree using msysgit 1.7.4, the cherry-pick applies fine. And where does this strangely named untracked file come from?\n\nNote the \"CHERRY-PICKING\" in the prompt which comes from js/cherry-pick-usability that we picked up from junio/next during our rebasing merge. At first I thought this issue might be related to that branch, but reverting 37f7a85 and d7e5c0c did not solve the issue for me.\n\nAny ideas what might be causing this?\n\nThanks for any insights.\n\n-- \nSebastian Schuberth\n"},{"id":"166814","messageId":"alpine.DEB.1.00.1105011138020.13761@bonsai2","threadId":"27232","inReplyTo":"ipj5k5$j04$1@dough.gmane.org","subject":"Re: Strange cherry-picking issue with 1.7.5.msysgit.1.1.g05d0e","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2011-05-01T09:45:01Z","receivedAt":"2011-05-01T09:45:01Z","isPatch":false,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi,\n\nOn Sun, 1 May 2011, Sebastian Schuberth wrote:\n\n> while preparing for the msysGit 1.7.5 release, I came across a strange \n> issue with cherry-picking when using my own local build of \n> 1.7.5.msysgit.1.1.g05d0e (built from the msysgit.git / 4msysgit.git \n> devel branches; note that 4msysgit.git's devel is based on git.git's aka \n> Junio's next).\n> \n> To test the recent \"BindImage\" additions to the Windows installer, I \n> did:\n> \n> # Create a topic branch based on msysgit.git devel:\n> sebastian@PASSAU / (devel)\n> $ git checkout -b ss/bindimage\n> M       git\n> Switched to a new branch 'ss/bindimage'\n> \n> # Cherry-pick the \"BindImage\" commit from msysgit.git's mob:\n> sebastian@PASSAU / (ss/bindimage)\n> $ git cherry-pick 59467863efd336ba3a443a88ce140ab1e59a29f0\n> error: could not apply 5946786... Call BindImage on installer for all the exe and dll files\n> hint: after resolving the conflicts, mark the corrected paths\n> hint: with 'git add <paths>' or 'git rm <paths>'\n> hint: and commit the result with 'git commit'\n> \n> # Inspect the working tree:\n> sebastian@PASSAU / (ss/bindimage|CHERRY-PICKING)\n> $ git status\n> # On branch ss/bindimage\n> # Changes to be committed:\n> #\n> #       modified:   share/WinGit/copy-files.sh\n> #       modified:   share/WinGit/install.iss\n> #\n> # Changes not staged for commit:\n> #   (use \"git add <file>...\" to update what will be committed)\n> #   (use \"git checkout -- <file>...\" to discard changes in working directory)\n> #\n> #       modified:   git (new commits)\n> #\n> # Untracked files:\n> #   (use \"git add <file>...\" to include in what will be committed)\n> #\n> #       share/InnoSetup/ISCmplr.dll~5946786... Call BindImage on installer for all the exe and dll files\n\nThis looks like some problem trying to check out that file from another \nbranch. Since you say that this problem does not occur when starting from \nthe 1.7.4 tag, and you updated ISCmplr.dll 6 days after that, I could \nimagine that something locks the .dll. After all, the current 'mob' is \nstill based off of 1.7.4 not off of the current 'devel'.\n\nThere should not have been a problem, though, as the .dll has not changed \nbetween the merge base and 5946786...\n\nFWIW on Linux, the same cherry-pick works just fine.\n\nCiao,\nDscho\n"},{"id":"166815","messageId":"BANLkTik1eYsz2eBkkUGM1XRVTOg8XEXnRQ@mail.gmail.com","threadId":"27232","inReplyTo":"alpine.DEB.1.00.1105011138020.13761@bonsai2","subject":"Re: [msysGit] Strange cherry-picking issue with 1.7.5.msysgit.1.1.g05d0e","fromName":"Sebastian Schuberth","fromEmail":"sschuberth@gmail.com","sentAt":"2011-05-01T10:02:03Z","receivedAt":"2011-05-01T10:02:03Z","isPatch":false,"sender":{"key":"sschuberth@gmail.com","avatar":"https://avatars.githubusercontent.com/u/349154?v=4"},"body":"On Sun, May 1, 2011 at 11:45, Johannes Schindelin\n<Johannes.Schindelin@gmx.de> wrote:\n\n>> # Untracked files:\n>> #   (use \"git add <file>...\" to include in what will be committed)\n>> #\n>> #       share/InnoSetup/ISCmplr.dll~5946786... Call BindImage on installer for all the exe and dll files\n>\n> This looks like some problem trying to check out that file from another\n> branch. Since you say that this problem does not occur when starting from\n> the 1.7.4 tag, and you updated ISCmplr.dll 6 days after that, I could\n> imagine that something locks the .dll. After all, the current 'mob' is\n> still based off of 1.7.4 not off of the current 'devel'.\n>\n> There should not have been a problem, though, as the .dll has not changed\n> between the merge base and 5946786...\n\nYou're right, I had Inno Setup running in the background, probably\nlocking the ISCmplr.dll. But even after closing Inno Setup and making\nsure ISCmplr.dll is not locked anymore, the above cherry-pick fails.\n\nHowever, if I pass --strategy=resolve, it works with 1.7.5. But as you\nalready mentioned, I'd not expect the cherry-pick to touch ISCmplr.dll\nin any case ... no what does that tell me, is it a regression in\n1.7.5, or a new \"feature\"?\n\n-- \nSebastian Schuberth\n"}]}