Summary: | dev-libs/clearsilver: Verify and potentially improve Python-related code | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Arfrever Frehtes Taifersar Arahesis (RETIRED) <arfrever> |
Component: | Current packages | Assignee: | Gentoo Web Application Packages Maintainers <web-apps> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | idella4 |
Priority: | High | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | All | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Bug Depends on: | |||
Bug Blocks: | 308257 | ||
Attachments: |
ebuild patch
revised ebuild patch revised ebuild patch revised ebuild patch revised ebuild patch revised ebuild patch clearsilver-0.10.5-r1.ebuild.patch |
Description
Arfrever Frehtes Taifersar Arahesis (RETIRED)
2010-03-30 19:35:43 UTC
Created attachment 292371 [details, diff]
ebuild patch
Tested for usual {2.6 2.7 3.1 3.2}, all ok. Usual bump to EAPI 3.
Use PYTHON_DEPEND="python? 2". python_pkg_setup() and python_convert_shebangs() should be called only with USE="python". Maybe call python_set_active_version(): pkg_setup() { if use python; then python_set_active_version 2 python_pkg_setup fi } Created attachment 292995 [details, diff]
revised ebuild patch
ok done
(In reply to comment #3) python_convert_shebangs() should be called only with USE="python". Don't change DESTDIR="${D}" to DESTDIR="${ED}". Created attachment 293313 [details, diff]
revised ebuild patch
(In reply to comment #5) Delete 'python? ( dev-lang/python )' from DEPEND. Created attachment 293635 [details, diff]
revised ebuild patch
This took an unexpected turn. The initial build failure took place from emerging in a different gentoo. After much investigating, the lack of current ruby support invokes an install failure, access violation. finally determined system that would emerge it had NO ruby installed, other did. For the bug that isn't,
if {{ system has dev-lang/ruby ]], throw a warning and bail out. Perhaps there's another preferred option, but that's what I used here.
AND, the shebang fixer did call die with the adjusted python.eclass, removed from this patch.
How's that for value?
(In reply to comment #7) This patch is identical to previous patch. Created attachment 293757 [details, diff]
revised ebuild patch
sorry, getting mixed up between systems and versions,. double checked this 1.
(In reply to comment #9) This patch is for media-sound/lash. Created attachment 293831 [details, diff]
revised ebuild patch
Created attachment 293967 [details, diff]
clearsilver-0.10.5-r1.ebuild.patch
|