Bug 91659 - [PATCH] Buildsystem finds python 3 instead of python 2
Summary: [PATCH] Buildsystem finds python 3 instead of python 2
Status: RESOLVED FIXED
Alias: None
Product: Telepathy
Classification: Unclassified
Component: tp-qt (show other bugs)
Version: git master
Hardware: Other All
: medium normal
Assignee: Telepathy bugs list
QA Contact: Telepathy bugs list
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-08-16 19:38 UTC by Alex Merry
Modified: 2015-10-06 06:38 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
Patch to fix the issue (1.69 KB, patch)
2015-08-16 19:38 UTC, Alex Merry
Details | Splinter Review

Description Alex Merry 2015-08-16 19:38:55 UTC
Created attachment 117721 [details] [review]
Patch to fix the issue

ArchLinux has Python 3 as the default python. When building telepathy-qt, this is the version of Python that will be found.

However, CMake's FindPythonInterp module is perfectly capable of finding Python 2 if you tell it that's what you want. The version just needs to be passed into the find_package() call.

The attached patch does this.
Comment 1 Alexandr Akulich 2015-10-06 06:38:57 UTC
Hello.

Sorry for the long reply. I just subscribed to the bugs mail list, so I would review further patches much faster :)

Reviewed, tested and pushed: http://cgit.freedesktop.org/telepathy/telepathy-qt/commit/?id=3716c048d65d4984513591cb8b1031858201db6c

Thanks!


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.