Summary: | app-misc/gramps: Verify and potentially improve Python-related code | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Arfrever Frehtes Taifersar Arahesis (RETIRED) <arfrever> |
Component: | Current packages | Assignee: | Steve Dibb (RETIRED) <beandog> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | fauli |
Priority: | High | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | All | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Bug Depends on: | |||
Bug Blocks: | 308257 |
Description
Arfrever Frehtes Taifersar Arahesis (RETIRED)
2010-03-26 21:30:29 UTC
For the record. This is fixed in 3.2.0, which will be stabilised as soon as it is time for it. Are you sure that it's fixed? app-misc/gramps-3.2.0 doesn't set active version of Python, but depends on e.g. dev-python/pygtk which doesn't support Python 3. I only see that calls to deprecated python_version() have been removed. (In reply to comment #2) > Are you sure that it's fixed? app-misc/gramps-3.2.0 doesn't set active version > of Python, but depends on e.g. dev-python/pygtk which doesn't support Python 3. > I only see that calls to deprecated python_version() have been removed. There is no assumption on missing setting, I assume. Reopen until I have time to test with Python 3. Does not work with Python 3, so setting a version of Python 2 as DEPEND. Dependencies are now correct, but are you sure that app-misc/gramps-3.2.0 builds/installs correctly when a version of Python 3 is set as main active version of Python? python_get_sitedir() (which is used in src_prepare()) prints e.g. "/usr/lib/python3.1/site-packages" when Python 3.1 is set as main active version of Python. Ebuilds can easily set active version of Python. In ebuilds not supporting installation for multiple Python ABIs, you should use python_set_active_version(). In ebuilds supporting installation for multiple Python ABIs, you should usually use python_execute_function(). python_set_active_version() and python_execute_function() are documented in documentation. So I got it right by setting the active version in pkg_setup only? If yes, this bug should be finally fixed. (In reply to comment #6) It now looks good (but I haven't tested it). |