Updating brew install command to build _tkinter by izumiberat · Pull Request #893 · python/devguide (original) (raw)

I think the point is to have a nice and compact way to get the job done.

$ PKG_CONFIG_PATH="$(brew --prefix tcl-tk)/lib/pkgconfig"
./configure --with-pydebug --with-openssl=$(brew --prefix openssl)

vs

$ export PKG_CONFIG_PATH="$(brew --prefix tcl-tk)/lib/pkgconfig" $ ./configure --with-pydebug
--with-openssl=$(brew --prefix openssl)
--with-tcltk-libs="$(pkg-config --libs tcl tk)"
--with-tcltk-includes="$(pkg-config --cflags tcl tk)"

They are not the same. The latter includes 2 commands instead of 1 command to run + forces PKG_CONFIG_PATH variable to be exported. So the developer experience is much different.
We will also be shading the latter in the future, so it's good to promote the new way to configure the build as the instructions will be updated anyway (like with 3.9 EOL).

I would agree it doesn't matter if it takes a lot of effort, but the fix is already here: #893 (review)