Differential D919

add kdev-python to KDevelop
ClosedPublic
Actions

Authored by brauch on Feb 8 2016, 12:05 PM.

Details

Reviewers

kfunk

Summary

create a new package python-libs which downloads the current python release from python.org, extracts it and places python.dll (only that) in a folder
make kdev-python depend on that
make kdevelop depend on kdev-python

Diff Detail

Lint

Lint Skipped

Unit

Unit Tests Skipped

brauch updated this revision to Diff 2232.Feb 8 2016, 12:05 PM

brauch retitled this revision from to add kdev-python to KDevelop.

brauch updated this object.

brauch edited the test plan for this revision. (Show Details)

brauch added a reviewer: kfunk.

brauch added projects: KDevelop, Craft.

Rest LGTM

portage/binary/python-libs/python-libs.py
26	I'd use forward slashes, or `join("path", "to", "foo")`
portage/extragear/kdevelop/kdevelop.py
43	I'd remove that hunk. Let's rather create some kdevelop-umbrella package, which depends on both kdevelop + kdev-python (+ potential other plugins).

This revision is now accepted and ready to land.Feb 9 2016, 10:49 AM

also install the lib/ directory of python into the bin folder, otherwise you get crashes when python is not installed
Python uses Python scripts to decode its source files (e.g. utf-8). So they have to be around. Pure elegance. And don't try to disentangle the encoding stuff from the rest of the 2 MB library, it's not going to work.

I'll add the umbrealla package the next days. Want me to push this already or shall I wait?

Feel free to commit this (without the change in kdevelop.py)

submitted.

Revision Contents
Changeset List

		Path
A	M	portage/binary/python-libs/python-libs.py (31 lines)
M		portage/extragear/kdevelop-plugins/kdev-python/kdev-python.py (3 lines)
M		portage/extragear/kdevelop/kdevelop.py (1 line)