Merge lp:~kroq-gar78/ubuntu/quantal/visualvm/fix-start into lp:ubuntu/quantal/visualvm

Proposed by Aditya V
Status: Merged
Merged at revision: 16
Proposed branch: lp:~kroq-gar78/ubuntu/quantal/visualvm/fix-start
Merge into: lp:ubuntu/quantal/visualvm
Diff against target: 31 lines (+10/-1)
2 files modified
debian/changelog (+8/-0)
debian/jvisualvm.sh (+2/-1)
To merge this branch: bzr merge lp:~kroq-gar78/ubuntu/quantal/visualvm/fix-start
Reviewer Review Type Date Requested Status
Jamie Strandboge Approve
Ubuntu branches Pending
Review via email: mp+107542@code.launchpad.net

Description of the change

I fixed the problems that caused jvisualvm not to start with OpenJDK 6 and 7.

To post a comment you must log in.
Revision history for this message
Jamie Strandboge (jdstrand) wrote :

Thanks for the merge. It looks great!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2012-04-22 14:50:10 +0000
3+++ debian/changelog 2012-05-27 05:30:26 +0000
4@@ -1,3 +1,11 @@
5+visualvm (1.3.2-0ubuntu3) quantal; urgency=low
6+
7+ * debian/jvisualvm.sh:
8+ - look for OpenJDK 7 files also (LP: #937710)
9+ - use multiarch-friendly paths for OpenJDK (LP: #1004370)
10+
11+ -- Aditya Vaidya <kroq.gar78@gmail.com> Sun, 27 May 2012 00:02:57 -0500
12+
13 visualvm (1.3.2-0ubuntu2) precise; urgency=low
14
15 * Build-depend on libnb-platform12-java, since the default is now
16
17=== modified file 'debian/jvisualvm.sh'
18--- debian/jvisualvm.sh 2011-09-08 13:38:19 +0000
19+++ debian/jvisualvm.sh 2012-05-27 05:30:26 +0000
20@@ -8,9 +8,10 @@
21 progdir=/usr/bin
22 APPNAME=visualvm
23 BASEDIR=/usr/lib/visualvm
24+ARCH=`dpkg --print-architecture`
25
26 jdkhome=
27-for j in /usr/lib/jvm/java-6-openjdk /usr/lib/jvm/java-6-sun; do
28+for j in /usr/lib/jvm/java-6-openjdk-$ARCH /usr/lib/jvm/java-6-sun /usr/lib/jvm/java-7-openjdk-$ARCH; do
29 if [ -x $j/bin/javac ]; then
30 jdkhome=$j
31 break

Subscribers

People subscribed via source and target branches

to all changes: