Re: [Codelite-developers] Mac OS X app bundle plugin

[M G. <aur...@gm...>]

Hi Eran,
> Sorry for the late reply about the plugin. But I finally got some free
> time - I added the sources to SVN (after compiling them under Windows
> using wx2.810)
> the main changes:
>
> - Converted strings to confirm with wx<  2.9
> - Updated ./configure + Mac packaging script to include this plugin for Mac only
> - Minor change: the settings plugin now remembers its size + position
>
> You can now use 'svn up' on the codelite source tree followed by
> configure&&  make to compile the plugin
>    
thanks for committing :) if you ever get some more free time I might 
welcome help on a project wizard
(but the current state is not THAT bad either; I guess it mostly needs 
some documentation).
Also, I don't know if you see some way to prevent the modification from 
being done twice on the same target?
My best idea was to leave some specific comment in the custom rules, but 
I don't like that. You're more familiar than
me with CL so you may see something I don't?

Finally, could you please commit this small patch to correct a FIXME?

Index: MacBundler/macbundler.cpp
===================================================================
--- MacBundler/macbundler.cpp   (revision 4072)
+++ MacBundler/macbundler.cpp   (working copy)
@@ -268,9 +268,8 @@
          file.AddLine(wxT( " <string>English</string>\n" ));
          file.AddLine(wxT( " <key>CFBundlePackageType</key>" ));
          file.AddLine(wxT( " <string>APPL</string>\n" ));
-        //FIXME: whatwxT( to do with this one?
          file.AddLine(wxT( " <key>CFBundleShortVersionString</key>" ));
-        file.AddLine(wxT( " <string></string>\n" ));
+        file.AddLine(wxT( " <string>" ) + m_version->GetValue()  + wxT( 
"</string>\n"  ));
          file.AddLine(wxT( "</dict>"));
          file.AddLine(wxT( "</plist>"));

thanks

-- Auria