1 |
Doug Goldstein <cardoe@g.o>: |
2 |
|
3 |
> Jonathan Adamczewski wrote: |
4 |
> > Doug Goldstein wrote: |
5 |
> >> That's what this commits review list feels like. |
6 |
> > Nearly every suggestion (from Donnie and others) has been over some |
7 |
> > issue that relates directly to either correctness or |
8 |
> > maintainability. It doesn't matter if you can "rattle off |
9 |
> > capabilities to a millimeter" - if they're not documented somewhere |
10 |
> > (like, say, in the comments of the ebuild) then the maintainer that |
11 |
> > comes after you gets to go and break it all over again. |
12 |
> Correctness? Fine. Go ahead. Stick $(use_enable xvmc) into the ebuild. |
13 |
> Do it. I dare you. Then try to compile. |
14 |
> Guess what? When it blows up... that's called INcorrect. The opposite |
15 |
> of the right thing. |
16 |
|
17 |
You were kindly asked if is not possible to use, so why do you feel |
18 |
attacked? Do a comment on it and everybody would be fine, even the |
19 |
people that would have to maintain it some time in the future. If you |
20 |
don't like the review process, just ignore it. |
21 |
Reviews are not a way to show what kind of idiot the committer is, but |
22 |
to improve the overall quality of the tree. Nothing more, nothing less. |
23 |
|
24 |
> The maintainer who comes after me would be someone with a experience |
25 |
> with the package. Some bumkin isn't going to come to maintain package |
26 |
> XYZ unless they know or use the package, and guess what? That means |
27 |
> experience. |
28 |
|
29 |
Yes, and the same goes for GNU Emacs, I needed some time to figure out |
30 |
what all those things did and I broke it several times because I tried |
31 |
to be clever. Now we documented it and I think everyone coming after us |
32 |
will have a less hard time to understand it. Better document it, you |
33 |
never know what happens. |
34 |
|
35 |
V-Li |
36 |
|
37 |
-- |
38 |
Christian Faulhammer, Gentoo Lisp project |
39 |
<URL:http://www.gentoo.org/proj/en/lisp/>, #gentoo-lisp on FreeNode |
40 |
|
41 |
<URL:http://www.faulhammer.org/> |