dee

Merge lp:~mhr3/dee/revert-python3-subdir into lp:dee

Proposed by Michal Hruby on 2012-06-27
Status: Merged
Approved by: Didier Roche on 2012-06-27
Approved revision: 372
Merged at revision: 370
Proposed branch: lp:~mhr3/dee/revert-python3-subdir
Merge into: lp:dee
Diff against target: 178 lines (+2/-130)
6 files modified
bindings/Makefile.am (+1/-1)
bindings/python/Dee.py (+1/-1)
bindings/python/Makefile.am (+0/-2)
bindings/python3/Dee.py (+0/-115)
bindings/python3/Makefile.am (+0/-10)
configure.ac (+0/-1)
To merge this branch: bzr merge lp:~mhr3/dee/revert-python3-subdir
Reviewer Review Type Date Requested Status
Didier Roche Approve on 2012-06-27
Łukasz Zemczak 2012-06-27 Approve on 2012-06-27
Review via email: mp+112385@code.launchpad.net

Commit Message

Revert the addition of python3 subdir

Description of the Change

Revert the addition of python3 subdir, this needs to be done on package level atm.
Keep the python3-compatible exception raising in the dee override file.

To post a comment you must log in.
Didier Roche (didrocks) wrote :

ok, after IRC discussion, the removal is right, we'll handle that in the packaging with 2 builds.

Unity Merger (unity-merger) wrote :

The Jenkins job https://jenkins.qa.ubuntu.com/job/automerge-dee/71/console reported an error when processing this lp:~mhr3/dee/revert-python3-subdir branch.
Not merging it.

Łukasz Zemczak (sil2100) wrote :

+1

review: Approve
lp:~mhr3/dee/revert-python3-subdir updated on 2012-06-27
372. By Michal Hruby on 2012-06-27

Really revert R368

Didier Roche (didrocks) wrote :

booooh, Michal doesn't know how to use bzr :p

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'bindings/Makefile.am'
2--- bindings/Makefile.am 2012-05-18 18:38:26 +0000
3+++ bindings/Makefile.am 2012-06-27 15:58:18 +0000
4@@ -1,2 +1,2 @@
5-SUBDIRS = python python3
6+SUBDIRS = python
7
8
9=== modified file 'bindings/python/Dee.py'
10--- bindings/python/Dee.py 2012-01-09 15:57:36 +0000
11+++ bindings/python/Dee.py 2012-06-27 15:58:18 +0000
12@@ -90,7 +90,7 @@
13 max_col = self.get_n_columns ()
14 for column, value in enumerate (row):
15 if column >= max_col:
16- raise IndexError, "Too many columns in row assignment: %s" % column
17+ raise IndexError("Too many columns in row assignment: %s" % column)
18 self.set_value (itr, column, value)
19
20 def get_row (self, itr):
21
22=== modified file 'bindings/python/Makefile.am'
23--- bindings/python/Makefile.am 2011-02-14 09:17:36 +0000
24+++ bindings/python/Makefile.am 2012-06-27 15:58:18 +0000
25@@ -1,5 +1,3 @@
26-PLATFORM_VERSION = 2.0
27-
28 pkgpyexecdir = $(pyexecdir)/gi
29
30 pygioverridesdir = $(pkgpyexecdir)/overrides
31
32=== removed directory 'bindings/python3'
33=== removed file 'bindings/python3/Dee.py'
34--- bindings/python3/Dee.py 2012-05-18 18:38:26 +0000
35+++ bindings/python3/Dee.py 1970-01-01 00:00:00 +0000
36@@ -1,115 +0,0 @@
37-from gi.overrides import override
38-from gi.importer import modules
39-
40-Dee = modules['Dee']._introspection_module
41-from gi.repository import GLib
42-
43-__all__ = []
44-
45-class RowWrapper:
46- def __init__ (self, model, itr):
47- self.model = model
48- self.itr = itr
49-
50- def __getitem__ (self, column):
51- return self.model.get_value(self.itr, column)
52-
53- def __setitem__ (self, column, val):
54- self.model.set_value (self.itr, column, val)
55-
56- def __iter__ (self):
57- for column in range(self.model.get_n_columns()):
58- yield self.model.get_value (self.itr, column)
59-
60- def __len__ (self):
61- return self.model.get_n_columns()
62-
63- def __str__ (self):
64- return "(%s)" % ", ".join(map(str,self))
65-
66- def __eq__ (self, other):
67- if not isinstance (other, RowWrapper):
68- return False
69- if self.model != other.model:
70- return False
71- return self.itr == other.itr
72-
73-class Model(Dee.Model):
74-
75- def __init__(self):
76- Dee.Model.__init__(self)
77-
78- def set_schema (self, *args):
79- self.set_schema_full (tuple(args), len(args))
80-
81- def _build_row (self, args):
82- result = []
83- for i, arg in enumerate(args):
84- if isinstance(arg, GLib.Variant):
85- result.append(arg)
86- else:
87- result.append(GLib.Variant(self.get_column_schema(i), arg))
88- return tuple(result)
89-
90- def prepend (self, *args):
91- return self.prepend_row (self._build_row(args))
92-
93- def append (self, *args):
94- return self.append_row (self._build_row(args))
95-
96- def insert (self, pos, *args):
97- return self.insert_row (pos, self._build_row(args))
98-
99- def insert_before (self, iter, *args):
100- return self.insert_row_before (iter, self._build_row(args))
101-
102- def insert_sorted (self, sort_func, *args):
103- return self.insert_row_sorted (self._build_row(args), sort_func, None)
104-
105- def find_sorted (self, sort_func, *args):
106- return self.find_row_sorted (self._build_row(args), sort_func, None)
107-
108- def get_schema (self):
109- return Dee.Model.get_schema(self)[0]
110-
111- def get_value (self, itr, column):
112- return Dee.Model.get_value (self, itr, column).unpack()
113-
114- def set_value (self, itr, column, value):
115- var = GLib.Variant (self.get_column_schema(column), value)
116- if isinstance (itr, int):
117- itr = self.get_iter_at_row(itr)
118- Dee.Model.set_value (self, itr, column, var)
119-
120- def __getitem__ (self, itr):
121- if isinstance (itr, int):
122- itr = self.get_iter_at_row(itr)
123- return RowWrapper(self, itr)
124-
125- def __setitem__ (self, itr, row):
126- max_col = self.get_n_columns ()
127- for column, value in enumerate (row):
128- if column >= max_col:
129- raise IndexError("Too many columns in row assignment: %s" % column)
130- self.set_value (itr, column, value)
131-
132- def get_row (self, itr):
133- return self[itr]
134-
135- def __iter__ (self):
136- itr = self.get_first_iter ()
137- last = self.get_last_iter ()
138- while itr != last:
139- yield self.get_row(itr)
140- itr = self.next(itr)
141- raise StopIteration
142-
143- def __len__ (self):
144- return self.get_n_rows()
145-
146-
147-
148-Model = override(Model)
149-__all__.append('Model')
150-
151-
152
153=== removed file 'bindings/python3/Makefile.am'
154--- bindings/python3/Makefile.am 2012-05-18 18:38:26 +0000
155+++ bindings/python3/Makefile.am 1970-01-01 00:00:00 +0000
156@@ -1,10 +0,0 @@
157-PLATFORM_VERSION = 3.0
158-
159-pkgpyexecdir = $(pyexecdir)/gi
160-
161-pygioverridesdir = $(pkgpyexecdir)/overrides
162-pygioverrides_PYTHON = \
163- Dee.py
164-
165-EXTRA_DIST = Dee.py
166-
167
168=== modified file 'configure.ac'
169--- configure.ac 2012-05-18 18:38:26 +0000
170+++ configure.ac 2012-06-27 15:58:18 +0000
171@@ -193,7 +193,6 @@
172 src/Makefile
173 bindings/Makefile
174 bindings/python/Makefile
175- bindings/python3/Makefile
176 doc/Makefile
177 doc/reference/Makefile
178 doc/reference/dee-1.0/Makefile

Subscribers

People subscribed via source and target branches

to all changes: