not including all the modules breaks the osx build in all sorts of "interesting" ways

[Antoine Martin]

Since r5716, we don't include all the modules by default.
This is the only sane way of avoiding the codecs loading errors on startup.

The problem is that it completely breaks the OSX build which relied heavily on everything being included... sigh.

[Antoine Martin]

After many (completely wasted) hours tweaking, it seems that maybe my whole build system is screwed. I get an image, but none of the Cython modules work (no video encodings, no rencode, etc..)

Even reverting to r5000 or r5500 doesn't fix my build problem :(

Please confirm that your builds are still OK.

[alas]

Our osx r5711 build is good, our osx r5722 build is broken.

xpra client version 0.12.0
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "xpra/scripts/main.pyc", line 74, in main
  File "xpra/scripts/main.pyc", line 611, in run_mode
  File "xpra/scripts/main.pyc", line 928, in run_client
  File "xpra/scripts/main.pyc", line 1002, in make_client
  File "xpra/client/gtk2/client.pyc", line 25, in <module>
  File "xpra/client/gtk_base/gtk_client_base.pyc", line 23, in <module>
  File "xpra/client/ui_client_base.pyc", line 24, in <module>
  File "xpra/client/client_tray.pyc", line 10, in <module>
  File "xpra/client/gtk_base/gtk_window_backing_base.pyc", line 11, in <module>
  File "xpra/client/window_backing_base.pyc", line 16, in <module>
ImportError: No module named codecs.codec_constants

Same error you're getting, I assume?

[Antoine Martin]

What about latest? (r5724) I'm slightly ahead of that, but it should be ok.

[Smo]

Here is the error i'm seeing. Seems to compile fine.

running build_scripts
creating build/bdist.macosx-10.6-i386/scripts-2.7
copying and adjusting scripts/xpra -> build/bdist.macosx-10.6-i386/scripts-2.7
copying and adjusting scripts/xpra_launcher -> build/bdist.macosx-10.6-i386/scripts-2.7
changing mode of build/bdist.macosx-10.6-i386/scripts-2.7/xpra from 644 to 755
changing mode of build/bdist.macosx-10.6-i386/scripts-2.7/xpra_launcher from 644 to 755
creating /Users/user1/xpra-src/trunk/src/build/bdist.macosx-10.6-i386/python2.7-standalone
creating /Users/user1/xpra-src/trunk/src/build/bdist.macosx-10.6-i386/python2.7-standalone/app
creating /Users/user1/xpra-src/trunk/src/build/bdist.macosx-10.6-i386/python2.7-standalone/app/collect
creating /Users/user1/xpra-src/trunk/src/build/bdist.macosx-10.6-i386/python2.7-standalone/app/temp
creating build/bdist.macosx-10.6-i386/python2.7-standalone/app/lib-dynload
creating build/bdist.macosx-10.6-i386/python2.7-standalone/app/Frameworks
error: You must specify either app or plugin
ERROR: py2app failed

[Antoine Martin]

The logging just before that should look like this:

setup_options:
* name=xpra
* author=Antoine Martin
* author_email=antoine@devloop.org.uk
* app=['osxxpra/osx_xpra_launcher.py']
* download_url=http://xpra.org/src/
* url=http://xpra.org/
* version=0.12.0
* data_files=[('share/man/man1', ['../src/man/xpra.1', '../src/man/xpra_launcher.1']), ('share/xpra', ['../src/README', '../src/COPYING']), ('share/xpra/icons', ['../src/icons/browse.png', '../src/icons/clipboard.png', '../src/icons/close.png', '../src/icons/compressed.png', '../src/icons/connect.png', '../src/icons/encoding.png', '../src/icons/features.png', '../src/icons/information.png', '../src/icons/keyboard.png', '../src/icons/microphone.png', '../src/icons/numlock.png', '../src/icons/opengl.png', '../src/icons/package.png', '../src/icons/quit.png', '../src/icons/raise.png', '../src/icons/retry.png', '../src/icons/slider.png', '../src/icons/speaker.png', '../src/icons/speed.png', '../src/icons/statistics.png', '../src/icons/ticked-small.png', '../src/icons/unticked-small.png', '../src/icons/xpra.png']), ('share/applications', ['../src/xdg/xpra_launcher.desktop']), ('share/applications', ['../src/xdg/xpra.desktop']), ('share/icons', ['../src/xdg/xpra.png']), ('share/xpra/webm', ['../src/xpra/codecs/webm/LICENSE'])]
* packages=['osxxpra']
* long_description='screen for X' -- a tool to detach/reattach running X programs
* description=Xpra: 'screen for X' utility

py2app options:
frameworks : ['CoreFoundation', 'Foundation', 'AppKit']
packages : ['osxxpra']
iconfile : ./xpra.icns
strip : False
site_packages : False
includes : ['glib', 'gio', 'cairo', 'pango', 'pangocairo', 'atk', 'gobject', 'gtk.keysyms', 'osxxpra', 'hashlib', 'PIL']
plist : {'CFBundleDocumentTypes': [{'CFBundleTypeName': 'Xpra Session Config File', 'CFBundleTypeRole': 'Viewer', 'CFBundleName': 'Xpra', 'CFBundleTypeExtensions': ['Xpra']}]}
argv_emulation : True

And you can see that app is defined in setup_options.

[Antoine Martin]

The problem above was fixed in r5739.

More improvements for builds on all platforms in r5740 + r5742.
The benefit of those changes is that we can now include just the modules we care about, and no more than that.

All builds now use the same file, and almost the same code paths.

Please verify that all the platforms you care about still build OK.

[Smo]

Win32 - Builds and Packages
Fedora 19, 20 Builds and Packages
OSX 10.5.8, 10.6.8 Build and Packages

Looks good!

[Antoine Martin]

Done!

[migration script]

this ticket has been moved to: ​https://github.com/Xpra-org/xpra/issues/529