Gentoo Archives: gentoo-dev

From: Tom Wijsman <TomWij@g.o>
To: gentoo-dev@l.g.o
Cc: phajdan.jr@g.o
Subject: Re: [gentoo-dev] RFC: Removing src_test from www-client/chromium
Date: Sun, 01 Jun 2014 14:42:01
Message-Id: 20140601164106.0fc446c2@gentoo.org
In Reply to: Re: [gentoo-dev] RFC: Removing src_test from www-client/chromium by "Paweł Hajdan
1 On Sun, 01 Jun 2014 15:41:35 +0200
2 ""Paweł Hajdan, Jr."" <phajdan.jr@g.o> wrote:
3
4 > On 5/31/14, 8:30 PM, Tom Wijsman wrote:
5 > > On Sat, 31 May 2014 19:50:20 +0200
6 > > ""Paweł Hajdan, Jr."" <phajdan.jr@g.o> wrote:
7 > >> This is one of my points: I don't remember a single chromium bug
8 > >> filed in Gentoo that would be caught by a test or that a failing
9 > >> test actually detected.
10 > >
11 > > Your point covers the lack of tests, or tests that are non-fatal;
12 > > however, it doesn't cover tests that are fatal, what if they fail?
13 >
14 > I'm confused by the distinction of fatal and non-fatal tests. Neither
15 > upstream nor the Gentoo chromium package makes that distinction.
16
17 Tests that break parts of your browser; you don't notice such tests in
18 what was said, until one day such a test does break one or another way.
19
20 > >> By the way, I don't remember seeing many reports about font issues
21 > >> or tab crashes. Please make sure to file them when they occur, or
22 > >> just point me to them in case I somehow missed them.
23 > >
24 > > They usually go straight to upstream, though I've managed to somehow
25 > > fix it up; as for Gentoo, some people create forum threads about
26 > > them.
27 >
28 > I can't speak for other people, but please consider reporting issues
29 > to Gentoo first. Our bug queue is under 30 bugs, while upstream is
30 > several thousand. Once we can confirm a bug clearly belongs to
31 > upstream, we can tell the reporter to file bug upstream or do that
32 > ourselves, but keeping Gentoo out of the loop seems to increase the
33 > time needed to fix a bug.
34
35 This confuses me; your thread opener seems to suggest you have too much
36 bugs, whereas this one seems to suggest you don't have enough bugs.
37
38 What's the purpose of disabling src_test if bug count isn't a problem?
39 Iotw, why are you making a project-internal decision here?
40
41 (Your last two paragraphs may respond to this; in which case, nvm)
42
43 > > (One was due to a library compiled with a less common flag, the
44 > > other due to fontconfig being a regression magnet; both fun to
45 > > debug, the former a test wolud've caught, the latter is due to the
46 > > lack thereof)
47 >
48 > If there's something that could be changed e.g. in chromium's
49 > dependencies, please let me know. There are cases where we require
50 > certain USE flags to be set on dependencies for things to work
51 > properly.
52
53 Wasn't the case of a different version or USE flag state; but that is
54 the case one or another day, I'll let you know.
55
56 > About the issue that a test would have caught: was that a chromium
57 > test? If so, which one?
58
59 Unable to tell, since I don't run them.
60
61 > >>> While I don't run tests myself; the need for them is clear, for
62 > >>> those that aim for more production ready systems (eg. university
63 > >>> network PCs).
64 > >>
65 > >> This seems too theoretical to me. I'd be fine with someone
66 > >> volunteering to maintain chromium's src_test in Gentoo. Unless we
67 > >> have such a person though, it seems to mostly take valuable focus
68 > >> away from bugs that definitely *do* affect our users, for no
69 > >> provable benefit for Gentoo.
70 > >
71 > > What about provable benefit for upstream? Does upstream /dev/null
72 > > them?
73 >
74 > Effectively yes. For an example see
75 > https://bugs.gentoo.org/show_bug.cgi?id=497512 and
76 > https://groups.google.com/a/chromium.org/d/msg/chromium-dev/OdX7ShsOqsM/-R9sexJAEa4J
77 >
78 > The failure is not Gentoo-specific, and is not a bug in code but
79 > problem with the test (it makes assumptions about internal glibc
80 > implementation). It actually fails on the latest Ubuntu LTS Trusty
81 > Tahr, which means the test will have to be fixed or disabled
82 > upstream. But 6 months of no reaction is not really a good sign IMHO.
83 >
84 > Paweł
85
86 Yeah; if failing tests on distributions aren't getting fixed by
87 upstream, then there's indeed no point to keep them running.
88
89 Though; on the other hand, one has to consider that this acts like a
90 priority queue and therefore tests that fail on most distributions would
91 get fixed before tests that fail on just one or two distributions.
92
93 It's a tricky decision to drop them; but it's not an irreversible
94 decision, thus a reevaluation in 5 years from now could be possible. If
95 that reevaluation then shows a responsive upstream, reconsider src_test.
96
97 Don't mind me, I've played devil's advocate to explore the reasoning;
98 go ahead if you want to, given it barely result in fatal test failures.
99
100 --
101 With kind regards,
102
103 Tom Wijsman (TomWij)
104 Gentoo Developer
105
106 E-mail address : TomWij@g.o
107 GPG Public Key : 6D34E57D
108 GPG Fingerprint : C165 AF18 AB4C 400B C3D2 ABF0 95B2 1FCD 6D34 E57D

Attachments

File name MIME type
signature.asc application/pgp-signature

Replies