git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Wtrlt: Antw: [EXT] Re: Strange "git stash pop" conflict (one chunk out of many)

From
UWUlrich Windl <ulrich.windl@rz.uni-regensburg.de>
Date
Dec 19, 2022, 07:31 UTC
Message-ID
<63A01350020000A100050851@gwsmtp.uni-regensburg.de>
In-Reply-To
<63A012F0020000A10005084B@gwsmtp.uni-regensburg.de>
Sorry, I forgot to reply to the list, too.
>>> Phillip Wood <phillip.wood123@gmail.com> schrieb am 17.12.2022 um 11:44 in
Nachricht <6c8e8271-432f-38e3-e70e-1445f874afc6@dunelm.org.uk>:
Show 18 quoted lines
> Hi Ulrich
> 
> On 15/12/2022 11:38, Ulrich Windl wrote:
>> Hi!
>> 
>> This is for a somewhat older git-2.26.2:
>> I had added interactively some changes  using edit of a few junks (I tried 
> to structure my big hacking into logical junks when committing).
>> To test whether the partial commit would be consistent, I did a "git stash 
> -k" before committing, and after committing I did a "git stash pop" to 
> continue hacking.
>> 
>> Unfortunately I had a "Merge conflict". Looking at it, it is rather 
> "interesting", however (meaning: I don't understand it).
>> Here is how the conflict looks (to me both variants seem identical):
> 
> Yes, it does look "interesting". Did you make any changes between 
> running "git stash -k" and "git stash pop"? I did wonder if there had 

Of course I did: I did an --interactive add before (required to separate to edits affecting the same lines), then I stashed the rest. Commited that. The I switched to another branch doing a stash pop there to interactively add another fix like above. The switched back to the previous branch, did a rebase and then popped the remaining stash to continue.

Yes, I know it's a bad procedure, but frequently when adding a neew feature you find an fix bugs that aren't really related to that feature, so you want the unrelated fixes to go to another branch. Problably I don't use the full power of git yet (meaning: maybe there's an easier workflow).
Show 5 quoted lines
> been some whitespace changes where spaces were replaced with tabs or 
> vice-versa ("git stash" uses "git apply" to create the stash so if you 
> have apply.whitespace set to "fix" the stashed changes will not 
> necessarily match those in the working copy) but diffing the two sides 
> of the conflict does not show any changes.
I'm not aware that I did any white-space fixes in the hunk reported.

Regards, Ulrich

Show 29 quoted lines
> 
> Best Wishes
> 
> Phillip
> 
>>          # pre-allocate translations and accesskeys
>> <<<<<<< Updated upstream
>>          foreach my $attr (LD_SEARCH_ATTR) {
>>              $attr{$attr} = [translate_attr($attr), undef];
>>              $attr{$attr}->[1] = add_access_key($aks, 0, $attr{$attr}->[0]);
>>          }
>>          foreach my $attr (LD_SEARCH_ATTR) {
>> =======
>>          foreach my $attr (LD_SEARCH_ATTR) {
>>              $attr{$attr} = [translate_attr($attr), undef];
>>              $attr{$attr}->[1] = add_access_key($aks, 0, $attr{$attr}->[0]);
>>          }
>>          foreach my $attr (LD_SEARCH_ATTR) {
>>>>>>>>> Stashed changes
>>              @n = (P_P_SRCH_ATTR . $attr, @{$attr{$attr}});
>> 
>> (the other conflict junks look reasonable)
>> 
>> Regards,
>> Ulrich
>> 
>> 
>> 
>> 
Previous: Phillip Wood
Message 6 of 6 in “Strange "git stash pop" conflict (one chunk out of many)”
  1. Ulrich WindlDec 15, 2022
  2. Bagas SanjayaDec 17, 2022
  3. Phillip WoodDec 17, 2022
  4. Antw: [EXT] Re: Strange "git stash pop" conflict (one chunk out of many)Ulrich Windl, Dec 19, 2022
  5. Phillip WoodDec 17, 2022
  6. Wtrlt: Antw: [EXT] Re: Strange "git stash pop" conflict (one chunk out of many)Ulrich Windl, Dec 19, 2022

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.