-
Notifications
You must be signed in to change notification settings - Fork 76
meson: simplify lookup of python command #83
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
According to the commit message of 82005b8 |
They solve problems with other meson specification implementations.
Lines 44 to 50 in b8260dd
So is this complex lookup of the version - which looks for '3.5', hence why its added. |
|
@fanc999 Do you think it's safe to change to wael thinks @wael444 I wonder if you misunderstood my But the version test is unnecessary. |
|
The purpose of the python module's find_installation method is to provide a rich object with methods for building python extensions. If you just need a script interpreter, and stick to the stdlib so you don't need to use the |
|
Hi @kjellahl , I think it might be ok, but I don't think it's really necessary. We could just take out the version check below, since Meson 0.55.x+ already requires Python 3.5.x or later. With blessings, and cheers! |
|
I've merged this PR and copied the commit to the libsigc++-2-12 branch. |
and remove the unused Meson variable cpp_compiler_id. Concerning Python installation, see https://gitlab.gnome.org/GNOME/glibmm/-/merge_requests/67 and #83
|
We're back where we started, at According to glibmm!67 there are Windows systems with both Although I don't know for sure, I suspect there might be systems (Windows or
|
No description provided.