1 |
On Wed, 15 Jan 2014 18:29:20 +0100 |
2 |
Sebastian Luther <SebastianLuther@×××.de> wrote: |
3 |
|
4 |
> Am 15.01.2014 17:20, schrieb Tom Wijsman: |
5 |
> > On Wed, 15 Jan 2014 07:29:19 +0100 |
6 |
> > Sebastian Luther <SebastianLuther@×××.de> wrote: |
7 |
> > |
8 |
> >> Am 15.01.2014 04:11, schrieb Tom Wijsman: |
9 |
> >> |
10 |
> >> |
11 |
> >> I send the first mail with this wording 8 days ago. Enough time to |
12 |
> >> comment on it. I'd prefer to discuss it on the list. |
13 |
> > |
14 |
> > Yes, but not all comments were discussed yet, therefore |
15 |
> > (dis)agreement on them is missing; and this last thing rather |
16 |
> > became a topic of discussion due to the work clashes that we saw |
17 |
> > happen twice. |
18 |
> > |
19 |
> I'd say the clashes occurred because nobody mentioned at all what they |
20 |
> are working on. Since people started using IN_PROGRESS to mean "I'm |
21 |
> working on it", this shouldn't happen again. |
22 |
|
23 |
Neither should it happen again if bugs get self-assigned; the point |
24 |
here is that neither of both is discussed, agreed on or documented |
25 |
properly. Therefore the clashes need further revising of the workflow. |
26 |
|
27 |
> > Yes, I see some commit messages not refer to bugs which is |
28 |
> > something we will want to avoid; think this might need to go into |
29 |
> > the commit policy. |
30 |
> > |
31 |
> There's nothing wrong with fixing/implementing something that nobody |
32 |
> filed a bug about. |
33 |
|
34 |
Sorry, consider the common case where a bug was filed but the commit |
35 |
message does not refer to that bug. Also note that I am trying to refer |
36 |
to the ChangeLog of Portage itself, not that of the ebuild; thus I |
37 |
mean the commit messages for the Portage repo, not to the Portage tree. |
38 |
|
39 |
> >> |
40 |
> >> The "way it was" is to not care about them at all. There was no |
41 |
> >> agreement on the the other thread if these things should be used or |
42 |
> >> not. So I left it vague so everyone could use it, but they are not |
43 |
> >> forced to. |
44 |
> > |
45 |
> > Hmm, could this result in conflicting usage of these? |
46 |
> |
47 |
> Maybe, but I'd first see if the usage patterns converge to something |
48 |
> that makes everyone happy. |
49 |
|
50 |
The whole point of documenting it in a workflow is to make it converge; |
51 |
if you instead leave things unclear or undocumented, you have no |
52 |
guaranteed convergence and might even see a disconvergence. |
53 |
|
54 |
It's already making people unhappy right now; because as it is |
55 |
documented now, it is turned from the meaningful experience that the |
56 |
previous Portage team had before to something that is meaningless. It |
57 |
is a regression in checking the list of bugs that block the tracker, as |
58 |
the states of the bugs no longer have a value as it is documented now. |
59 |
|
60 |
If we change something about this, there should be a good reason to... |
61 |
|
62 |
> >>>> +There are a number of bugs named "[TRACKER] *" that collect bugs |
63 |
> >>>> +for specific topics. Confirmed bugs should be marked as blocking |
64 |
> >>>> +these tracker bugs if appropriate. |
65 |
> >>> |
66 |
> >>> For clarity, it should be mentioned that this does not mean to |
67 |
> >>> block the tracker for the next version; this could be |
68 |
> >>> misinterpreted. |
69 |
> >> |
70 |
> >> Considering that the tracker gets renamed, I'm not sure what you |
71 |
> >> mean here. |
72 |
> > |
73 |
> > As you are confused yourself by misinterpreting what you have |
74 |
> > written, you demonstrate the case for the need of clarity here; |
75 |
> > this is not about the next version tracker or it being renamed at |
76 |
> > all, it's about all other trackers that are not version trackers. |
77 |
> > The part of the policy quoted here doesn't make that clear, it had |
78 |
> > me puzzling for a moment too when I first read that; I think you |
79 |
> > were puzzled too now... |
80 |
> > |
81 |
> |
82 |
> Sorry, I failed to properly read what you quoted. |
83 |
> |
84 |
> I think once you know that these other trackers exist, it's clear. If |
85 |
> you want something added there, that's fine with me too. |
86 |
|
87 |
We could add the sentence that I wrote earlier in the quote |
88 |
|
89 |
This does not mean to block the tracker for the next version. |
90 |
|
91 |
but to avoid misreading we could write it up more positive and clear as |
92 |
|
93 |
These are other trackers than the next version tracker. |
94 |
|
95 |
Optionally adding the redundant words "Note that" in front. |
96 |
|
97 |
-- |
98 |
With kind regards, |
99 |
|
100 |
Tom Wijsman (TomWij) |
101 |
Gentoo Developer |
102 |
|
103 |
E-mail address : TomWij@g.o |
104 |
GPG Public Key : 6D34E57D |
105 |
GPG Fingerprint : C165 AF18 AB4C 400B C3D2 ABF0 95B2 1FCD 6D34 E57D |