public inbox for gentoo-dev@lists.gentoo.org
 help / color / mirror / Atom feed
From: "\"Paweł Hajdan, Jr.\"" <phajdan.jr@gentoo.org>
To: gentoo-dev@lists.gentoo.org
Subject: Re: [gentoo-dev] RFC: Removing src_test from www-client/chromium
Date: Sun, 01 Jun 2014 15:41:35 +0200	[thread overview]
Message-ID: <538B2D8F.6030200@gentoo.org> (raw)
In-Reply-To: <20140531203058.49aa2332@gentoo.org>

[-- Attachment #1: Type: text/plain, Size: 2892 bytes --]

On 5/31/14, 8:30 PM, Tom Wijsman wrote:
> On Sat, 31 May 2014 19:50:20 +0200
> ""Paweł Hajdan, Jr."" <phajdan.jr@gentoo.org> wrote:
>> This is one of my points: I don't remember a single chromium bug filed
>> in Gentoo that would be caught by a test or that a failing test
>> actually detected.
> 
> Your point covers the lack of tests, or tests that are non-fatal;
> however, it doesn't cover tests that are fatal, what if they fail?

I'm confused by the distinction of fatal and non-fatal tests. Neither
upstream nor the Gentoo chromium package makes that distinction.

>> By the way, I don't remember seeing many reports about font issues or
>> tab crashes. Please make sure to file them when they occur, or just
>> point me to them in case I somehow missed them.
> 
> They usually go straight to upstream, though I've managed to somehow
> fix it up; as for Gentoo, some people create forum threads about them.

I can't speak for other people, but please consider reporting issues to
Gentoo first. Our bug queue is under 30 bugs, while upstream is several
thousand. Once we can confirm a bug clearly belongs to upstream, we can
tell the reporter to file bug upstream or do that ourselves, but keeping
Gentoo out of the loop seems to increase the time needed to fix a bug.

> (One was due to a library compiled with a less common flag, the other
> due to fontconfig being a regression magnet; both fun to debug, the
> former a test wolud've caught, the latter is due to the lack thereof)

If there's something that could be changed e.g. in chromium's
dependencies, please let me know. There are cases where we require
certain USE flags to be set on dependencies for things to work properly.

About the issue that a test would have caught: was that a chromium test?
If so, which one?

>>> While I don't run tests myself; the need for them is clear, for
>>> those that aim for more production ready systems (eg. university
>>> network PCs).
>>
>> This seems too theoretical to me. I'd be fine with someone
>> volunteering to maintain chromium's src_test in Gentoo. Unless we
>> have such a person though, it seems to mostly take valuable focus
>> away from bugs that definitely *do* affect our users, for no provable
>> benefit for Gentoo.
> 
> What about provable benefit for upstream? Does upstream /dev/null them?

Effectively yes. For an example see
https://bugs.gentoo.org/show_bug.cgi?id=497512 and
https://groups.google.com/a/chromium.org/d/msg/chromium-dev/OdX7ShsOqsM/-R9sexJAEa4J

The failure is not Gentoo-specific, and is not a bug in code but problem
with the test (it makes assumptions about internal glibc
implementation). It actually fails on the latest Ubuntu LTS Trusty Tahr,
which means the test will have to be fixed or disabled upstream. But 6
months of no reaction is not really a good sign IMHO.

Paweł



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 841 bytes --]

  reply	other threads:[~2014-06-01 13:41 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-05-27  7:02 [gentoo-dev] RFC: Removing src_test from www-client/chromium "Paweł Hajdan, Jr."
2014-05-27  8:05 ` Tom Wijsman
2014-05-27 14:09   ` Ian Stakenvicius
2014-05-27 14:47     ` Jeroen Roovers
2014-05-29  9:09     ` Steev Klimaszewski
2014-05-29 10:46       ` Tom Wijsman
2014-05-31 17:50         ` "Paweł Hajdan, Jr."
2014-05-31 18:30           ` Tom Wijsman
2014-06-01 13:41             ` "Paweł Hajdan, Jr." [this message]
2014-06-01 14:41               ` Tom Wijsman
2014-06-02  6:44                 ` "Paweł Hajdan, Jr."
2014-05-31  3:38 ` [gentoo-dev] " Ryan Hill

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=538B2D8F.6030200@gentoo.org \
    --to=phajdan.jr@gentoo.org \
    --cc=gentoo-dev@lists.gentoo.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox