Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 29392 - Fluxspace has python2.2 paths hardcoded
Summary: Fluxspace has python2.2 paths hardcoded
Status: RESOLVED UPSTREAM
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Commonbox Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 33372
  Show dependency tree
 
Reported: 2003-09-22 17:43 UTC by Bryan Østergaard (RETIRED)
Modified: 2003-12-06 06:42 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
Removes hardcoded python2.2 paths (fluxspace-0.0.3.ebuild.diff,678 bytes, patch)
2003-09-22 17:45 UTC, Bryan Østergaard (RETIRED)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Bryan Østergaard (RETIRED) gentoo-dev 2003-09-22 17:43:24 UTC
The fluxspace ebuild should choose the correct paths based on currently used
python version. This will make sure it installs correctly when when the python
2.3 update happens at some point. It will also bring the dependencies and the
code in src_install() more in line with each other.

I will attach a diff for an updated ebuild doing just that.


Reproducible: Always
Steps to Reproduce:
Comment 1 Bryan Østergaard (RETIRED) gentoo-dev 2003-09-22 17:45:05 UTC
Created attachment 18172 [details, diff]
Removes hardcoded python2.2 paths
Comment 2 Brandon Hale (RETIRED) gentoo-dev 2003-10-02 18:52:33 UTC
This works, but looks pretty messy to me. This should really be handled ideally
by the upstream fluxspace guys, and less ideally agreed upon by our own python
team. Comments you guys?
Comment 3 Bryan Østergaard (RETIRED) gentoo-dev 2003-10-03 01:48:44 UTC
The fluxspace website says it depends on python and is tested with version
2.2, so I agree it might be messy allowing any python version.

But I don't think the current ebuild is much better than this.
a. it has DEPEND=">=dev-lang/python-2.1"
b. src_install contains "dosym /usr/lib/python2.2/site-packages/fluxspace.py
.."

I still think this is messy and should be corrected.

In my opinion we have two options:
1. set DEPEND="dev-lang/python-2.2*" (as per the website)
2. remove hardcoded paths from src_install (as per my patch or another cleaner
patch if somebody offers one)
Comment 4 Commonbox Team gentoo-dev 2003-10-19 07:45:06 UTC
http://sourceforge.net/tracker/index.php?func=detail&atid=548097&aid=826379&group_id=76737

Giving that a try, but upstream doesnt seem very active..
Comment 5 Markus Nigbur (RETIRED) gentoo-dev 2003-12-06 00:24:11 UTC
we will monitor any upstream activity (see comment #4)