1 |
On Tue, Jan 21, 2014 at 07:51:34AM +0100, justin wrote: |
2 |
> On 20/01/14 18:53, Reinis Danne wrote: |
3 |
> > Hi! |
4 |
> > |
5 |
> > I have cleaned up a bit alternatives-2.eclass to make it easier to review. |
6 |
> > Also if someone knows any shortcommings of this eclass as it is, then point |
7 |
> > it out, I'll try to address them. |
8 |
> > |
9 |
> > I hope this can be soon moved to the main tree thus closing the divide between |
10 |
> > science overlay and portage tree. |
11 |
> > |
12 |
> > Reinis Danne (10): |
13 |
> > alternatives-2.eclass: Update copyright |
14 |
> > alternatives-2.eclass: Remove unused variables |
15 |
> > alternatives-2.eclass: Move EXPORT_FUNCTIONS to top |
16 |
> > alternatives-2.eclass: Remove commented-out code |
17 |
> > alternatives-2.eclass: Put commonly used path in local variable |
18 |
> > alternatives-2.eclass: Fix eclass name in comments |
19 |
> > alternatives-2.eclass: Add EAPI check |
20 |
> > alternatives-2.eclass: Add documentation comments |
21 |
> > alternatives-2.eclass: Minor comment changes |
22 |
> > alternatives-2.eclass: Use consistent quoting |
23 |
> > |
24 |
> > eclass/alternatives-2.eclass | 110 +++++++++++++++++++++++++++++++------------ |
25 |
> > 1 file changed, 79 insertions(+), 31 deletions(-) |
26 |
> > |
27 |
> |
28 |
> Thanks for all your work. |
29 |
> |
30 |
> Sebastien, Dennis, Ulrich and everybody else, let's gather a list of the |
31 |
> remaining issue to resolve. |
32 |
> |
33 |
> |
34 |
> Reinis, could you branch the sci overlay on github and and push your |
35 |
> changes there. Everybody, let's stick to clear commit msgs for the |
36 |
> eclass so that the review goes easy. |
37 |
|
38 |
They are in alternatives-2 branch on |
39 |
https://github.com/Reinis/sci |
40 |
|
41 |
Added also: |
42 |
alternatives-2.eclass: Fix einfo message in pkg_postinst |
43 |
alternatives-2.eclass: Fix check for uninstalling |
44 |
alternatives-2.eclass: Don't source eselect modules |
45 |
|
46 |
With that it keeps provider selected if it is reinstalled/updated |
47 |
and avoids recreating the eselect module every single time. |
48 |
|
49 |
|
50 |
Reinis |