Merge lp:~mikemc/ubuntuone-windows-installer/byo-libjpeg into lp:ubuntuone-windows-installer

Proposed by Mike McCracken
Status: Merged
Approved by: dobey
Approved revision: 142
Merged at revision: 140
Proposed branch: lp:~mikemc/ubuntuone-windows-installer/byo-libjpeg
Merge into: lp:ubuntuone-windows-installer
Diff against target: 37 lines (+10/-3)
1 file modified
scripts/setup-mac.py (+10/-3)
To merge this branch: bzr merge lp:~mikemc/ubuntuone-windows-installer/byo-libjpeg
Reviewer Review Type Date Requested Status
dobey (community) Approve
Roberto Alsina (community) Approve
Review via email: mp+126538@code.launchpad.net

Commit message

- Package libjpeg for PIL so we don't break when a user doesn't have it installed. (LP: #1050534)

Description of the change

- Package libjpeg for PIL so we don't break when a user doesn't have it installed. (LP: #1050534)

TO TEST, build the app and check that libjpeg.?.dylib is in UbuntuOne.app/Contents/Frameworks/

also try running SSO like this:

DYLD_PRINT_LIBRARIES=1 UbuntuOne.app/Contents/Resources/Ubuntu\ Single\ Sign-On.app/Contents/MacOS/ubuntu-sso-login-qt --app_name "Ubuntu One"

you will see a ton of spam and in there should be something like these lines:

dyld: loaded: /Users/mmccrack/Documents/Canonical/Source/test-improve-buildout/scripts/dist/build_to_sign/UbuntuOne.app/Contents/Resources/Ubuntu Single Sign-On.app/Contents/Resources/lib/python2.7/lib-dynload/_imaging.so

dyld: loaded: /Users/mmccrack/Documents/Canonical/Source/test-improve-buildout/scripts/dist/build_to_sign/UbuntuOne.app/Contents/Resources/Ubuntu Single Sign-On.app/Contents/MacOS/../Frameworks/libjpeg.8.dylib

then you're good!

Prior to this branch, loading _imaging.so on a dev's machine would grab libjpeg from /usr/local/lib/, where brew installs it. but if it's on a machine without that, whoops!

To post a comment you must log in.
Revision history for this message
Roberto Alsina (ralsina) :
review: Approve
Revision history for this message
dobey (dobey) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'scripts/setup-mac.py'
--- scripts/setup-mac.py 2012-09-14 20:36:23 +0000
+++ scripts/setup-mac.py 2012-09-26 20:21:24 +0000
@@ -502,7 +502,14 @@
502 qt_network_plugins = glob.glob(qt_plugin_path + "/bearer/*.dylib")502 qt_network_plugins = glob.glob(qt_plugin_path + "/bearer/*.dylib")
503 qt_imageformat_plugins = glob.glob(qt_plugin_path +503 qt_imageformat_plugins = glob.glob(qt_plugin_path +
504 "/imageformats/*.dylib")504 "/imageformats/*.dylib")
505 dylib_paths = qt_imageformat_plugins + qt_network_plugins505 qt_dylib_paths = qt_imageformat_plugins + qt_network_plugins
506
507 libjpeg_paths = glob.glob("/usr/local/lib/libjpeg.*.dylib")
508 if len(libjpeg_paths) != 1:
509 print "Warning: expected to find just one version of libjpeg"
510 print "found:", libjpeg_paths
511
512 dylib_paths = qt_dylib_paths + [libjpeg_paths[0]]
506513
507 data_files = glob.glob("data/*")514 data_files = glob.glob("data/*")
508515
@@ -618,7 +625,7 @@
618 "com.ubuntu.one.u1sdtool",625 "com.ubuntu.one.u1sdtool",
619 "u1sdtool")626 "u1sdtool")
620627
621 control_panel_app_path = do_setup(False,628 control_panel_app_path = do_setup(True,
622 "com.ubuntu.one.controlpanel",629 "com.ubuntu.one.controlpanel",
623 "ubuntuone-control-panel-qt",630 "ubuntuone-control-panel-qt",
624 is_main_app=True)631 is_main_app=True)
@@ -641,7 +648,7 @@
641 control_panel_frameworks_path = os.path.join(control_panel_contents_path,648 control_panel_frameworks_path = os.path.join(control_panel_contents_path,
642 "Frameworks")649 "Frameworks")
643650
644 plugin_names = [os.path.basename(f) for f in dylib_paths]651 plugin_names = [os.path.basename(f) for f in qt_dylib_paths]
645652
646 bearers_dir = os.path.join(control_panel_frameworks_path,653 bearers_dir = os.path.join(control_panel_frameworks_path,
647 "plugins", "bearer")654 "plugins", "bearer")

Subscribers

People subscribed via source and target branches