use correct TCL configure options for Python 3.11+

[Flamefire]

(created using eb --new-pr)

See https://docs.python.org/3/whatsnew/3.11.html

Note Use the environment variables TCLTK_CFLAGS and TCLTK_LIBS to manually specify the location of Tcl/Tk headers and libraries. The configure options --with-tcltk-includes and --with-tcltk-libs have been removed.

[Flamefire]

Test report by @Flamefire

Overview of tested easyconfigs (in order)

• SUCCESS Tk-8.6.13-GCCcore-12.3.0.eb
• SUCCESS Tkinter-3.11.3-GCCcore-12.3.0.eb

Build succeeded for 2 out of 2 (2 easyconfigs in total)
taurusml2 - Linux RHEL 7.6, POWER, 8335-GTX (power9le), 6 x NVIDIA Tesla V100-SXM2-32GB, 440.64.00, Python 2.7.5
See https://gist.github.com/Flamefire/3468f6de06fd7c7c50962c83634b73f9 for a full test report.

[boegel]

@Flamefire The test reports should be for Python itself, no?

It seems like that isn't actually causing real problems up until now, since the Tcl & Tk dependencies are being picked up just fine when these $TCLTK_* environment variables are not set?

We should fix it of course, just wondering why we haven't really bumped into trouble here...

[boegel]

Test report by @boegel

Overview of tested easyconfigs (in order)

• SUCCESS Python-3.10.8-GCCcore-12.2.0.eb
• SUCCESS Python-3.11.2-GCCcore-12.2.0-bare.eb
• SUCCESS Python-3.11.3-GCCcore-12.3.0.eb
• SUCCESS Python-3.11.5-GCCcore-13.2.0.eb

Build succeeded for 4 out of 4 (4 easyconfigs in total)
node3147.skitty.os - Linux RHEL 8.8, x86_64, Intel(R) Xeon(R) Gold 6140 CPU @ 2.30GHz, Python 3.6.8
See https://gist.github.com/boegel/67f984b6e32ea68337ff1639e6316cfd for a full test report.

[branfosj]

@Flamefire The test reports should be for Python itself, no?

It seems like that isn't actually causing real problems up until now, since the Tcl & Tk dependencies are being picked up just fine when these $TCLTK_* environment variables are not set?

We should fix it of course, just wondering why we haven't really bumped into trouble here...

Tkinter should be tested - https://github.com/easybuilders/easybuild-easyblocks/blob/develop/easybuild/easyblocks/t/tkinter.py#L47

[branfosj]

Test report by @branfosj

Overview of tested easyconfigs (in order)

• SUCCESS Tkinter-2.7.18-GCCcore-11.2.0.eb
• SUCCESS Tkinter-3.10.8-GCCcore-12.2.0.eb
• SUCCESS Tkinter-3.11.3-GCCcore-12.3.0.eb

Build succeeded for 3 out of 3 (3 easyconfigs in total)
bear-pg0105u03a - Linux RHEL 8.6, x86_64, Intel(R) Xeon(R) Platinum 8360Y CPU @ 2.40GHz (icelake), Python 3.6.8
See https://gist.github.com/branfosj/bc4bb985076b33be5aee3203ac1af191 for a full test report.

[boegel]

@branfosj all good, ready to merge?

[branfosj]

Going in, thanks @Flamefire!

[Flamefire]

It seems like that isn't actually causing real problems up until now, since the Tcl & Tk dependencies are being picked up just fine when these $TCLTK_* environment variables are not set?

We should fix it of course, just wondering why we haven't really bumped into trouble here...

Then we don't need to set those configureopts at all. #3025 detects that those are (now) unused. Not setting the options/variables runs the risk of picking up a system installed Tcl/Tk which seems to be why those options were added in the easyblock.