Bug 200518 - games/golly: Force CMake to use python27 (and ignore python34)
Summary: games/golly: Force CMake to use python27 (and ignore python34)
Status: Closed Not A Bug
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Max Brazhnikov
URL:
Keywords:
Depends on: 168159
Blocks:
  Show dependency tree / graph
 
Reported: 2015-05-29 15:05 UTC by me
Modified: 2015-06-01 08:25 UTC (History)
1 user (show)

See Also:
bugzilla: maintainer-feedback? (makc)

Attachments
Patch the already existing patch to CMakeLists.txt to force python27 (572 bytes, patch)
2015-05-29 15:05 UTC, me 		no flags Details | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description me 2015-05-29 15:05:34 UTC 
Created attachment 157250 [details]
Patch the already existing patch to CMakeLists.txt to force python27

If python34 is present, cmake detects it and tries to use it, although python27 should be used. The build fails with:

/usr/ports/games/golly/work/golly-2.4-src/wxpython.cpp:3007:7: error: use of undeclared identifier 'Py_InitModule'

With the patch attached, cmake is forced to use python27 and the build succeeds even with python34 present.

Unfortunately, there is a runtime error still:

An assertion failed!
./include/wx/string.h(1536): assert "!empty()" failed in Last(): wxString: index out of bounds

I do not know, if this has anything to do with the python version and if it has any real consequences.
Comment 1 Max Brazhnikov freebsd_committer freebsd_triage 2015-05-31 19:50:50 UTC 
The correct fix has been submitted in https://bugs.freebsd.org/168159.
Comment 2 Kubilay Kocak freebsd_committer freebsd_triage 2015-06-01 06:52:28 UTC 
Is attachment 157250 [details] still necessary in lieu of bug 168159 fixed? If not, please obsolete it
Comment 3 Max Brazhnikov freebsd_committer freebsd_triage 2015-06-01 08:20:26 UTC 
Comment on attachment 157250 [details]
Patch the already existing patch to CMakeLists.txt to force python27

The patch is superseded by a fix in bug 168159
Comment 4 Kubilay Kocak freebsd_committer freebsd_triage 2015-06-01 08:25:35 UTC 
Superseded by bug 168159