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
diff --git a/debian/changelog b/debian/changelog
index c3b0906..b087262 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,12 @@
1triton (2.0.0.post1-3ubuntu1) mantic; urgency=medium
2
3 * Fix FTBFS (LP: #2025279):
4 - debian/control: use explicit Build-Depends: llvm-14-dev instead of
5 llvm-dev
6 - debian/rules: disable LTO
7
8 -- Nick Rosbrook <nick.rosbrook@canonical.com> Wed, 28 Jun 2023 13:26:16 -0400
9
1triton (2.0.0.post1-3) unstable; urgency=medium10triton (2.0.0.post1-3) unstable; urgency=medium
211
3 * Uploading to unstable.12 * Uploading to unstable.
diff --git a/debian/control b/debian/control
index 6274a1e..3269b67 100644
--- a/debian/control
+++ b/debian/control
@@ -1,13 +1,14 @@
1Source: triton1Source: triton
2Section: python2Section: python
3Priority: optional3Priority: optional
4Maintainer: Debian Deep Learning Team <debian-ai@lists.debian.org>4Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
5XSBC-Original-Maintainer: Debian Deep Learning Team <debian-ai@lists.debian.org>
5Uploaders: M. Zhou <lumin@debian.org>, Petter Reinholdtsen <pere@debian.org>6Uploaders: M. Zhou <lumin@debian.org>, Petter Reinholdtsen <pere@debian.org>
6Build-Depends: cmake,7Build-Depends: cmake,
7 debhelper-compat (= 13),8 debhelper-compat (= 13),
8 dh-python,9 dh-python,
9 libmlir-14-dev,10 libmlir-14-dev,
10 llvm-dev,11 llvm-14-dev,
11 mlir-14-tools,12 mlir-14-tools,
12 python3-all-dev,13 python3-all-dev,
13 python3-sphinx,14 python3-sphinx,
diff --git a/debian/rules b/debian/rules
index 4b77a79..3864999 100755
--- a/debian/rules
+++ b/debian/rules
@@ -1,5 +1,6 @@
1#! /usr/bin/make -f1#! /usr/bin/make -f
22
3export DEB_BUILD_MAINT_OPTIONS=optimize=-lto
3export PYBUILD_NAME=triton4export PYBUILD_NAME=triton
4%:5%:
5 dh $@ --with python3,sphinxdoc --buildsystem=pybuild6 dh $@ --with python3,sphinxdoc --buildsystem=pybuild

Subscribers

People subscribed via source and target branches