[MERGE] Support for tbzr c++ implemented shell extension

Mark Hammond mhammond at skippinet.com.au
Tue Dec 16 21:59:44 GMT 2008


> I think it could be as simple as having a known function to call with
> given arguments.
> So in bzr's setup.py you have
> 
> from tbzrlib import setup_config
> setup_config.configure_for_setup(current_state...)

For Tortoise, integrating with the .iss.cog file will also be a challenge.

> Which could probably be re-used in tbzr's setup.py (if it had one).

It does - but its setup.py doesn't have anything py2exe related.

> >> +    # py2exe may find references to part of the plugin (eg, the tests)
> >> +    # so we tell py2exe to leave the plugins out of the .zip file
...

> > We ship plugins as normal files on the file-system - however, the
> build process can cause *some* of these plugin files to end up in
> library.zip.  Thus, I saw (eg) "plugins/svn/test" in library.zip, and
> then saw import errors related to that as the rest of the svn plugin
> wasn't.  The above change avoids any part of the plugins ending up in
> library.zip file.
> >
> 
> K. I think I'd like to have that statement verbatim here. To explain
> the difference.

To be clear - you want the 2 comment lines above that say we are excluding the plugins from library.zip replaced with the 6 lines above, as they stand, which says we are excluding the plugins from library.zip?

> > See above - I think a single installer is the simplest and easiest,
> and it detects a 32bit OS and doesn't attempt to install the 64bit
> shell extension.  The downside though is that the 32bit installer is
> bigger than it needs to be.
> >
> 
> But only for the single .dll, right? Does that add any appreciable
> size?

The 64bit DLL itself is 260kb, but we also need to ship a 64bit version of tortoise-overlays at 227kb.  I expect our DLL will compress well, but the MSI probably already is compressed.

> PS> BB:tweak ish, though BB won't let me vote this far down in the
> thread.

To be clear - I should resubmit the patch with only that comment changed to get an "approve"?

Cheers,

Mark




More information about the bazaar mailing list