1 |
This looks good overall, thanks. |
2 |
|
3 |
If we stay with the whitespace separated columns, the spec should be |
4 |
clear that implementations should be able to deal with future |
5 |
additional "columns" in their parsing code. |
6 |
|
7 |
Below some paint choices from me. |
8 |
|
9 |
> We introduce a new file "arches.desc" which essentially describes if |
10 |
> an arch |
11 |
> (not a profile) is stable or not. The meaning of profiles.desc is not |
12 |
> affected; |
13 |
|
14 |
Essentially the proposal extends profiles/arch.list but due to |
15 |
backwards compatibility can't just add details there. |
16 |
As such, in my opinion the file should be called arch.desc (not plural |
17 |
arches.desc) to go along with that. |
18 |
|
19 |
> 1] File location: |
20 |
> profiles/arches.desc or metadata/arches.desc |
21 |
|
22 |
profiles/arch.desc or metadata/repoman/arch.desc |
23 |
|
24 |
> 3] Meaning of the three values "stable", "testing", "unstable" for |
25 |
> repoman |
26 |
> |
27 |
> * stable: When a profile of arch is tested, then repoman checks |
28 |
> consistency for |
29 |
> "arch" and for "~arch" separately. |
30 |
> Which profiles of the arch are tested is still controlled by |
31 |
> profiles.desc (and |
32 |
> -d / -e switches). |
33 |
> This is the current behaviour and should be the default if nothing is |
34 |
> specified |
35 |
> for an arch. |
36 |
> |
37 |
> * testing: When a profile of arch is tested, then repoman treats |
38 |
> "arch" as |
39 |
> "~arch", and tests consistency only for "~arch". |
40 |
> Which profiles of the arch are tested is still controlled by |
41 |
> profiles.desc (and |
42 |
> -d / -e switches). |
43 |
> A new switch for repoman may be provided to temporarily upgrade an |
44 |
> arch from |
45 |
> "testing" to "stable" (for arch team work). |
46 |
> |
47 |
> * unstable: When a profile of arch is tested, then repoman treats |
48 |
> "arch" as an |
49 |
> error and aborts. Consistency is only tested for "~arch". |
50 |
> Which profiles of the arch are tested is still controlled by |
51 |
> profiles.desc (and |
52 |
> -d / -e switches). |
53 |
|
54 |
This sounds more like "testing" to me - architecture is only meant to |
55 |
have "testing" keywords, which is what I tend to call ~arch because |
56 |
it's in testing to become "stable" in ~30days or so, instead of calling |
57 |
it "unstable" (which feels appropriate only for a package that doesn't |
58 |
carry any stable keywords in older versions either). |
59 |
While taken from another perspective, the meaning for "testing" as in |
60 |
this proposal makes sense too - treat all as "testing" keywords. |
61 |
This goes back to the overloaded terminology concerns that have been |
62 |
echoed by others as well. |
63 |
But I don't have any good suggestions for alternatives either right |
64 |
now. stable/no_stable_check/testing_only? shrug. |
65 |
|
66 |
> 4] Meaning for other tools |
67 |
> All arches set to "stable" are considered "stable arches", meaning |
68 |
> * they get listed first in eshowkw |
69 |
> * they require stabilization requests in bugzilla |
70 |
> * ... |
71 |
|
72 |
If other tools use this, then maybe the repoman specific |
73 |
metadata/repoman/ path isn't appropriate afterall. So then |
74 |
profiles/arch.desc or metadata/arch.desc |
75 |
|
76 |
|
77 |
Mart |