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

Subscribers

People subscribed via source and target branches

to all changes: