<table border="1" cellspacing="0" cellpadding="8">
    <tr>
        <th>Issue</th>
        <td>
            <a href=https://github.com/llvm/llvm-project/issues/59088>59088</a>
        </td>
    </tr>

    <tr>
        <th>Summary</th>
        <td>
            Python minimal required version setting lost
        </td>
    </tr>

    <tr>
      <th>Labels</th>
      <td>
            new issue
      </td>
    </tr>

    <tr>
      <th>Assignees</th>
      <td>
      </td>
    </tr>

    <tr>
      <th>Reporter</th>
      <td>
          Arfrever
      </td>
    </tr>
</table>

<pre>
    Commit 39970764af39415ad62136ff75b0f89577c18037 added setting of `LLVM_MINIMUM_PYTHON_VERSION` variable in `llvm/cmake/modules/HandleLLVMOptions.cmake`, and this variable was used in 4 other files, which have `include(HandleLLVMOptions)`.

Commit 45c056b1fb5a9bc797e935b160aa07bf53764a3a moved setting of `LLVM_MINIMUM_PYTHON_VERSION` variable from `llvm/cmake/modules/HandleLLVMOptions.cmake` to `llvm/CMakeLists.txt`.
This means that `LLVM_MINIMUM_PYTHON_VERSION` variable is no longer defined in:
https://github.com/llvm/llvm-project/blob/5d1ae6346b15539421d055d3dfc94fabe0d7c558/clang/CMakeLists.txt#L86-L87
https://github.com/llvm/llvm-project/blob/5d1ae6346b15539421d055d3dfc94fabe0d7c558/lld/CMakeLists.txt#L53-L54
https://github.com/llvm/llvm-project/blob/5d1ae6346b15539421d055d3dfc94fabe0d7c558/mlir/cmake/modules/MLIRDetectPythonEnv.cmake#L17-L18
https://github.com/llvm/llvm-project/blob/5d1ae6346b15539421d055d3dfc94fabe0d7c558/mlir/cmake/modules/MLIRDetectPythonEnv.cmake#L28-L29

CMake output **before** second commit may contain e.g.:
```
-- Found Python3: /usr/bin/python3.10 (found suitable version "3.10.8", minimum required is "3.0") found components: Interpreter
```
CMake output **after** second commit may contain e.g.:
```
-- Found Python3: /usr/bin/python3.10 (found version "3.10.8") found components: Interpreter
```

If setting of this variable cannot be centralized, then please set this variable separately in these files...
And maybe rename it (e.g. `CLANG_MINIMUM_PYTHON_VERSION`, `LLD_MINIMUM_PYTHON_VERSION`, `MLIR_MINIMUM_PYTHON_VERSION`).

I would also suggest to require at least `3.6` unconditionally.

@christetreault-llvm
@DimitryAndric
</pre>
<img width="1px" height="1px" alt="" src="http://email.email.llvm.org/o/eJzNVV1v2zYU_TX2C2GB-qA-HvSQJs1qwE6Kriuwp-BKJG1uFOmRlDPv1_dScppkTTc0D90AQ6J8Ly8PD8897Cw_tZd2GFQgedNUtCoLkHlTpAx4maV5KWXFOirrhlVVn9Y0rwhwLjjxIgRldsRKsijpZvNpe7dd36y3v2zv3v_68d3tzd2ntx9-Xt_eYJQcwSnotCDKxGytj8Miu-4H-F3ge7B81MLj6B0YrkUsdnsIyhqfzDklXWSXBIMk7JV_LHcPnowe0WDdgtiwF45INdW6JPd71e_JHo4irqlMr0eOy9VfLbLIGkxIFvRqQS_m55mTgvWUlV0qOwZN11dNJZqcdWlJAWjVSZZHwnIggz2-mhPp7PBKVkiwT2ZebvHfjfLBJ-HP8Lilj5GzQYDxSB-E7zovT4wl2podMsuFVGYie5GfedqHcPDxK7vG306F_dglvY1ozqDia3Vw9jfRB_zstO3wxXgKoswL5JYx1FuWcsoYz7nsm0JCJyivesbqyIcGs_t6d1m-qcvVpq5-GBCt-YswWL7asOKHwRi0ci_KZLtZf7gSAeu-P4W9NW_N8SwUBJlWq01a_89BZvVqg734tA8j28SO4TCibjOEfNEJaZ2Yx9hxvUVX6Od2HeCEQxMA7UAku-SLTKOBzL_pc7Ui13bEeTOGHPOw-PXoI-YO5Z1dH-ZIklKM1HLK9qMKU1cchfPYhxjJYkaCO86i4wzKqGEciBN_jMrFTvFzDp0SGjLXQbQHa4QJ8RDI2gThDg45cS-CfYECkDH5P2Dg5Y2_bl_zcy2fuuZzd-_BGBtIhyMs6kCrvwSPPKPPG3LQAryIs_82zYsDOAhCn-K1gLmYNV0KSXI2xAsEi0xhYScMDGhzkdk6EhbN8XJzcfPTt90xIpgs9Opfc6La_ympeXbprMm9HTUnoL1Fse12wofo8Gc5EbTuuOfJwPOkjEY9mnj6Kt4KoPXpWblFQfu9Q6cSwQkYdVhN3f0Qu1IoGHdCLpzql6JNy7JuaJ3ldMnbnDd5A8ugghbtrJFZ3aAf1f2ghocD1NaH5eh0-90Go7wfJ3tgiKBe7tum4GnflVz0RQEgKc2KsiqAiS5vcjScpUa70b5dsDcoQSPuyVQiypFdLVWb0SxL04zSNK9Qp7SrhISUNoyBLEHi9sUASicRR2LdbunaCVI37jwG9eTvX4LgvdoZIablsD6MSIhrL5x0AklYTmu3E_bPpVfn-A">