Gentoo Archives: gentoo-science

From: Reinis Danne <rei4dan@×××××.com>
To: gentoo-science@l.g.o
Subject: Re: [gentoo-science] [PATCH 00/10] alternatives-2.eclass updates
Date: Tue, 21 Jan 2014 18:32:37
Message-Id: 20140121183229.GA13149@dhcppc1
In Reply to: Re: [gentoo-science] [PATCH 00/10] alternatives-2.eclass updates by justin
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