05:56 <PaulFertser> borneoa_: the modification just removed this.mergeable when deciding whether to invoke search for conflicts:NNNN or not.

06:51 <PaulFertser> borneoa_: so basically what you see in the list on https://review.openocd.org/c/openocd/+/6508/ is the same list you get with https://review.openocd.org/q/conflicts:6508+is:open

06:55 <borneoa_> PaulFertser: looks like the search for conflicts identifies changes just by filename, not by changed line. A bug in this version of gerrit? I will check the source code to get any hint. Do you remember which version of gerrit was installed before?

06:57 <PaulFertser> borneoa_: 2.13.6

06:57 <PaulFertser> borneoa_: please hold on, I'm checking myself

06:58 <PaulFertser> borneoa_: the source code mentions SubmitDryRun so it can't be just the filenames.

07:01 <borneoa_> PaulFertser: if I search open and cortex_m.c I get 20 hits, so the 13 conflicting with 6508 are not for filename only

07:04 <PaulFertser> borneoa_: probably it does something like git fetch https://review.openocd.org/openocd refs/changes/08/6508/1 && git checkout FETCH_HEAD ; git fetch https://review.openocd.org/openocd refs/changes/19/6319/5 && git cherry-pick FETCH_HEAD and that fails, hence the conflict.

07:06 <borneoa_> PaulFertser: could be, and probably the change 6319 was not mergeable anymore by itself

07:10 <borneoa_> PaulFertser: all the changes in conflict are with "merge conflict" status

07:28 <PaulFertser> borneoa_: but it doesn't really mean you do not want to see them, and they're still somehow related apparently, at least changing the same file.

07:28 <PaulFertser> It's unclear how this can be improved.

07:33 <Haohmaru> with ketchup ;P~

07:33 <borneoa_> PaulFertser: I would left it as is and just think it over.

09:56 <tarekb> Hi Paul, All, I hope you are doing well

09:56 <tarekb> PaulFertser: I have encountered some handshake failres with gerrit redirection from zylin server to the new one

09:57 <tarekb> s/failres/failures/

09:58 <tarekb> I hope this is only my case (everything is OK after changing the server in my config to the new one)

10:04 <PaulFertser> tarekb: hey :) ssh config?

10:12 <tarekb> I have changed both my git remote url and added an entry in ~/.netrc

10:13 <tarekb> to use directly the review.openocd.org server

10:15 <tarekb> theoritacally passing by the old server should be OK and redirected without issues, but in my setup, I'm facing hanshake failures 4/10 times

10:16 <PaulFertser> tarekb: handshake with what protocol/utility?

10:18 <tarekb> ah, I'm using https://review.openocd.org/...

10:19 <tarekb> no ssh

10:20 <PaulFertser> The old name is currently going through socat on the old server but then you get redirected right away by Location: header.

15:35 <borneoa_> PaulFertser: https://review.openocd.org/Documentation/config-gerrit.html#change.mergeabilityComputationBehavior

15:35 <borneoa_> PaulFertser: today it's set to NEVER.

15:36 <borneoa_> PaulFertser: do you remember how was it set before? Any idea how much computational expensive are the other choices?

15:41 <karlp> given the sizes of the places gerrit comes from, probably not siginificant for us?

15:41 <karlp> small codebase too, relatively speaking

16:15 <PaulFertser> borneoa_: let's just flip it and see.

16:17 <PaulFertser> So API_REF_UPDATED_AND_CHANGE_REINDEX ?

17:45 <Fleck> hi tarekb :)

17:45 <Fleck> thanks for fixing stm32 configs ;)

17:57 <borneoa_> PaulFertser: probably yes. Looks like the more complete option

17:58 <PaulFertser> borneoa_: can do it right now then

18:01 <PaulFertser> borneoa_: restarted with it enabled

18:08 <borneoa_> PaulFertser: thanks. Strange, is:mergeable finds only 2 changes. Maybe it's still running in background to find the others

18:10 <borneoa_> PaulFertser: apparently it considers mergeable only the changes that are rebased on current master

18:10 <PaulFertser> borneoa_: or probably only changes that were pushed after the switch?

18:11 <PaulFertser> No, unlikely

18:11 <nohit> hi, im trying to connect to a gdb server from WSL but i get connection timeout, any ideas what might be wrong ? https://usercontent.irccloud-cdn.com/file/GNeZQXaq/ocd.png

18:12 <PaulFertser> nohit: OpenOCD binds to loopback interface only by default.

18:12 <PaulFertser> nohit: and WSL2 is a VM

18:12 <PaulFertser> Is that a single WSL instance?

18:13 <nohit> yes

18:13 <PaulFertser> nohit: ah, missing space between ext and :

18:13 <PaulFertser> You can use "tar" as short for "target" btw

18:21 <nohit> i switched to WSL1 and now it works. thanks PaulFertser

18:21 <PaulFertser> nohit: I do not think it's the real reason.

18:21 <PaulFertser> nohit: I clearly see wrong command in your first screenshot.

18:21 <PaulFertser> You should be using wsl2

18:32 <nohit> but it is https://usercontent.irccloud-cdn.com/file/dqOcWCKS/ocd2.png

18:34 <PaulFertser> nohit: you have _no space_ between ext and :3333

18:34 <PaulFertser> But it still works, ok.

18:35 <nohit> i fixed it and then removed it to prove a point

18:35 <nohit> that space

18:36 <PaulFertser> nohit: thank you, point taken

18:51 <PaulFertser> Tarek introduced new warning :) https://build.openocd.org/job/openocd-clang/scan-build_20Report/report-42cdcc.html#EndPath

22:51 <tarekb> Hi Fleck, if the change is OK, please leave a note in gerrit

22:51 <tarekb> I was not able to add you as reviewer since IDK your login (if any)