)]}'
{
  "commit": "490469846dcef89fe53668bdbba73591c64bed61",
  "tree": "2d31c7bff4cbab5e4913946cf621fdda27aaecda",
  "parents": [
    "ffca19ab8e9cf5f57a6ea81a108d9113de9e8336"
  ],
  "author": {
    "name": "Pedro Alves",
    "email": "pedro@palves.net",
    "time": "Thu Nov 27 18:06:12 2025 +0000"
  },
  "committer": {
    "name": "Pedro Alves",
    "email": "pedro@palves.net",
    "time": "Fri Jul 10 19:03:27 2026 +0100"
  },
  "message": "gdb.base/execl-update-breakpoints.exp: Don\u0027t execl forever\n\nexecl-update-breakpoints.c is compiled as both\nexecl-update-breakpoints1 and execl-update-breakpoints2.\n\nexecl-update-breakpoints1 checks to see if the last character in its\nfilename is \u00271\u0027, and if so, replace it with \u00272\u0027 and execl the result.\n\nOn Windows, this fails to take into account that the filename may have\nan \".exe\" extension.  The result is that the execl-update-breakpoints1\nexecutable execs itself forever.\n\nThis patch fixes it by taking the .exe extension into account.\n\n(Note that neither native Windows nor Cygwin support following execs,\nand maybe the testcase will eventually be skipped there, but even\nthen, if someone enables follow-exec testing there, we\u0027d want it to\nhandle the .exe extension, so I think it\u0027s better to adjust the\ntestcase than not.)\n\nNow, on e.g. GNU/Linux, if you run execl-update-breakpoints1 manually,\nyou\u0027ll see that it also execs forever, but this time, what happens is\nthat execl-update-breakpoints1 execs execl-update-breakpoints2, and\nthen it\u0027s execl-update-breakpoints2 that execs itself forever.  There\nis no reason for execl-update-breakpoints2 to exec.  It\u0027s just that\nthe testcase is assuming that the process is killed when the testcase\nis over.  It\u0027s just cleaner if execl-update-breakpoints1 execs\nexecl-update-breakpoints2, and then execl-update-breakpoints2 just\ncleanly exits.  The patch does that too.\n\nAnd then, the test program should really be erroring out if neither\n\u00271\u0027 nor \u00272\u0027 is found at the tail of the executable.  Done too.\n\nApproved-By: Simon Marchi \u003csimon.marchi@efficios.com\u003e\nChange-Id: Id88830cd8dff2c63dd69b827d4ba79743aa9ebca\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "adaf2e3bbbd0b7c776090347dd2a4b9740e8d199",
      "old_mode": 33188,
      "old_path": "gdb/testsuite/gdb.base/execl-update-breakpoints.c",
      "new_id": "1682c9ad40a95c2c93a76feafe0d725581ac4f1c",
      "new_mode": 33188,
      "new_path": "gdb/testsuite/gdb.base/execl-update-breakpoints.c"
    }
  ]
}
