1 |
lör 2001-11-10 klockan 16.56 skrev Karl Trygve Kalleberg: |
2 |
> On 10 Nov 2001 12:34:15 +0100 |
3 |
> Mikael Hallendal <hallski@g.o> wrote: |
4 |
> |
5 |
> > Hi! |
6 |
> > |
7 |
> > A while back drobbins mailed about the issue that A= shouldn't be set by |
8 |
> > the ebuild-editor. It's set automatically by Portage. He also said that |
9 |
> > it should be removed. I'm surprised to see that some developers still |
10 |
> > updates ebuilds and leaves 'A=' there. |
11 |
> > |
12 |
> > Same thing about try->die. If you edit/update an ebuild, please fix this |
13 |
> > kinds of issues before commiting it. |
14 |
> > |
15 |
> > I'm currently fixing lots of ebuilds but it's hard work. |
16 |
> |
17 |
> The ebuildchecker posted earlier will tell you which ebuilds are affected |
18 |
> by tab bugs, A= settings and the use of try, among other things. |
19 |
> |
20 |
> I have been thinking of writing a cleanup script that fixes most of these |
21 |
> things automatically. It would mean that we developers would only have to |
22 |
> look over the ebuilds instead of actually doing the rewrites. However, it |
23 |
> might turn out to be a complete mess, so unless we think this is a Big |
24 |
> Issue, I'd rather spend my time elsewhere. |
25 |
|
26 |
The A=-issue is pretty straight forward, just grep for A= (same for |
27 |
try). |
28 |
|
29 |
I think the time you have to spend on writing some script that would do |
30 |
this automatic would be to great to be meaningful. Also, when going |
31 |
through all ebuilds developers can fix other things, such as style-fixes |
32 |
and such. |
33 |
|
34 |
We should write some spec. on how ebuilds should look like (guidelines |
35 |
for style) and then make a emacs and a vi mode for editing ebuilds that |
36 |
controls the style. |
37 |
|
38 |
Currently ebuilds looks very different from each other confusing people. |
39 |
|
40 |
Regards, |
41 |
Mikael Hallendal |
42 |
|
43 |
-- |
44 |
|
45 |
Mikael Hallendal |
46 |
Gentoo Linux Developer, Desktop Team Leader |
47 |
CodeFactory AB, Stockholm, Sweden |