Merge lp:~tormodvolden/ubuntu/karmic/googleearth-package/karmic-bugs into lp:ubuntu/karmic/googleearth-package

Proposed by Tormod Volden
Status: Merged
Merge reported by: James Westby
Merged at revision: not available
Proposed branch: lp:~tormodvolden/ubuntu/karmic/googleearth-package/karmic-bugs
Merge into: lp:ubuntu/karmic/googleearth-package
Diff against target: None lines
To merge this branch: bzr merge lp:~tormodvolden/ubuntu/karmic/googleearth-package/karmic-bugs
Reviewer Review Type Date Requested Status
Bryce Harrington Approve
Ubuntu branches Pending
Review via email: mp+11016@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Tormod Volden (tormodvolden) wrote :

[PATCH] Do not use local libssl either.
Move the local libssl away the same way we do for libcrypto.
The standard libssl0.9.8 Debian package should work fine.
(Closes: 528029, 537837)

[PATCH] Fix bashism in make-googleearth-package
The string equality operator in the Bourne shell is "=". Anyway, if
we want to probe for the empty string, use -z instead.
(Closes: 524686)

Revision history for this message
Bryce Harrington (bryce) wrote :

Looks good to me

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/changelog'
--- debian/changelog 2009-04-16 15:10:48 +0000
+++ debian/changelog 2009-09-01 21:45:13 +0000
@@ -1,3 +1,10 @@
1googleearth-package (0.5.6ubuntu1) karmic; urgency=low
2
3 * Fix bashism in make-googleearth-package (Closes: #524686)
4 * Do not use the shipped libssl (Closes: #537837) LP: #369471
5
6 -- Tormod Volden <debian.tormod@gmail.com> Tue, 01 Sep 2009 23:19:45 +0200
7
1googleearth-package (0.5.6) unstable; urgency=low8googleearth-package (0.5.6) unstable; urgency=low
29
3 * Fix typo in googleearth wrapper script that broke things.10 * Fix typo in googleearth wrapper script that broke things.
411
=== modified file 'make-googleearth-package'
--- make-googleearth-package 2009-04-16 15:10:48 +0000
+++ make-googleearth-package 2009-09-01 21:28:44 +0000
@@ -300,7 +300,7 @@
300#exec /lib/ld-linux.so.2 --library-path /usr/lib/googleearth /usr/lib/googleearth/googleearth-bin "$@"300#exec /lib/ld-linux.so.2 --library-path /usr/lib/googleearth /usr/lib/googleearth/googleearth-bin "$@"
301301
302GOOGLE_EARTH_LD_LIBRARY_PATH=/usr/lib/googleearth302GOOGLE_EARTH_LD_LIBRARY_PATH=/usr/lib/googleearth
303if [ "$LD_LIBRARY_PATH" == "" ]; then303if [ -z "$LD_LIBRARY_PATH" ]; then
304 LD_LIBRARY_PATH=${LD_LIBRARY_PATH}:${GOOGLE_EARTH_LD_LIBRARY_PATH}304 LD_LIBRARY_PATH=${LD_LIBRARY_PATH}:${GOOGLE_EARTH_LD_LIBRARY_PATH}
305else305else
306 LD_LIBRARY_PATH=${GOOGLE_EARTH_LD_LIBRARY_PATH}306 LD_LIBRARY_PATH=${GOOGLE_EARTH_LD_LIBRARY_PATH}
@@ -379,6 +379,7 @@
379379
380 # Workaround symbol problem in libcrypto380 # Workaround symbol problem in libcrypto
381 mv libcrypto.so.0.9.8 libcrypto.so.0.9.8.moved.for.workaround381 mv libcrypto.so.0.9.8 libcrypto.so.0.9.8.moved.for.workaround
382 mv libssl.so.0.9.8 libssl.so.0.9.8.moved.for.workaround
382383
383 # debian menu entry384 # debian menu entry
384 cd "$instdir"385 cd "$instdir"

Subscribers

People subscribed via source and target branches

to all changes: