| … | |
… | |
| 6 | PEP, see http://www.python.org/peps/pep-0001.html for instructions and links |
6 | PEP, see http://www.python.org/peps/pep-0001.html for instructions and links |
| 7 | to templates. DO NOT USE THIS HTML FILE AS YOUR TEMPLATE! |
7 | to templates. DO NOT USE THIS HTML FILE AS YOUR TEMPLATE! |
| 8 | --> |
8 | --> |
| 9 | <head> |
9 | <head> |
| 10 | <meta http-equiv="Content-Type" content="text/html; charset=utf-8" /> |
10 | <meta http-equiv="Content-Type" content="text/html; charset=utf-8" /> |
| 11 | <meta name="generator" content="Docutils 0.3.5: http://docutils.sourceforge.net/" /> |
11 | <meta name="generator" content="Docutils 0.3.9: http://docutils.sourceforge.net/" /> |
| 12 | <title>GLEP 33 -- Eclass Restructure/Redesign</title> |
12 | <title>GLEP 33 -- Eclass Restructure/Redesign</title> |
| 13 | <link rel="stylesheet" href="tools/glep.css" type="text/css" /> |
13 | <link rel="stylesheet" href="tools/glep.css" type="text/css" /> |
| 14 | </head> |
14 | </head> |
| 15 | <body bgcolor="white"> |
15 | <body bgcolor="white"> |
| 16 | <table class="navigation" cellpadding="0" cellspacing="0" |
16 | <table class="navigation" cellpadding="0" cellspacing="0" |
| … | |
… | |
| 22 | <td class="textlinks" align="left"> |
22 | <td class="textlinks" align="left"> |
| 23 | [<b><a href="http://www.gentoo.org/">Gentoo Linux Home</a></b>] |
23 | [<b><a href="http://www.gentoo.org/">Gentoo Linux Home</a></b>] |
| 24 | [<b><a href="http://www.gentoo.org/proj/en/glep">GLEP Index</a></b>] |
24 | [<b><a href="http://www.gentoo.org/proj/en/glep">GLEP Index</a></b>] |
| 25 | [<b><a href="./glep-0033.txt">GLEP Source</a></b>] |
25 | [<b><a href="./glep-0033.txt">GLEP Source</a></b>] |
| 26 | </td></tr></table> |
26 | </td></tr></table> |
| 27 | <div class="document"> |
|
|
| 28 | <table class="rfc2822 field-list" frame="void" rules="none"> |
27 | <table class="rfc2822 docutils field-list" frame="void" rules="none"> |
| 29 | <col class="field-name" /> |
28 | <col class="field-name" /> |
| 30 | <col class="field-body" /> |
29 | <col class="field-body" /> |
| 31 | <tbody valign="top"> |
30 | <tbody valign="top"> |
| 32 | <tr class="field"><th class="field-name">GLEP:</th><td class="field-body">33</td> |
31 | <tr class="field"><th class="field-name">GLEP:</th><td class="field-body">33</td> |
| 33 | </tr> |
32 | </tr> |
| 34 | <tr class="field"><th class="field-name">Title:</th><td class="field-body">Eclass Restructure/Redesign</td> |
33 | <tr class="field"><th class="field-name">Title:</th><td class="field-body">Eclass Restructure/Redesign</td> |
| 35 | </tr> |
34 | </tr> |
| 36 | <tr class="field"><th class="field-name">Version:</th><td class="field-body">1.2</td> |
35 | <tr class="field"><th class="field-name">Version:</th><td class="field-body">1.5</td> |
| 37 | </tr> |
36 | </tr> |
| 38 | <tr class="field"><th class="field-name">Last-Modified:</th><td class="field-body"><a class="reference" href="http://www.gentoo.org/cgi-bin/viewcvs/xml/htdocs/proj/en/glep/glep-0033.txt?cvsroot=gentoo">2005/03/06 20:33:20</a></td> |
37 | <tr class="field"><th class="field-name">Last-Modified:</th><td class="field-body"><a class="reference" href="http://www.gentoo.org/cgi-bin/viewcvs/xml/htdocs/proj/en/glep/glep-0033.txt?cvsroot=gentoo">2005/09/15 21:02:11</a></td> |
| 39 | </tr> |
38 | </tr> |
| 40 | <tr class="field"><th class="field-name">Author:</th><td class="field-body">Brian Harring <ferringb at gentoo.org>, John Mylchreest <johnm at gentoo.org></td> |
39 | <tr class="field"><th class="field-name">Author:</th><td class="field-body">Brian Harring <ferringb at gentoo.org>, John Mylchreest <johnm at gentoo.org></td> |
| 41 | </tr> |
40 | </tr> |
| 42 | <tr class="field"><th class="field-name">Status:</th><td class="field-body">Draft</td> |
41 | <tr class="field"><th class="field-name">Status:</th><td class="field-body">Approved</td> |
| 43 | </tr> |
42 | </tr> |
| 44 | <tr class="field"><th class="field-name">Type:</th><td class="field-body">Standards Track</td> |
43 | <tr class="field"><th class="field-name">Type:</th><td class="field-body">Standards Track</td> |
| 45 | </tr> |
44 | </tr> |
| 46 | <tr class="field"><th class="field-name">Content-Type:</th><td class="field-body"><a class="reference" href="glep-0012.html">text/x-rst</a></td> |
45 | <tr class="field"><th class="field-name">Content-Type:</th><td class="field-body"><a class="reference" href="http://www.python.org/peps/glep-0012.html">text/x-rst</a></td> |
| 47 | </tr> |
46 | </tr> |
| 48 | <tr class="field"><th class="field-name">Created:</th><td class="field-body">29-Jan-2005</td> |
47 | <tr class="field"><th class="field-name">Created:</th><td class="field-body">29-Jan-2005</td> |
| 49 | </tr> |
48 | </tr> |
| 50 | <tr class="field"><th class="field-name">Post-History:</th><td class="field-body">29-Jan-2005</td> |
49 | <tr class="field"><th class="field-name">Post-History:</th><td class="field-body">29-Jan-2005 6-Mar-2005 15-Sep-2005</td> |
| 51 | </tr> |
50 | </tr> |
| 52 | </tbody> |
51 | </tbody> |
| 53 | </table> |
52 | </table> |
| 54 | <hr /> |
53 | <hr /> |
| 55 | <div class="contents topic" id="contents"> |
54 | <div class="contents topic" id="contents"> |
| 56 | <p class="topic-title first"><a name="contents">Contents</a></p> |
55 | <p class="topic-title first"><a name="contents">Contents</a></p> |
| 57 | <ul class="simple"> |
56 | <ul class="simple"> |
|
|
57 | <li><a class="reference" href="#status" id="id2" name="id2">Status</a></li> |
| 58 | <li><a class="reference" href="#abstract" id="id2" name="id2">Abstract</a></li> |
58 | <li><a class="reference" href="#abstract" id="id3" name="id3">Abstract</a></li> |
| 59 | <li><a class="reference" href="#terminology" id="id3" name="id3">Terminology</a></li> |
59 | <li><a class="reference" href="#terminology" id="id4" name="id4">Terminology</a></li> |
| 60 | <li><a class="reference" href="#motivation-and-rationale" id="id4" name="id4">Motivation and Rationale</a></li> |
60 | <li><a class="reference" href="#motivation-and-rationale" id="id5" name="id5">Motivation and Rationale</a></li> |
| 61 | <li><a class="reference" href="#specification" id="id5" name="id5">Specification.</a><ul> |
61 | <li><a class="reference" href="#specification" id="id6" name="id6">Specification</a><ul> |
| 62 | <li><a class="reference" href="#ebuild-libraries-elibs-for-short" id="id6" name="id6">Ebuild Libraries (elibs for short)</a></li> |
62 | <li><a class="reference" href="#ebuild-libraries-elibs-for-short" id="id7" name="id7">Ebuild Libraries (elibs for short)</a></li> |
| 63 | <li><a class="reference" href="#the-reduced-role-of-eclasses-and-a-clarification-of-existing-eclass-requirements" id="id7" name="id7">The reduced role of Eclasses, and a clarification of existing Eclass requirements</a></li> |
63 | <li><a class="reference" href="#the-reduced-role-of-eclasses-and-a-clarification-of-existing-eclass-requirements" id="id8" name="id8">The reduced role of Eclasses, and a clarification of existing Eclass requirements</a></li> |
| 64 | <li><a class="reference" href="#the-end-of-backwards-compatibility" id="id8" name="id8">The end of backwards compatibility...</a></li> |
64 | <li><a class="reference" href="#the-end-of-backwards-compatibility" id="id9" name="id9">The end of backwards compatibility...</a></li> |
| 65 | <li><a class="reference" href="#tree-restructuring" id="id9" name="id9">Tree restructuring.</a></li> |
65 | <li><a class="reference" href="#tree-restructuring" id="id10" name="id10">Tree restructuring</a></li> |
| 66 | <li><a class="reference" href="#the-start-of-a-different-phase-of-backwards-compatibility" id="id10" name="id10">The start of a different phase of backwards compatibility</a></li> |
66 | <li><a class="reference" href="#the-start-of-a-different-phase-of-backwards-compatibility" id="id11" name="id11">The start of a different phase of backwards compatibility</a></li> |
| 67 | <li><a class="reference" href="#migrating-to-the-new-setup" id="id11" name="id11">Migrating to the new setup</a></li> |
67 | <li><a class="reference" href="#migrating-to-the-new-setup" id="id12" name="id12">Migrating to the new setup</a></li> |
| 68 | </ul> |
68 | </ul> |
| 69 | </li> |
69 | </li> |
| 70 | <li><a class="reference" href="#backwards-compatibility" id="id12" name="id12">Backwards Compatibility</a></li> |
70 | <li><a class="reference" href="#backwards-compatibility" id="id13" name="id13">Backwards Compatibility</a></li> |
| 71 | <li><a class="reference" href="#copyright" id="id13" name="id13">Copyright</a></li> |
71 | <li><a class="reference" href="#copyright" id="id14" name="id14">Copyright</a></li> |
| 72 | </ul> |
72 | </ul> |
| 73 | </div> |
73 | </div> |
|
|
74 | <div class="section" id="status"> |
|
|
75 | <h1><a class="toc-backref" href="#id2" name="status">Status</a></h1> |
|
|
76 | <p>Approved by the Gentoo Council on 15 September 2005.</p> |
|
|
77 | </div> |
| 74 | <div class="section" id="abstract"> |
78 | <div class="section" id="abstract"> |
| 75 | <h1><a class="toc-backref" href="#id2" name="abstract">Abstract</a></h1> |
79 | <h1><a class="toc-backref" href="#id3" name="abstract">Abstract</a></h1> |
| 76 | <p>For any design, the transition from theoretical to applied exposes inadequacies |
80 | <p>For any design, the transition from theoretical to applied exposes inadequacies |
| 77 | in the original design. This document is intended to document, and propose a |
81 | in the original design. This document is intended to document, and propose a |
| 78 | revision of the current eclass setup to address current eclass inadequacies.</p> |
82 | revision of the current eclass setup to address current eclass inadequacies.</p> |
| 79 | <p>This document proposes several things- the creation of ebuild libraries, 'elibs', |
83 | <p>This document proposes several things- the creation of ebuild libraries, 'elibs', |
| 80 | a narrowing of the focus of eclasses, a move of eclasses w/in the tree, the |
84 | a narrowing of the focus of eclasses, a move of eclasses w/in the tree, the |
| 81 | addition of changelogs, and a way to allow for simple eclass gpg signing. |
85 | addition of changelogs, and a way to allow for simple eclass gpg signing. |
| 82 | In general, a large scale restructuring of what eclasses are and how they're |
86 | In general, a large scale restructuring of what eclasses are and how they're |
| 83 | implemented. Essentially version two of the eclass setup.</p> |
87 | implemented. Essentially version two of the eclass setup.</p> |
| 84 | </div> |
88 | </div> |
| 85 | <div class="section" id="terminology"> |
89 | <div class="section" id="terminology"> |
| 86 | <h1><a class="toc-backref" href="#id3" name="terminology">Terminology</a></h1> |
90 | <h1><a class="toc-backref" href="#id4" name="terminology">Terminology</a></h1> |
| 87 | <p>From this point on, the proposed eclass setup will be called 'new eclasses', the |
91 | <p>From this point on, the proposed eclass setup will be called 'new eclasses', the |
| 88 | existing crop (as of this writing) will be referenced as 'old eclasses'. The |
92 | existing crop (as of this writing) will be referenced as 'old eclasses'. The |
| 89 | distinction is elaborated on within this document.</p> |
93 | distinction is elaborated on within this document.</p> |
| 90 | </div> |
94 | </div> |
| 91 | <div class="section" id="motivation-and-rationale"> |
95 | <div class="section" id="motivation-and-rationale"> |
| 92 | <h1><a class="toc-backref" href="#id4" name="motivation-and-rationale">Motivation and Rationale</a></h1> |
96 | <h1><a class="toc-backref" href="#id5" name="motivation-and-rationale">Motivation and Rationale</a></h1> |
| 93 | <p>Eclasses within the tree currently are a bit of a mess- they're forced to |
97 | <p>Eclasses within the tree currently are a bit of a mess- they're forced to |
| 94 | maintain backwards compatibility w/ all previous functionality. In effect, |
98 | maintain backwards compatibility w/ all previous functionality. In effect, |
| 95 | their api is constant, and can only be added to- never changing the existing |
99 | their api is constant, and can only be added to- never changing the existing |
| 96 | functionality. This obviously is quite limiting, and leads to cruft accruing in |
100 | functionality. This obviously is quite limiting, and leads to cruft accruing in |
| 97 | eclasses as a eclasses design is refined. This needs to be dealt with prior to |
101 | eclasses as a eclasses design is refined. This needs to be dealt with prior to |
| 98 | eclass code reaching a critical mass where they become unmanageable/fragile |
102 | eclass code reaching a critical mass where they become unmanageable/fragile |
| 99 | (recent pushes for eclass versioning could be interpreted as proof of this).</p> |
103 | (recent pushes for eclass versioning could be interpreted as proof of this).</p> |
| 100 | <p>Beyond that, eclasses were originally intended as a method to allow for ebuilds |
104 | <p>Beyond that, eclasses were originally intended as a method to allow for ebuilds |
| 101 | to use a pre-existing block of code, rather then having to duplicate the code in |
105 | to use a pre-existing block of code, rather then having to duplicate the code in |
| 102 | each ebuild. This is a good thing, but there are ill effects that result from |
106 | each ebuild. This is a good thing, but there are ill effects that result from |
| 103 | the current design. Eclasses inherit other eclasses to get a single function- in |
107 | the current design. Eclasses inherit other eclasses to get a single function- in |
| 104 | doing so, modifying the the exported 'template' (default src_compile, default |
108 | doing so, modifying the the exported 'template' (default src_compile, default |
| 105 | src_unpack, various vars, etc). All the eclass designer was after was reusing a |
109 | src_unpack, various vars, etc). All the eclass designer was after was reusing a |
| 106 | function, not making their eclass sensitive to changes in the template of the |
110 | function, not making their eclass sensitive to changes in the template of the |
| 107 | eclass it's inheriting. The eclass designer -should- be aware of changes in the |
111 | eclass it's inheriting. The eclass designer -should- be aware of changes in the |
| 108 | function they're using, but shouldn't have to worry about their default src_* |
112 | function they're using, but shouldn't have to worry about their default src_* |
| 109 | and pkg_* functions being overwritten, let alone the env changes.</p> |
113 | and pkg_* functions being overwritten, let alone the env changes.</p> |
| 110 | <p>Addressing up front why a collection of eclass refinements are being rolled into |
114 | <p>Addressing up front why a collection of eclass refinements are being rolled into |
| … | |
… | |
| 113 | eclass specification was this, and that the second specification is that, |
117 | eclass specification was this, and that the second specification is that, |
| 114 | rather then requiring them to be aware of what phase of eclass changes is in |
118 | rather then requiring them to be aware of what phase of eclass changes is in |
| 115 | progress.</p> |
119 | progress.</p> |
| 116 | <p>By rolling all changes into one large change, a line is intentionally drawn in |
120 | <p>By rolling all changes into one large change, a line is intentionally drawn in |
| 117 | the sand. Old eclasses allowed for this, behaved this way. New eclasses allow |
121 | the sand. Old eclasses allowed for this, behaved this way. New eclasses allow |
| 118 | for that, and behave this way. This should reduce misconceptions about what is |
122 | for that, and behave this way. This should reduce misconceptions about what is |
| 119 | allowed/possible with eclasses, thus reducing bugs that result from said |
123 | allowed/possible with eclasses, thus reducing bugs that result from said |
| 120 | misconceptions.</p> |
124 | misconceptions.</p> |
| 121 | <p>A few words on elibs- think of them as a clear definition between behavioral |
125 | <p>A few words on elibs- think of them as a clear definition between behavioral |
| 122 | functionality of an eclass, and the library functionality. Eclass's modify |
126 | functionality of an eclass, and the library functionality. Eclass's modify |
| 123 | template data, and are the basis for other ebuilds- elibs, however are <em>just</em> |
127 | template data, and are the basis for other ebuilds- elibs, however are <em>just</em> |
| 124 | common bash functionality.</p> |
128 | common bash functionality.</p> |
| 125 | <p>Consider the majority of the portage bin/* scripts- these all are candidates for |
129 | <p>Consider the majority of the portage bin/* scripts- these all are candidates for |
| 126 | being added to the tree as elibs, as is the bulk of eutils.</p> |
130 | being added to the tree as elibs, as is the bulk of eutils.</p> |
| 127 | </div> |
131 | </div> |
| 128 | <div class="section" id="specification"> |
132 | <div class="section" id="specification"> |
| 129 | <h1><a class="toc-backref" href="#id5" name="specification">Specification.</a></h1> |
133 | <h1><a class="toc-backref" href="#id6" name="specification">Specification</a></h1> |
| 130 | <p>The various parts of this proposal are broken down into a set of changes and |
134 | <p>The various parts of this proposal are broken down into a set of changes and |
| 131 | elaborations on why a proposed change is preferable. It's advisable to the |
135 | elaborations on why a proposed change is preferable. It's advisable to the |
| 132 | reader that this be read serially, rather then jumping around.</p> |
136 | reader that this be read serially, rather then jumping around.</p> |
| 133 | <div class="section" id="ebuild-libraries-elibs-for-short"> |
137 | <div class="section" id="ebuild-libraries-elibs-for-short"> |
| 134 | <h2><a class="toc-backref" href="#id6" name="ebuild-libraries-elibs-for-short">Ebuild Libraries (elibs for short)</a></h2> |
138 | <h2><a class="toc-backref" href="#id7" name="ebuild-libraries-elibs-for-short">Ebuild Libraries (elibs for short)</a></h2> |
| 135 | <p>As briefly touched upon in Motivation and Rationale, the original eclass design |
139 | <p>As briefly touched upon in Motivation and Rationale, the original eclass design |
| 136 | allowed for the eclass to modify the metadata of an ebuild, metadata being the |
140 | allowed for the eclass to modify the metadata of an ebuild, metadata being the |
| 137 | DEPENDS, RDEPENDS, SRC_URI, IUSE, etc, vars that are required to be constant, |
141 | DEPENDS, RDEPENDS, SRC_URI, IUSE, etc, vars that are required to be constant, |
| 138 | and used by portage for dep resolution, fetching, etc. Using the earlier |
142 | and used by portage for dep resolution, fetching, etc. Using the earlier |
| 139 | example, if you're after a single function from an eclass (say epatch from |
143 | example, if you're after a single function from an eclass (say epatch from |
| … | |
… | |
| 151 | initialization of the library that is absolutely needed. Additionally, they |
155 | initialization of the library that is absolutely needed. Additionally, they |
| 152 | cannot modify any ebuild template functions- src_compile, src_unpack. Since they are |
156 | cannot modify any ebuild template functions- src_compile, src_unpack. Since they are |
| 153 | required to not modify the metadata keys, nor in any way affect the ebuild aside |
157 | required to not modify the metadata keys, nor in any way affect the ebuild aside |
| 154 | from providing functionality, they can be conditionally pulled in. They also |
158 | from providing functionality, they can be conditionally pulled in. They also |
| 155 | are allowed to pull in other elibs, but strictly just elibs- no eclasses, just |
159 | are allowed to pull in other elibs, but strictly just elibs- no eclasses, just |
| 156 | other elibs. A real world example would be the eutils eclass.</p> |
160 | other elibs. A real world example would be the eutils eclass.</p> |
| 157 | <p>Portage, since the elib's don't modify metadata, isn't required to track elibs |
161 | <p>Portage, since the elib's don't modify metadata, isn't required to track elibs |
| 158 | as it tracks eclasses. Thus a change in an elib doesn't result in half the tree |
162 | as it tracks eclasses. Thus a change in an elib doesn't result in half the tree |
| 159 | forced to be regenerated/marked stale when changed (this is more of an infra |
163 | forced to be regenerated/marked stale when changed (this is more of an infra |
| 160 | benefit, although regen's that take too long due to eclass changes have been |
164 | benefit, although regen's that take too long due to eclass changes have been |
| 161 | known to cause rsync issues due to missing timestamps).</p> |
165 | known to cause rsync issues due to missing timestamps).</p> |
| 162 | <p>Elibs will not be available in the global scope of an eclass, or ebuild- nor during the |
166 | <p>Elibs will not be available in the global scope of an eclass, or ebuild- nor during the |
| 163 | depends phase (basically a phase that sources the ebuild, to get it's metadata). Elib |
167 | depends phase (basically a phase that sources the ebuild, to get its metadata). Elib |
| 164 | calls in the global scope will be tracked, but the elib will not be loaded till just before |
168 | calls in the global scope will be tracked, but the elib will not be loaded till just before |
| 165 | the setup phase (pkg_setup). There are two reasons for this- first, it ensures elibs are |
169 | the setup phase (pkg_setup). There are two reasons for this- first, it ensures elibs are |
| 166 | completely incapable of modifying metadata. There is no room for confusion, late loading |
170 | completely incapable of modifying metadata. There is no room for confusion, late loading |
| 167 | of elibs gives you the functionality for all phases, except for depends- depends being the |
171 | of elibs gives you the functionality for all phases, except for depends- depends being the |
| 168 | only phase that is capable of specifying metadata. Second, as an added bonus, late |
172 | only phase that is capable of specifying metadata. Second, as an added bonus, late |
| 169 | loading reduces the amount of bash sourced for a regen- faster regens. This however is minor, |
173 | loading reduces the amount of bash sourced for a regen- faster regens. This however is minor, |
| 170 | and is an ancillary benefit of the first reason.</p> |
174 | and is an ancillary benefit of the first reason.</p> |
| … | |
… | |
| 184 | func x() { echo "this is invalid, do not do it."; } |
188 | func x() { echo "this is invalid, do not do it."; } |
| 185 | fi |
189 | fi |
| 186 | </pre> |
190 | </pre> |
| 187 | <p>Regarding maintainability of elibs, it should be a less of a load then old |
191 | <p>Regarding maintainability of elibs, it should be a less of a load then old |
| 188 | eclasses. One of the major issues with old eclasses is that their functions are |
192 | eclasses. One of the major issues with old eclasses is that their functions are |
| 189 | quite incestuous- they're bound tightly to the env they're defined in. This |
193 | quite incestuous- they're bound tightly to the env they're defined in. This |
| 190 | makes eclass functions a bit fragile- the restrictions on what can, and cannot |
194 | makes eclass functions a bit fragile- the restrictions on what can, and cannot |
| 191 | be done in elibs will address this, making functionality less fragile (thus a |
195 | be done in elibs will address this, making functionality less fragile (thus a |
| 192 | bit more maintainable).</p> |
196 | bit more maintainable).</p> |
| 193 | <p>There is no need for backwards compatibility with elibs- they just must work |
197 | <p>There is no need for backwards compatibility with elibs- they just must work |
| 194 | against the current tree. Thus elibs can be removed when the tree no longer |
198 | against the current tree. Thus elibs can be removed when the tree no longer |
| … | |
… | |
| 199 | what elibs are, what they are capable of, and how to use them is nailed down as much as |
203 | what elibs are, what they are capable of, and how to use them is nailed down as much as |
| 200 | possible to avoid <em>any</em> ambiguity related to them. The intention is to make it clear, |
204 | possible to avoid <em>any</em> ambiguity related to them. The intention is to make it clear, |
| 201 | such that no misconceptions occur, resulting in bugs.</p> |
205 | such that no misconceptions occur, resulting in bugs.</p> |
| 202 | </div> |
206 | </div> |
| 203 | <div class="section" id="the-reduced-role-of-eclasses-and-a-clarification-of-existing-eclass-requirements"> |
207 | <div class="section" id="the-reduced-role-of-eclasses-and-a-clarification-of-existing-eclass-requirements"> |
| 204 | <h2><a class="toc-backref" href="#id7" name="the-reduced-role-of-eclasses-and-a-clarification-of-existing-eclass-requirements">The reduced role of Eclasses, and a clarification of existing Eclass requirements</a></h2> |
208 | <h2><a class="toc-backref" href="#id8" name="the-reduced-role-of-eclasses-and-a-clarification-of-existing-eclass-requirements">The reduced role of Eclasses, and a clarification of existing Eclass requirements</a></h2> |
| 205 | <p>Since elibs are now intended on holding common bash functionality, the focus of |
209 | <p>Since elibs are now intended on holding common bash functionality, the focus of |
| 206 | eclasses should be in defining an appropriate template for ebuilds. For example, |
210 | eclasses should be in defining an appropriate template for ebuilds. For example, |
| 207 | defining common DEPENDS, RDEPENDS, src_compile functions, src_unpack, etc. |
211 | defining common DEPENDS, RDEPENDS, src_compile functions, src_unpack, etc. |
| 208 | Additionally, eclasses should pull in any elibs they need for functionality.</p> |
212 | Additionally, eclasses should pull in any elibs they need for functionality.</p> |
| 209 | <p>Eclass functionality that isn't directly related to the metadata, or src_* and |
213 | <p>Eclass functionality that isn't directly related to the metadata, or src_* and |
| 210 | pkg_* funcs should be shifted into elibs to allow for maximal code reuse. This |
214 | pkg_* funcs should be shifted into elibs to allow for maximal code reuse. This |
| 211 | however isn't a hard requirement, merely a strongly worded suggestion.</p> |
215 | however isn't a hard requirement, merely a strongly worded suggestion.</p> |
| … | |
… | |
| 238 | just that. As such new eclasses (the true distinction of new vs old is |
242 | just that. As such new eclasses (the true distinction of new vs old is |
| 239 | elaborated in the next section) can be removed from the tree once they're no |
243 | elaborated in the next section) can be removed from the tree once they're no |
| 240 | longer in use.</p> |
244 | longer in use.</p> |
| 241 | </div> |
245 | </div> |
| 242 | <div class="section" id="the-end-of-backwards-compatibility"> |
246 | <div class="section" id="the-end-of-backwards-compatibility"> |
| 243 | <h2><a class="toc-backref" href="#id8" name="the-end-of-backwards-compatibility">The end of backwards compatibility...</a></h2> |
247 | <h2><a class="toc-backref" href="#id9" name="the-end-of-backwards-compatibility">The end of backwards compatibility...</a></h2> |
| 244 | <p>With current eclasses, once the eclass is in use, it's api can no longer be |
248 | <p>With current eclasses, once the eclass is in use, its api can no longer be |
| 245 | changed, nor can the eclass ever be removed from the tree. This is why we still |
249 | changed, nor can the eclass ever be removed from the tree. This is why we still |
| 246 | have <em>ancient</em> eclasses that are completely unused sitting in the tree, for |
250 | have <em>ancient</em> eclasses that are completely unused sitting in the tree, for |
| 247 | example inherit.eclass . The reason for this, not surprisingly is a portage |
251 | example inherit.eclass. The reason for this, not surprisingly, is a portage |
| 248 | deficiency- on unmerging an installed ebuild, portage used the eclass from the |
252 | deficiency: on unmerging an installed ebuild, portage used the eclass from the |
| 249 | current tree.</p> |
253 | current tree.</p> |
| 250 | <p>For a real world example of this, if you merged a glibc 2 years back, whatever |
254 | <p>For a real world example of this, if you merged a glibc 2 years back, whatever |
| 251 | eclasses it used must still be compatible, or you may not be able to unmerge the |
255 | eclasses it used must still be compatible, or you may not be able to unmerge the |
| 252 | older glibc version during an upgrade to a newer version. So either the glibc |
256 | older glibc version during an upgrade to a newer version. So either the glibc |
| 253 | maintainer is left with the option of leaving people using ancient versions out |
257 | maintainer is left with the option of leaving people using ancient versions out |
| … | |
… | |
| 281 | still be able to access them.</p> |
285 | still be able to access them.</p> |
| 282 | <p>In other words, these new eclasses would in effect, be old eclasses since older |
286 | <p>In other words, these new eclasses would in effect, be old eclasses since older |
| 283 | portage versions could still access them.</p> |
287 | portage versions could still access them.</p> |
| 284 | </div> |
288 | </div> |
| 285 | <div class="section" id="tree-restructuring"> |
289 | <div class="section" id="tree-restructuring"> |
| 286 | <h2><a class="toc-backref" href="#id9" name="tree-restructuring">Tree restructuring.</a></h2> |
290 | <h2><a class="toc-backref" href="#id10" name="tree-restructuring">Tree restructuring</a></h2> |
| 287 | <p>There are only two way to block the existing (as of this writing) inherit |
291 | <p>There are only two way to block the existing (as of this writing) inherit |
| 288 | functionality from accessing the new eclasses- either change the extension of |
292 | functionality from accessing the new eclasses- either change the extension of |
| 289 | eclasses to something other then 'eclass', or to have them stored in a separate |
293 | eclasses to something other then 'eclass', or to have them stored in a separate |
| 290 | subdirectory of the tree then eclass.</p> |
294 | subdirectory of the tree then eclass.</p> |
| 291 | <p>The latter is preferable, and the proposed solution. Reasons are- the current |
295 | <p>The latter is preferable, and the proposed solution. Reasons are- the current |
| … | |
… | |
| 297 | <p>If it's unclear as to why the old inherit function <em>cannot</em> access the new |
301 | <p>If it's unclear as to why the old inherit function <em>cannot</em> access the new |
| 298 | eclasses, please reread the previous section. It's unfortunately a requirement |
302 | eclasses, please reread the previous section. It's unfortunately a requirement |
| 299 | to take advantage of all that the next major portage release will allow.</p> |
303 | to take advantage of all that the next major portage release will allow.</p> |
| 300 | <p>The proposed directory structure is ${PORTDIR}/include/{eclass,elib}. |
304 | <p>The proposed directory structure is ${PORTDIR}/include/{eclass,elib}. |
| 301 | Something like ${PORTDIR}/new-eclass, or ${PORTDIR}/eclass-ng could be used |
305 | Something like ${PORTDIR}/new-eclass, or ${PORTDIR}/eclass-ng could be used |
| 302 | (although many would cringe at the -ng), but such a name is unwise. Consider the |
306 | (although many would cringe at the -ng), but such a name is unwise. Consider the |
| 303 | possibility (likely a fact) that new eclasses someday may be found lacking, and |
307 | possibility (likely a fact) that new eclasses someday may be found lacking, and |
| 304 | refined further (version three as it were). Or perhaps we want to add yet more |
308 | refined further (version three as it were). Or perhaps we want to add yet more |
| 305 | functionality with direct relation to sourcing new files, and we would then need |
309 | functionality with direct relation to sourcing new files, and we would then need |
| 306 | to further populate ${PORTDIR}.</p> |
310 | to further populate ${PORTDIR}.</p> |
| 307 | <p>The new-eclass directory will be (at least) 2 levels deep- for example:</p> |
311 | <p>The new-eclass directory will be (at least) 2 levels deep- for example:</p> |
| 308 | <dl> |
312 | <dl class="docutils"> |
| 309 | <dt>::</dt> |
313 | <dt>::</dt> |
| 310 | <dd>kernel/ |
314 | <dd>kernel/ |
| 311 | kernel/linux-info.eclass |
315 | kernel/linux-info.eclass |
| 312 | kernel/linux-mod.eclass |
316 | kernel/linux-mod.eclass |
| 313 | kernel/kernel-2.6.eclass |
317 | kernel/kernel-2.6.eclass |
| … | |
… | |
| 322 | saner/easier signing of eclasses- you can just stick a signed |
326 | saner/easier signing of eclasses- you can just stick a signed |
| 323 | Manifest file w/in that grouping, thus providing the information portage needs |
327 | Manifest file w/in that grouping, thus providing the information portage needs |
| 324 | to ensure no files are missing, and that nothing has been tainted.</p> |
328 | to ensure no files are missing, and that nothing has been tainted.</p> |
| 325 | <p>The elib directory will be structured in the same way, for the same reasons.</p> |
329 | <p>The elib directory will be structured in the same way, for the same reasons.</p> |
| 326 | <p>Repoman will have to be extended to work within new eclass and elib groups, and |
330 | <p>Repoman will have to be extended to work within new eclass and elib groups, and |
| 327 | to handle signing and committing. This is intentional, and a good thing. This |
331 | to handle signing and committing. This is intentional, and a good thing. This |
| 328 | gives repoman the possibility of doing sanity checks on elibs/new eclasses.</p> |
332 | gives repoman the possibility of doing sanity checks on elibs/new eclasses.</p> |
| 329 | <p>Note these checks will not prevent developers from doing dumb things with eclass- |
333 | <p>Note these checks will not prevent developers from doing dumb things with eclass- |
| 330 | these checks would only be capable of doing basic sanity checks, such as syntax checks. |
334 | these checks would only be capable of doing basic sanity checks, such as syntax checks. |
| 331 | There is no way to prevent people from doing dumb things (exempting perhaps repeated |
335 | There is no way to prevent people from doing dumb things (exempting perhaps repeated |
| 332 | applications of a cattle prod)- these are strictly automatic checks, akin to repoman's |
336 | applications of a cattle prod)- these are strictly automatic checks, akin to repoman's |
| 333 | dependency checks.</p> |
337 | dependency checks.</p> |
| 334 | </div> |
338 | </div> |
| 335 | <div class="section" id="the-start-of-a-different-phase-of-backwards-compatibility"> |
339 | <div class="section" id="the-start-of-a-different-phase-of-backwards-compatibility"> |
| 336 | <h2><a class="toc-backref" href="#id10" name="the-start-of-a-different-phase-of-backwards-compatibility">The start of a different phase of backwards compatibility</a></h2> |
340 | <h2><a class="toc-backref" href="#id11" name="the-start-of-a-different-phase-of-backwards-compatibility">The start of a different phase of backwards compatibility</a></h2> |
| 337 | <p>As clarified above, new eclasses will exist in a separate directory that will be |
341 | <p>As clarified above, new eclasses will exist in a separate directory that will be |
| 338 | intentionally inaccessible to the inherit function. As such, users of older |
342 | intentionally inaccessible to the inherit function. As such, users of older |
| 339 | portage versions <em>will</em> have to upgrade to merge any ebuild that uses elibs/new |
343 | portage versions <em>will</em> have to upgrade to merge any ebuild that uses elibs/new |
| 340 | eclasses. A depend on the next major portage version would transparently handle |
344 | eclasses. A depend on the next major portage version would transparently handle |
| 341 | this for rsync users.</p> |
345 | this for rsync users.</p> |
| 342 | <p>There still is the issue of users who haven't upgraded to the required portage |
346 | <p>There still is the issue of users who haven't upgraded to the required portage |
| 343 | version. This is a minor concern frankly- portage releases include new |
347 | version. This is a minor concern frankly- portage releases include new |
| 344 | functionality, and bug fixes. If they won't upgrade, it's assumed they have |
348 | functionality, and bug fixes. If they won't upgrade, it's assumed they have |
| 345 | their reasons and are big boys, thus able to handle the complications themselves.</p> |
349 | their reasons and are big boys, thus able to handle the complications themselves.</p> |
| 346 | <p>The real issue is broken envs, whether in binpkgs, or for installed packages. |
350 | <p>The real issue is broken envs, whether in binpkgs, or for installed packages. |
| 347 | Two options exist- either the old eclasses are left in the tree indefinitely, or |
351 | Two options exist- either the old eclasses are left in the tree indefinitely, or |
| 348 | they're left for N months, then shifted out of the tree, and into a tarball that |
352 | they're left for N months, then shifted out of the tree, and into a tarball that |
| … | |
… | |
| 377 | the old eclass compat ebuild.</p> |
381 | the old eclass compat ebuild.</p> |
| 378 | <p>Note for this to happen requires either rather... unwise uses of root, or significant |
382 | <p>Note for this to happen requires either rather... unwise uses of root, or significant |
| 379 | fs corruption. Regardless of the cause, it's quite likely for this to even become an |
383 | fs corruption. Regardless of the cause, it's quite likely for this to even become an |
| 380 | issue, the system's vdb is completely unusable. It's a moot issue at that point. |
384 | issue, the system's vdb is completely unusable. It's a moot issue at that point. |
| 381 | If you lose your vdb, or it gets seriously damaged, it's akin to lobotomizing portage- |
385 | If you lose your vdb, or it gets seriously damaged, it's akin to lobotomizing portage- |
| 382 | it doesn't know what's installed, it doesn't know of it's own files, and in general, |
386 | it doesn't know what's installed, it doesn't know of its own files, and in general, |
| 383 | a rebuilding of the system is about the only sane course of action. The missing env is |
387 | a rebuilding of the system is about the only sane course of action. The missing env is |
| 384 | truly the least of the users concern in such a case.</p> |
388 | truly the least of the users concern in such a case.</p> |
| 385 | <p>Continuing with the more likely scenario, users unwilling to upgrade portage will |
389 | <p>Continuing with the more likely scenario, users unwilling to upgrade portage will |
| 386 | <em>not</em> be left out in the rain. Merging the old eclass compat ebuild will provide |
390 | <em>not</em> be left out in the rain. Merging the old eclass compat ebuild will provide |
| 387 | the missing eclasses, thus providing that lost functionality .</p> |
391 | the missing eclasses, thus providing that lost functionality.</p> |
| 388 | <p>Note the intention isn't to force them to upgrade, hence the ability to restore the |
392 | <p>Note the intention isn't to force them to upgrade, hence the ability to restore the |
| 389 | lost functionality. The intention is to clean up the existing mess, and allow us |
393 | lost functionality. The intention is to clean up the existing mess, and allow us |
| 390 | to move forward. The saying "you've got to break a few eggs to make an omelet" |
394 | to move forward. The saying "you've got to break a few eggs to make an omelet" |
| 391 | is akin, exempting the fact we're providing a way to make the eggs whole again |
395 | is akin, exempting the fact we're providing a way to make the eggs whole again |
| 392 | (the king's men would've loved such an option).</p> |
396 | (the king's men would've loved such an option).</p> |
| 393 | </div> |
397 | </div> |
| 394 | <div class="section" id="migrating-to-the-new-setup"> |
398 | <div class="section" id="migrating-to-the-new-setup"> |
| 395 | <h2><a class="toc-backref" href="#id11" name="migrating-to-the-new-setup">Migrating to the new setup</a></h2> |
399 | <h2><a class="toc-backref" href="#id12" name="migrating-to-the-new-setup">Migrating to the new setup</a></h2> |
| 396 | <p>As has been done in the past whenever a change in the tree results in ebuilds |
400 | <p>As has been done in the past whenever a change in the tree results in ebuilds |
| 397 | requiring a specific version of portage, as ebuilds migrate to the new eclasses, |
401 | requiring a specific version of portage, as ebuilds migrate to the new eclasses, |
| 398 | they should depend on a version of portage that supports it. From the users |
402 | they should depend on a version of portage that supports it. From the users |
| 399 | viewpoint, this transparently handles the migration.</p> |
403 | viewpoint, this transparently handles the migration.</p> |
| 400 | <p>This isn't so transparent for devs or a particular infrastructure server however. |
404 | <p>This isn't so transparent for devs or a particular infrastructure server however. |
| … | |
… | |
| 406 | <p>Additionally, prior to any ebuilds in the tree using the new eclasses, the |
410 | <p>Additionally, prior to any ebuilds in the tree using the new eclasses, the |
| 407 | infrastructure server that generates the cache for rsync users will have to |
411 | infrastructure server that generates the cache for rsync users will have to |
| 408 | either be upgraded to a version of portage supporting new eclasses, or patched. |
412 | either be upgraded to a version of portage supporting new eclasses, or patched. |
| 409 | The former being much more preferable then the latter for the portage devs.</p> |
413 | The former being much more preferable then the latter for the portage devs.</p> |
| 410 | <p>Beyond that, an appropriate window for old eclasses to exist in the tree must be |
414 | <p>Beyond that, an appropriate window for old eclasses to exist in the tree must be |
| 411 | determined, and prior to that window passing an ebuild must be added to the tree |
415 | determined, and prior to that window passing, an ebuild must be added to the tree |
| 412 | so users can get the old eclasses if needed.</p> |
416 | so users can get the old eclasses if needed.</p> |
| 413 | <p>For eclass devs to migrate from old to new, it is possible for them to just |
417 | <p>For eclass devs to migrate from old to new, it is possible for them to just |
| 414 | transfer the old eclass into an appropriate grouping in the new eclass directory, |
418 | transfer the old eclass into an appropriate grouping in the new eclass directory, |
| 415 | although it's advisable they cleanse all cruft out of the eclass. You can |
419 | although it's advisable they cleanse all cruft out of the eclass. You can |
| 416 | migrate ebuilds gradually over to the new eclass, and don't have to worry about |
420 | migrate ebuilds gradually over to the new eclass, and don't have to worry about |
| 417 | having to support ebuilds from X years back.</p> |
421 | having to support ebuilds from X years back.</p> |
| 418 | <p>Essentially, you have a chance to nail the design perfectly/cleanly, and have a |
422 | <p>Essentially, you have a chance to nail the design perfectly/cleanly, and have a |
| 419 | window in which to redesign it. It's humbly suggested eclass devs take |
423 | window in which to redesign it. It's humbly suggested eclass devs take |
| 420 | advantage of it. :)</p> |
424 | advantage of it. :)</p> |
| 421 | </div> |
425 | </div> |
| 422 | </div> |
426 | </div> |
| 423 | <div class="section" id="backwards-compatibility"> |
427 | <div class="section" id="backwards-compatibility"> |
| 424 | <h1><a class="toc-backref" href="#id12" name="backwards-compatibility">Backwards Compatibility</a></h1> |
428 | <h1><a class="toc-backref" href="#id13" name="backwards-compatibility">Backwards Compatibility</a></h1> |
| 425 | <p>All backwards compatibility issues are addressed in line, but a recap is offered- |
429 | <p>All backwards compatibility issues are addressed in line, but a recap is offered- |
| 426 | it's suggested that if the a particular compatibility issue is |
430 | it's suggested that if the a particular compatibility issue is |
| 427 | questioned/worried over, the reader read the relevant section. There should be |
431 | questioned/worried over, the reader read the relevant section. There should be |
| 428 | a more in depth discussion of the issue, along with a more extensive explanation |
432 | a more in depth discussion of the issue, along with a more extensive explanation |
| 429 | of the potential solutions, and reasons for the chosen solution.</p> |
433 | of the potential solutions, and reasons for the chosen solution.</p> |
| 430 | <p>To recap:</p> |
434 | <p>To recap:</p> |
| 431 | <pre class="literal-block"> |
435 | <pre class="literal-block"> |
| 432 | New eclasses and elib functionality will be tied to a specific portage |
436 | New eclasses and elib functionality will be tied to a specific portage |
| 433 | version. A DEPENDs on said portage version should address this for rsync |
437 | version. A DEPENDs on said portage version should address this for rsync |
| 434 | users who refuse to upgrade to a portage version that supports the new |
438 | users who refuse to upgrade to a portage version that supports the new |
| 435 | eclasses/elibs and will gradually be unable to merge ebuilds that use said |
439 | eclasses/elibs and will gradually be unable to merge ebuilds that use said |
| 436 | functionality. It is their choice to upgrade, as such, the gradual |
440 | functionality. It is their choice to upgrade, as such, the gradual |
| 437 | 'thinning' of available ebuilds should they block the portage upgrade is |
441 | 'thinning' of available ebuilds should they block the portage upgrade is |
| 438 | their responsibility. |
442 | their responsibility. |
| 439 | |
443 | |
| 440 | Old eclasses at some point in the future should be removed from the tree, |
444 | Old eclasses at some point in the future should be removed from the tree, |
| 441 | and released in a tarball/ebuild. This will cause installed ebuilds that |
445 | and released in a tarball/ebuild. This will cause installed ebuilds that |
| 442 | rely on the old eclass to be unable to unmerge, with the same applying for |
446 | rely on the old eclass to be unable to unmerge, with the same applying for |
| 443 | merging of binpkgs dependent on the following paragraph. |
447 | merging of binpkgs dependent on the following paragraph. |
| 444 | |
448 | |
| 445 | The old eclass-compat is only required for users who do not upgrade their |
449 | The old eclass-compat is only required for users who do not upgrade their |
| 446 | portage installation, and one further exemption- if the user has somehow |
450 | portage installation, and one further exemption- if the user has somehow |
| … | |
… | |
| 451 | strictly to be complete, it's not much of a real world scenario as elaborated |
455 | strictly to be complete, it's not much of a real world scenario as elaborated |
| 452 | above. |
456 | above. |
| 453 | </pre> |
457 | </pre> |
| 454 | </div> |
458 | </div> |
| 455 | <div class="section" id="copyright"> |
459 | <div class="section" id="copyright"> |
| 456 | <h1><a class="toc-backref" href="#id13" name="copyright">Copyright</a></h1> |
460 | <h1><a class="toc-backref" href="#id14" name="copyright">Copyright</a></h1> |
| 457 | <p>This document has been placed in the public domain.</p> |
461 | <p>This document has been placed in the public domain.</p> |
| 458 | </div> |
462 | </div> |
| 459 | </div> |
|
|
| 460 | |
463 | |
|
|
464 | </div> |
|
|
465 | <div class="footer"> |
| 461 | <hr class="footer" /> |
466 | <hr class="footer" /> |
| 462 | <div class="footer"> |
|
|
| 463 | <a class="reference" href="glep-0033.txt">View document source</a>. |
467 | <a class="reference" href="glep-0033.txt">View document source</a>. |
| 464 | Generated on: 2005-03-06 20:38 UTC. |
468 | Generated on: 2005-09-15 21:03 UTC. |
| 465 | Generated by <a class="reference" href="http://docutils.sourceforge.net/">Docutils</a> from <a class="reference" href="http://docutils.sourceforge.net/rst.html">reStructuredText</a> source. |
469 | Generated by <a class="reference" href="http://docutils.sourceforge.net/">Docutils</a> from <a class="reference" href="http://docutils.sourceforge.net/rst.html">reStructuredText</a> source. |
|
|
470 | |
| 466 | </div> |
471 | </div> |
| 467 | </body> |
472 | </body> |
| 468 | </html> |
473 | </html> |
| 469 | |
474 | |