Merge ~seb128/britney:speaking-ng-386 into ~ubuntu-release/britney/+git/hints-ubuntu:devel

Proposed by Sebastien Bacher
Status: Merged
Merged at revision: f0c5eafb3271d702a3ae8c96b8c4bd69de50be75
Proposed branch: ~seb128/britney:speaking-ng-386
Merge into: ~ubuntu-release/britney/+git/hints-ubuntu:devel
Diff against target: 11 lines (+3/-0)
1 file modified
ubuntu-release (+3/-0)
Reviewer Review Type Date Requested Status
Steve Langasek Approve
Review via email: mp+423674@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Steve Langasek (vorlon) wrote :

The consideration here is that espeak-ng is a package which does build i386 libraries, and this will reduce our test coverage for the compat libraries. Wouldn't it be better to carry a small delta to disable the uninstallable tests on i386 in Ubuntu?

Revision history for this message
Sebastien Bacher (seb128) wrote :

The less ongoing maintenance cost we have to carry deltas over Debian the more we have capacity to focus on other work. The library isn't important for i386, I didn't investigate the details of why it's built but it seems it's needed to build one of the gstreamer plugins.

Do you maybe know if there is a way to skip some of the autopkgtests in ubuntu/i386 which could be upstreamed to Debian?

Revision history for this message
Steve Langasek (vorlon) wrote :

I don't really find the maintenance cost argument persuasive; yes there's a cost for each delta, but managing what deltas exist is a core responsibility of the teams and the delta in question would be trivially low-maintenance.

I would also prefer to have the package in sync if possible, but there's no way to express that this test should be skipped on i386 only in Ubuntu without a delta.

Anyway, I do find it persuasive that this is a library we don't care about on i386, it's only built because it's a build-dependency for an irrelevant binary from the speech-dispatcher source (speech-dispatcher-espeak-ng). So yes, let's go ahead and ignore the autopkgtests here.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/ubuntu-release b/ubuntu-release
2index 86767e2..fa52aec 100644
3--- a/ubuntu-release
4+++ b/ubuntu-release
5@@ -21,3 +21,6 @@ force-badtest exim4/4.95-4ubuntu2/ppc64el
6
7 # aqsis is not installable on i386, remove hint after migration
8 force-badtest aqsis/1.8.2+git20210815-1ubuntu1/i386
9+
10+# espeak-ng now include tests depending on clang which isn't installable
11+force-badtest espeak-ng/1.51+dfsg-7/i386

Subscribers

People subscribed via source and target branches