| 1 |
g2boojum |
1.1 |
<?xml version="1.0" encoding="utf-8" ?> |
| 2 |
|
|
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd"> |
| 3 |
|
|
<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en"> |
| 4 |
|
|
<!-- |
| 5 |
|
|
This HTML is auto-generated. DO NOT EDIT THIS FILE! If you are writing a new |
| 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! |
| 8 |
|
|
--> |
| 9 |
|
|
<head> |
| 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/" /> |
| 12 |
|
|
<title>GLEP 35 -- Automated consistency check for ebuilds</title> |
| 13 |
|
|
<link rel="stylesheet" href="tools/glep.css" type="text/css" /> |
| 14 |
|
|
</head> |
| 15 |
|
|
<body bgcolor="white"> |
| 16 |
|
|
<table class="navigation" cellpadding="0" cellspacing="0" |
| 17 |
|
|
width="100%" border="0"> |
| 18 |
|
|
<tr><td class="navicon" width="150" height="35"> |
| 19 |
|
|
<a href="http://www.gentoo.org/" title="Gentoo Linux Home Page"> |
| 20 |
|
|
<img src="http://www.gentoo.org/images/gentoo-new.gif" alt="[Gentoo]" |
| 21 |
|
|
border="0" width="150" height="35" /></a></td> |
| 22 |
|
|
<td class="textlinks" align="left"> |
| 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>] |
| 25 |
|
|
[<b><a href="./glep-0035.txt">GLEP Source</a></b>] |
| 26 |
|
|
</td></tr></table> |
| 27 |
|
|
<div class="document"> |
| 28 |
|
|
<table class="rfc2822 field-list" frame="void" rules="none"> |
| 29 |
|
|
<col class="field-name" /> |
| 30 |
|
|
<col class="field-body" /> |
| 31 |
|
|
<tbody valign="top"> |
| 32 |
|
|
<tr class="field"><th class="field-name">GLEP:</th><td class="field-body">35</td> |
| 33 |
|
|
</tr> |
| 34 |
|
|
<tr class="field"><th class="field-name">Title:</th><td class="field-body">Automated consistency check for ebuilds</td> |
| 35 |
|
|
</tr> |
| 36 |
|
|
<tr class="field"><th class="field-name">Version:</th><td class="field-body">1.1</td> |
| 37 |
|
|
</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-0035.txt?cvsroot=gentoo">2005/03/12 20:26:01</a></td> |
| 39 |
|
|
</tr> |
| 40 |
|
|
<tr class="field"><th class="field-name">Author:</th><td class="field-body">Adrian Lambeck <adrian at basicsedv.de>,</td> |
| 41 |
|
|
</tr> |
| 42 |
|
|
<tr class="field"><th class="field-name">Status:</th><td class="field-body">Draft</td> |
| 43 |
|
|
</tr> |
| 44 |
|
|
<tr class="field"><th class="field-name">Type:</th><td class="field-body">Standards Track</td> |
| 45 |
|
|
</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> |
| 47 |
|
|
</tr> |
| 48 |
|
|
<tr class="field"><th class="field-name">Created:</th><td class="field-body">12-Mar-2005</td> |
| 49 |
|
|
</tr> |
| 50 |
|
|
<tr class="field"><th class="field-name">Post-History:</th><td class="field-body">12-Mar-2005</td> |
| 51 |
|
|
</tr> |
| 52 |
|
|
</tbody> |
| 53 |
|
|
</table> |
| 54 |
|
|
<hr /> |
| 55 |
|
|
<div class="contents topic" id="contents"> |
| 56 |
|
|
<p class="topic-title first"><a name="contents">Contents</a></p> |
| 57 |
|
|
<ul class="simple"> |
| 58 |
|
|
<li><a class="reference" href="#abstract" id="id5" name="id5">Abstract</a></li> |
| 59 |
|
|
<li><a class="reference" href="#motivation" id="id6" name="id6">Motivation</a></li> |
| 60 |
|
|
<li><a class="reference" href="#specification" id="id7" name="id7">Specification</a></li> |
| 61 |
|
|
<li><a class="reference" href="#implementation" id="id8" name="id8">Implementation</a></li> |
| 62 |
|
|
<li><a class="reference" href="#backwards-compatibility" id="id9" name="id9">Backwards Compatibility</a></li> |
| 63 |
|
|
<li><a class="reference" href="#id1" id="id10" name="id10">References</a></li> |
| 64 |
|
|
</ul> |
| 65 |
|
|
</div> |
| 66 |
|
|
<div class="section" id="abstract"> |
| 67 |
|
|
<h1><a class="toc-backref" href="#id5" name="abstract">Abstract</a></h1> |
| 68 |
|
|
<p>This proposal is meant to enhance productivity for Gentoo developers. |
| 69 |
|
|
It aims to reduce the number of trivial bugs by automatically detecting them |
| 70 |
|
|
through a consistency check that is performed before checking and on a regular |
| 71 |
|
|
basis through the whole tree. |
| 72 |
|
|
Why bother with trivial bugs when automated tests find them ? |
| 73 |
|
|
Save time and improve quality !</p> |
| 74 |
|
|
</div> |
| 75 |
|
|
<div class="section" id="motivation"> |
| 76 |
|
|
<h1><a class="toc-backref" href="#id6" name="motivation">Motivation</a></h1> |
| 77 |
|
|
<p>When browsing <a class="reference" href="http://bugs.gentoo.org">bugs.gentoo.org</a> <a class="footnote-reference" href="#id2" id="id3" name="id3">[1]</a> you will find Bugs that take away a good |
| 78 |
|
|
amount of scarce developing time that could be used otherwise. These are |
| 79 |
|
|
trivial bugs, i.e. wrong SRC_URI or cycles in DEPEND. Even worst - these bugs |
| 80 |
|
|
are sometimes reported several times so that they need to be marked as |
| 81 |
|
|
dublicates. Bugs of that kind are easy to find and easy to fix. By using |
| 82 |
|
|
automatic checks on a regular schedule these bugs can be found. Users have to |
| 83 |
|
|
be asked NOT to commit these bugs to <a class="reference" href="http://bugs.gentoo.org">bugs.gentoo.org</a> <a class="footnote-reference" href="#id2" id="id4" name="id4">[1]</a>. So there will |
| 84 |
|
|
(hopefully) be fewer bugs that need to be checked and assigned and they might |
| 85 |
|
|
get fixed faster.</p> |
| 86 |
|
|
<p>The Bugs found should be kept in an automatically generated list so that users |
| 87 |
|
|
can see that the problem has been caught and that it is being worked on.</p> |
| 88 |
|
|
</div> |
| 89 |
|
|
<div class="section" id="specification"> |
| 90 |
|
|
<h1><a class="toc-backref" href="#id7" name="specification">Specification</a></h1> |
| 91 |
|
|
<p>Checks need to be performed for every ebuild.</p> |
| 92 |
|
|
<p>A report needs to be generated</p> |
| 93 |
|
|
<blockquote> |
| 94 |
|
|
<ul class="simple"> |
| 95 |
|
|
<li>links to the specific problem need to be included</li> |
| 96 |
|
|
<li>reports need to be send to the groups responsible</li> |
| 97 |
|
|
</ul> |
| 98 |
|
|
</blockquote> |
| 99 |
|
|
<p>Checks could be:</p> |
| 100 |
|
|
<blockquote> |
| 101 |
|
|
<ul class="simple"> |
| 102 |
|
|
<li>cycles within DEPEND</li> |
| 103 |
|
|
<li>invalid SRC_URI</li> |
| 104 |
|
|
<li>"non-official" USE Flags</li> |
| 105 |
|
|
<li>Packages within DEPEND that are "*" for the arch specified</li> |
| 106 |
|
|
<li>broken shell scripts with invalid or missing commands</li> |
| 107 |
|
|
<li>inheritance of eclasses</li> |
| 108 |
|
|
<li>...</li> |
| 109 |
|
|
</ul> |
| 110 |
|
|
</blockquote> |
| 111 |
|
|
<p>There might be other checks and tests that should be run |
| 112 |
|
|
that have not come to my mind yet. Also I might have suggested something that |
| 113 |
|
|
is not useful at all.</p> |
| 114 |
|
|
<p>If there are major problems (needs to be defined) within an ebuild a possible |
| 115 |
|
|
action could be to disable the ebuild (with <tt class="literal"><span class="pre">"-*"</span></tt>,) perhaps, and send a |
| 116 |
|
|
mail to the maintainer.</p> |
| 117 |
|
|
<p>These kind of errors are not always the fault of the developers.</p> |
| 118 |
|
|
<p>There should be no compilation or something like that. If an ebuild fails to |
| 119 |
|
|
build somewhere then the user should file it as a bug as usual.</p> |
| 120 |
|
|
</div> |
| 121 |
|
|
<div class="section" id="implementation"> |
| 122 |
|
|
<h1><a class="toc-backref" href="#id8" name="implementation">Implementation</a></h1> |
| 123 |
|
|
<p>The functionality described could be implemented in three ways:</p> |
| 124 |
|
|
<blockquote> |
| 125 |
|
|
<ol class="arabic"> |
| 126 |
|
|
<li><dl class="first"> |
| 127 |
|
|
<dt>On the developers machine ("client") where it is run before checking</dt> |
| 128 |
|
|
<dd><p class="first last">only for the ebuilds that changed. (client does not fit here because |
| 129 |
|
|
the server and client should not communicate with each other at all)</p> |
| 130 |
|
|
</dd> |
| 131 |
|
|
</dl> |
| 132 |
|
|
</li> |
| 133 |
|
|
<li><p class="first">On the server where checks are run, i.e. once a week.</p> |
| 134 |
|
|
</li> |
| 135 |
|
|
<li><p class="first">On the "client" AND server</p> |
| 136 |
|
|
</li> |
| 137 |
|
|
</ol> |
| 138 |
|
|
<p>Of course there are cons and pros (what came to my mind so far)</p> |
| 139 |
|
|
<ol class="arabic"> |
| 140 |
|
|
<li><blockquote class="first"> |
| 141 |
|
|
<dl> |
| 142 |
|
|
<dt>pro: </dt> |
| 143 |
|
|
<dd><ul class="first last simple"> |
| 144 |
|
|
<li>the tree can not become inconsistent in the first place (? see contra)</li> |
| 145 |
|
|
<li>once an ebuild is checked there is no need to do this again</li> |
| 146 |
|
|
<li>no dedicated machine necessary</li> |
| 147 |
|
|
<li>generate traffic only once on one machine</li> |
| 148 |
|
|
<li>errors that are caught here do not bother later on</li> |
| 149 |
|
|
</ul> |
| 150 |
|
|
</dd> |
| 151 |
|
|
<dt>contra:</dt> |
| 152 |
|
|
<dd><ul class="first last"> |
| 153 |
|
|
<li><dl class="first"> |
| 154 |
|
|
<dt>the consistency is based on the tool installed </dt> |
| 155 |
|
|
<dd><p class="first last">(what happens when different devs use different versions ?)</p> |
| 156 |
|
|
</dd> |
| 157 |
|
|
</dl> |
| 158 |
|
|
</li> |
| 159 |
|
|
<li><dl class="first"> |
| 160 |
|
|
<dt>what happens when the ebuild layout changes and some ebuilds</dt> |
| 161 |
|
|
<dd><p class="first last">do not get updated ?</p> |
| 162 |
|
|
</dd> |
| 163 |
|
|
</dl> |
| 164 |
|
|
</li> |
| 165 |
|
|
</ul> |
| 166 |
|
|
</dd> |
| 167 |
|
|
</dl> |
| 168 |
|
|
</blockquote> |
| 169 |
|
|
</li> |
| 170 |
|
|
<li><blockquote class="first"> |
| 171 |
|
|
<dl> |
| 172 |
|
|
<dt>pro:</dt> |
| 173 |
|
|
<dd><ul class="first last simple"> |
| 174 |
|
|
<li>Properties of other ebuilds might change that fit while writing an ebuild</li> |
| 175 |
|
|
</ul> |
| 176 |
|
|
</dd> |
| 177 |
|
|
<dt>contra:</dt> |
| 178 |
|
|
<dd><ul class="first last"> |
| 179 |
|
|
<li><p class="first">the errors are found when the ebuild is already in CVS</p> |
| 180 |
|
|
</li> |
| 181 |
|
|
<li><p class="first">the whole tree needs to be checked</p> |
| 182 |
|
|
</li> |
| 183 |
|
|
<li><dl class="first"> |
| 184 |
|
|
<dt>possibly creates a lot of traffic on every run</dt> |
| 185 |
|
|
<dd><p class="first last">(-> is there an FTP equivalent to HTTP`s HEAD ?)</p> |
| 186 |
|
|
</dd> |
| 187 |
|
|
</dl> |
| 188 |
|
|
</li> |
| 189 |
|
|
</ul> |
| 190 |
|
|
</dd> |
| 191 |
|
|
</dl> |
| 192 |
|
|
</blockquote> |
| 193 |
|
|
</li> |
| 194 |
|
|
<li><p class="first">see 1. and 2.</p> |
| 195 |
|
|
</li> |
| 196 |
|
|
</ol> |
| 197 |
|
|
</blockquote> |
| 198 |
|
|
<p>My favorite is 3 . All properties are checked before check-in and |
| 199 |
|
|
the properties that change might be checked on a regular basis on the server. |
| 200 |
|
|
Only solution 3 brings the best from 1 and 2 together while delivering the best result.</p> |
| 201 |
|
|
<p>I never had a look at portage source but I can imagine that there is a library |
| 202 |
|
|
that has everything that a developer needs to "query" ebuilds. If not, this |
| 203 |
|
|
would be a reason for another GLEP (?).</p> |
| 204 |
|
|
<p>For performance I would use a database (on the server) to store the whole tree before |
| 205 |
|
|
running the checks. This is not necessary for the "client".</p> |
| 206 |
|
|
</div> |
| 207 |
|
|
<div class="section" id="backwards-compatibility"> |
| 208 |
|
|
<h1><a class="toc-backref" href="#id9" name="backwards-compatibility">Backwards Compatibility</a></h1> |
| 209 |
|
|
<p>Not a problem for this GLEP.</p> |
| 210 |
|
|
</div> |
| 211 |
|
|
<div class="section" id="id1"> |
| 212 |
|
|
<h1><a class="toc-backref" href="#id10" name="id1">References</a></h1> |
| 213 |
|
|
<table class="footnote" frame="void" id="id2" rules="none"> |
| 214 |
|
|
<colgroup><col class="label" /><col /></colgroup> |
| 215 |
|
|
<tbody valign="top"> |
| 216 |
|
|
<tr><td class="label"><a name="id2">[1]</a></td><td><em>(<a class="fn-backref" href="#id3">1</a>, <a class="fn-backref" href="#id4">2</a>)</em> <a class="reference" href="http://bugs.gentoo.org">http://bugs.gentoo.org</a></td></tr> |
| 217 |
|
|
</tbody> |
| 218 |
|
|
</table> |
| 219 |
|
|
</div> |
| 220 |
|
|
</div> |
| 221 |
|
|
|
| 222 |
|
|
<hr class="footer" /> |
| 223 |
|
|
<div class="footer"> |
| 224 |
|
|
<a class="reference" href="glep-0035.txt">View document source</a>. |
| 225 |
|
|
Generated on: 2005-03-12 20:26 UTC. |
| 226 |
|
|
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. |
| 227 |
|
|
</div> |
| 228 |
|
|
</body> |
| 229 |
|
|
</html> |