Update comments

We now have the --enable-canonical-installation-tree-structure option
which does what was suggested.

Change-Id: I41215275d6d8c8f11b4a45390d44998c967fcc53

1 files changed, 3 insertions, 9 deletions

diff --git a/solenv/bin/modules/installer/simplepackage.pm b/solenv/bin/modules/installer/simplepackage.pm
index 3a745ecc0e42..8f8cddd588d3 100644
--- a/solenv/bin/modules/installer/simplepackage.pm
+++ b/solenv/bin/modules/installer/simplepackage.pm
@@ -407,8 +407,9 @@ sub create_package
                 # Sign the .app as a whole, which means (re-)signing
                 # the CFBundleExecutable from Info.plist, i.e.
                 # soffice, plus the contents of the Resources tree
-                # (which at the moment is not much, far from all of
-                # our non-code "resources").
+                # (which unless you used
+                # --enable-canonical-installation-tree-structure is
+                # not much, far from all of our non-code "resources").
 
                 # Don't bother yet to sign each individual .dylib. (We
                 # do that for "make dev-install", but not here.)
@@ -417,13 +418,6 @@ sub create_package
                 # gb_LinkTarget__command_dynamiclink in
                 # solenv/gbuild/platform/macosx.mk.
 
-                # Eventually it would be a good idea to re-organise
-                # the app bundle structure to be more Mac-like and
-                # actually put all non-code resources (including
-                # extension scripts!) into Resources so that they
-                # participate in the signing and their validity can be
-                # guaranteed.
-
                 $entitlements = '';
                 $entitlements = "--entitlements $ENV{'BUILDDIR'}/lo.xcent" if defined($ENV{'ENABLE_MACOSX_SANDBOX'});