)]}'
{
  "commit": "e5efa9eff99da897f0bbd69455d8f7856985b2db",
  "tree": "00ad64ff28e8c7a696a38dfc3c9327a0ce08c5cd",
  "parents": [
    "2026474385dc23e9896577db08d963bc3a96f256"
  ],
  "author": {
    "name": "Thomas Schwinge",
    "email": "tschwinge@baylibre.com",
    "time": "Tue Jul 28 20:46:16 2026 +0200"
  },
  "committer": {
    "name": "Thomas Schwinge",
    "email": "tschwinge@baylibre.com",
    "time": "Fri Aug 14 12:38:51 2026 +0200"
  },
  "message": "libgomp: Revise libgomp/Fortran support conditionals\n\nSubversion r152931 (Git commit 0046b1a727f73f0848733e28219069426b2dc218)\n\"Fix libgomp without --enable-languages\u003dfortran\", and its follow-up\nSubversion r156232 (Git commit d0709b6aeec6b00e71011502fa311751fc138147)\n\"configure.ac: Test for executability of GFORTRAN\",\nSubversion r156264 (Git commit 2122aa973ed2c829caee5b010a60ad01922a3650)\n\"configure.ac: Test for executability of _the first word_ of GFORTRAN\"\nfixed the libgomp target library build in case that\n\u0027--enable-languages\u003dfortran\u0027 was not in effect: in that case, disable\nlibgomp/Fortran support.\n\nThese changes certainly do the right thing in GCC configurations where\n\u0027$GFORTRAN\u0027 would be expected to be \"just compiled\" (built with the GCC\ntop-level build system).  However, in case that it\u0027s \"pre-installed\" (provided\nexternally, as determined by the GCC top-level build system), these changes\ndidn\u0027t work, and instead kept libgomp/Fortran support disabled, instead of\nenabling it.  \"Pre-installed\" \u0027GFORTRAN\u003d\"x86_64-w64-mingw32-gfortran [...]\"\u0027 is\nnot accepted by the current \u0027test -x [...]\u0027 check, for instance, as discussed\nin \"libgomp: Fix gfortran detection when program is found via PATH\",\n\u003chttps://inbox.sourceware.org/20260529021023.9764-1-peter0x44@disroot.org\u003e\n\u003chttps://forge.sourceware.org/gcc/gcc/pulls/172\u003e,\n\u003chttps://inbox.sourceware.org/bmm.hj9f44xa6a.gcc.gcc-TEST.Peter0x44.172.1.0@forge-stage.sourceware.org\u003e.\n\nThat proposed \"libgomp: Fix gfortran detection when program is found via PATH\"\npatch has the issue that it picks up the \"gfortran [...] found via PATH\" also\nin the case when GCC is configured without \u0027--enable-languages\u003dfortran\u0027 in\neffect, and it then still builds and installs the libgomp/Fortran support files\nin GCC configurations without Fortran support enabled.\n\nInstead of \"libgomp: Fix gfortran detection when program is found via PATH\" as\nproposed, we may revise libgomp/Fortran support conditionals generally, and\nhandle that in a conceptually simpler way: revert the three aforementioned\ncommits (in other words: simply again assign \u0027FC\u003d\"$GFORTRAN\"\u0027 (like\n\u0027libgfortran/configure.ac\u0027 also is doing), and conditionalize \u0027USE_FORTRAN\u0027 on\nwhether \u0027--enable-languages\u003dfortran\u0027 is in effect.  In other words: in case\nthat \u0027--enable-languages\u003dfortran\u0027 is in effect, assume that the\n\u0027GFORTRAN\u003d[...]\u0027 passed to the libgomp build is functional -- no matter where\nit\u0027s coming from: \"just compiled\" or \"pre-installed\".\n\n\tPR libgomp/41418\n\tlibgomp/\n\t* configure.ac (FC): Revert ealier changes; just set to \u0027\"$GFORTRAN\"\u0027.\n\t(USE_FORTRAN): Conditionalize on \u0027--enable-languages\u003dfortran\u0027 being in\n\teffect.\n\t* configure: Regenerate.\n\nReported-by: Peter Damianov \u003cpeter0x44@disroot.org\u003e\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "2f395a883f2ad5753e41f398ca96c1c6989e51f0",
      "old_mode": 33261,
      "old_path": "libgomp/configure",
      "new_id": "6ff3899df153dc788e9ec5ada4fdd883addb4e7c",
      "new_mode": 33261,
      "new_path": "libgomp/configure"
    },
    {
      "type": "modify",
      "old_id": "057debe921415fcb59227278659d87d834e695c3",
      "old_mode": 33188,
      "old_path": "libgomp/configure.ac",
      "new_id": "9ebac1fc3365919b1cbd71d28240e8fa16716a8e",
      "new_mode": 33188,
      "new_path": "libgomp/configure.ac"
    }
  ]
}