Merge lp:~mtmiller/ubuntu/trusty/octave/lp1293876 into lp:ubuntu/trusty/octave

Proposed by Mike Miller
Status: Merged
Merge reported by: Mike Miller
Merged at revision: not available
Proposed branch: lp:~mtmiller/ubuntu/trusty/octave/lp1293876
Merge into: lp:ubuntu/trusty/octave
Diff against target: 73 lines (+12/-16)
4 files modified
debian/NEWS (+0/-5)
debian/changelog (+6/-0)
debian/control (+1/-1)
debian/rules (+5/-10)
To merge this branch: bzr merge lp:~mtmiller/ubuntu/trusty/octave/lp1293876
Reviewer Review Type Date Requested Status
Chris J Arges Approve
Review via email: mp+214537@code.launchpad.net

Description of the change

Contains the fix for LP #1293876 by no longer linking Octave with LLVM. This is a port of the same fix made to the Debian package to fix the corresponding Debian bug #743260.

This fix is fairly important to get into Trusty, since it will fix a segfault that would have affected all users plotting with Octave's default plot configuration who are using Mesa llvmpipe graphics drivers (nouveau, radeon, swrast, etc.).

To post a comment you must log in.
Revision history for this message
Chris J Arges (arges) wrote :

Looks good, sponsored in Trusty.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/NEWS'
--- debian/NEWS 2014-02-25 20:41:26 +0000
+++ debian/NEWS 2014-04-07 13:36:59 +0000
@@ -9,9 +9,4 @@
9 interface, there is the `octave-cli' executable which is not linked9 interface, there is the `octave-cli' executable which is not linked
10 against Qt.10 against Qt.
1111
12 * Starting with this version, Octave incorporates a just-in-time (JIT)
13 compiler, which can offer performance improvements in some situations.
14 Since it is still experimental, the JIT is disabled by default, but
15 you can activate it with the `jit_enable' command.
16
17 -- Sébastien Villemot <sebastien@debian.org> Fri, 20 Dec 2013 20:47:36 +010012 -- Sébastien Villemot <sebastien@debian.org> Fri, 20 Dec 2013 20:47:36 +0100
1813
=== modified file 'debian/changelog'
--- debian/changelog 2014-03-10 09:11:34 +0000
+++ debian/changelog 2014-04-07 13:36:59 +0000
@@ -1,3 +1,9 @@
1octave (3.8.0-5ubuntu6) trusty; urgency=medium
2
3 * Disable JIT on all archs to fix LLVM library conflicts. (LP: #1293876)
4
5 -- Mike Miller <mtmiller@debian.org> Mon, 07 Apr 2014 08:13:30 -0400
6
1octave (3.8.0-5ubuntu5) trusty; urgency=medium7octave (3.8.0-5ubuntu5) trusty; urgency=medium
28
3 * Fix installing packages where dependency name contains '-'. (LP: #1288136)9 * Fix installing packages where dependency name contains '-'. (LP: #1288136)
410
=== modified file 'debian/control'
--- debian/control 2014-02-26 03:44:16 +0000
+++ debian/control 2014-04-07 13:36:59 +0000
@@ -15,7 +15,7 @@
15 libqhull-dev, desktop-file-utils, libfltk1.3-dev, libgl2ps-dev,15 libqhull-dev, desktop-file-utils, libfltk1.3-dev, libgl2ps-dev,
16 libgraphicsmagick++1-dev, libftgl-dev, libfontconfig1-dev,16 libgraphicsmagick++1-dev, libftgl-dev, libfontconfig1-dev,
17 libqrupdate-dev, libarpack2-dev (>= 2.1+parpack96.dfsg-2), dh-exec,17 libqrupdate-dev, libarpack2-dev (>= 2.1+parpack96.dfsg-2), dh-exec,
18 dpkg-dev (>= 1.16.1), llvm-3.3-dev [!arm64 !armel !armhf !mips !mipsel !powerpc !ppc64el],18 dpkg-dev (>= 1.16.1),
19 default-jdk, libqscintilla2-dev, libxft-dev, javahelper19 default-jdk, libqscintilla2-dev, libxft-dev, javahelper
20Standards-Version: 3.9.520Standards-Version: 3.9.5
21Homepage: http://www.octave.org/21Homepage: http://www.octave.org/
2222
=== modified file 'debian/rules'
--- debian/rules 2014-02-26 16:39:01 +0000
+++ debian/rules 2014-04-07 13:36:59 +0000
@@ -30,15 +30,10 @@
30WITH_JAVA_FLAGS := --with-java-homedir=$(JAVA_HOME) --with-java-libdir=$(JAVA_LIBDIR)30WITH_JAVA_FLAGS := --with-java-homedir=$(JAVA_HOME) --with-java-libdir=$(JAVA_LIBDIR)
31endif31endif
3232
33# Disable the JIT on some archs. JIT unit tests with LLVM currently segfault33# Disable the experimental JIT unconditionally for now. The LLVM library
34# on those. This arch list must be kept in sync with the arch34# interface is unstable. Linking with LLVM may also cause conflicts with other
35# list qualifying llvm-dev in debian/control.35# libraries, for example the mesa graphics stack which also uses LLVM.
36NO_JIT_ARCHS := arm64 armel armhf mips mipsel powerpc ppc64el36JIT_FLAG := --disable-jit
37ifneq (,$(filter $(DEB_HOST_ARCH),$(NO_JIT_ARCHS)))
38DISABLE_JIT_FLAG := --disable-jit
39endif
40
41export LLVM_CONFIG = llvm-config-3.3
4237
43%:38%:
44 dh $@ --with autoreconf --parallel39 dh $@ --with autoreconf --parallel
@@ -48,7 +43,7 @@
4843
49# override normal dh_auto_configure call to pass OpenMP flag to it (#631831)44# override normal dh_auto_configure call to pass OpenMP flag to it (#631831)
50override_dh_auto_configure:45override_dh_auto_configure:
51 dh_auto_configure -- --enable-openmp $(WITH_JAVA_FLAGS) $(DISABLE_JIT_FLAG)46 dh_auto_configure -- --enable-openmp $(WITH_JAVA_FLAGS) $(JIT_FLAG)
5247
53# dh_auto_test tries to run "make test", so override it48# dh_auto_test tries to run "make test", so override it
54override_dh_auto_test:49override_dh_auto_test:

Subscribers

People subscribed via source and target branches

to all changes: