Merge ~enr0n/ubuntu/+source/triton:fix-lp2025279 into ubuntu/+source/triton:ubuntu/devel

Proposed by Nick Rosbrook
Status: Needs review
Proposed branch: ~enr0n/ubuntu/+source/triton:fix-lp2025279
Merge into: ubuntu/+source/triton:ubuntu/devel
Diff against target: 49 lines (+13/-2)
3 files modified
debian/changelog (+9/-0)
debian/control (+3/-2)
debian/rules (+1/-0)
Reviewer Review Type Date Requested Status
Athos Ribeiro (community) Approve
Steve Langasek (community) Needs Information
Review via email: mp+445580@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Julian Andres Klode (juliank) wrote :

I mean this looks good to me, but I'd prefer someone not in foundations to sponsor this so we get more sponsor coverage for Nick's coredev application.

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

(please resubscribe ubuntu-sponsors after addressing)

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

On Mon, Jul 03, 2023 at 05:56:00PM -0000, Nick Rosbrook wrote:

> Ah, sorry. After resolving the first issue, the build still failed with
> an LTO related issue. So disabling LTO is also needed to resolve the
> FTBFS.

Please show more detail (somewhere) about what the build failure was so that
we can see it was LTO-related.

Revision history for this message
Nick Rosbrook (enr0n) wrote :

> On Mon, Jul 03, 2023 at 05:56:00PM -0000, Nick Rosbrook wrote:
>
> > Ah, sorry. After resolving the first issue, the build still failed with
> > an LTO related issue. So disabling LTO is also needed to resolve the
> > FTBFS.
>
> Please show more detail (somewhere) about what the build failure was so that
> we can see it was LTO-related.

Mentioned on the bug report: https://bugs.launchpad.net/ubuntu/+source/triton/+bug/2025279/comments/2.

Revision history for this message
Athos Ribeiro (athos-ribeiro) wrote :

LGTM

Uploaded.

Uploading to ubuntu (via ftp to upload.ubuntu.com):
  Uploading triton_2.0.0.post1-3ubuntu1.dsc: done.
  Uploading triton_2.0.0.post1-3ubuntu1.debian.tar.xz: done.
  Uploading triton_2.0.0.post1-3ubuntu1_source.buildinfo: done.
  Uploading triton_2.0.0.post1-3ubuntu1_source.changes: done.
Successfully uploaded packages.

review: Approve

Unmerged commits

1c63085... by Nick Rosbrook

update changelog

5623382... by Nick Rosbrook

update maintainer

0dbfdfa... by Nick Rosbrook

debian/rules: disable LTO

a8f7c1b... by Nick Rosbrook

debian/control: use explicit Build-Depends: llvm-14-dev

Since Build-Depends: libmlir-14-dev mlir-14-tools imply llvm-14-dev is
required for the build, be explicit in that dep rather than using
llvm-dev.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/debian/changelog b/debian/changelog
2index c3b0906..b087262 100644
3--- a/debian/changelog
4+++ b/debian/changelog
5@@ -1,3 +1,12 @@
6+triton (2.0.0.post1-3ubuntu1) mantic; urgency=medium
7+
8+ * Fix FTBFS (LP: #2025279):
9+ - debian/control: use explicit Build-Depends: llvm-14-dev instead of
10+ llvm-dev
11+ - debian/rules: disable LTO
12+
13+ -- Nick Rosbrook <nick.rosbrook@canonical.com> Wed, 28 Jun 2023 13:26:16 -0400
14+
15 triton (2.0.0.post1-3) unstable; urgency=medium
16
17 * Uploading to unstable.
18diff --git a/debian/control b/debian/control
19index 6274a1e..3269b67 100644
20--- a/debian/control
21+++ b/debian/control
22@@ -1,13 +1,14 @@
23 Source: triton
24 Section: python
25 Priority: optional
26-Maintainer: Debian Deep Learning Team <debian-ai@lists.debian.org>
27+Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
28+XSBC-Original-Maintainer: Debian Deep Learning Team <debian-ai@lists.debian.org>
29 Uploaders: M. Zhou <lumin@debian.org>, Petter Reinholdtsen <pere@debian.org>
30 Build-Depends: cmake,
31 debhelper-compat (= 13),
32 dh-python,
33 libmlir-14-dev,
34- llvm-dev,
35+ llvm-14-dev,
36 mlir-14-tools,
37 python3-all-dev,
38 python3-sphinx,
39diff --git a/debian/rules b/debian/rules
40index 4b77a79..3864999 100755
41--- a/debian/rules
42+++ b/debian/rules
43@@ -1,5 +1,6 @@
44 #! /usr/bin/make -f
45
46+export DEB_BUILD_MAINT_OPTIONS=optimize=-lto
47 export PYBUILD_NAME=triton
48 %:
49 dh $@ --with python3,sphinxdoc --buildsystem=pybuild

Subscribers

People subscribed via source and target branches