Re: Re "create empty .lproj directories for languages supported by OS X"

Hi Stephan,

I don't remember the details. Probably the original problem was that French
open dialog popped up for English UI. I wanted to solve that in the first
place. For production builds I make, I always use code signing, so maybe
it's not a special case. One more target that should be checked, it is the
App Store package. I don't remember if the fix was for App Store package or
for the dmg.

Best regards,
Andras

On Fri, Apr 26, 2019 at 9:14 AM Stephan Bergmann <sbergman@redhat.com>
wrote:

> I have some questions about
> <
> https://gerrit.libreoffice.org/plugins/gitiles/core/+/6e9a41dbc25402350e4b767d8f10e8e855f7531a%5E!/>
>
> "create empty .lproj directories for languages supported by OS X"
> (quoted below; affecting only the non-default test-install make target,
> and only for macOS when configured with --enable-macosx-code-signing):
>
> * What is the reason to still generate those now-empty
> Contents/Resources/*.proj/ directories?
>
> * Since
> <
> https://gerrit.libreoffice.org/plugins/gitiles/core/+/7a08bfeabe21193e04b9747a831653efcfc63190%5E!/>
>
> "tdf#122244 Put InfoPlist.strings files at correct places on macOS", at
> least some of those Contents/Resources/*.proj/ directories in
> instset/LibreOffice*.app/ and in corresponding (--with-package-format=*)
> installation sets will contain empty InfoPlist.strings files.  I do not
> know if that is a problem for the --with-package-format=dmg installation
> sets, nor whether it is a problem here for this special
> --enable-macosx-code-signing test-install target.
>
>
> > commit 6e9a41dbc25402350e4b767d8f10e8e855f7531a
> > Author: Andras Timar <andras.timar@collabora.com>
> > Date:   Mon May 11 22:26:29 2015 +0200
> >
> >     create empty .lproj directories for languages supported by OS X
> >
> >     The previous solution had problems. InfoPlist.strings were empty,
> >     because localization has not been working since the
> CFBundleTypeIconFile
> >     entries were removed from Info.plist. Icon file basename was the key
> >     in documents.ulf. So we packaged 0 bytes long files. The second
> problem
> >     was that we used LibreOffice language codes, and OS X language codes
> are
> >     different in some cases. This caused problems such as French strings
> on
> >     English UI (e.g. Open/Save dialogs), because the system did not
> >     recognize en-US.lproj and en-GB.lproj, and fell back to the next
> one: fr.
> >     Conflicts:
> >             Makefile.in
> >
> >     (cherry picked from commit 91902ef1411943f65da296fefd15fff9170d9c0c)
> >
> >     Change-Id: I9c502cdf737b497ca2ceef8f3c535ccfea2f6134
> >
> > diff --git a/Makefile.in b/Makefile.in
> > index 8e84f4682517..ff8f36ba0334 100644
> > --- a/Makefile.in
> > +++ b/Makefile.in
> > @@ -306,14 +306,13 @@ ifeq ($(OS_FOR_BUILD),WNT)
> >  else
> >         @$(SRCDIR)/solenv/bin/ooinstall $(TESTINSTALLDIR)
> >  ifneq ($(MACOSX_CODESIGNING_IDENTITY),)
> > -# Unzip bin/InfoPlist_*.zip files into corresponding Resources/*.lproj
> directories.
> > -       set -x; for F in $(TESTINSTALLDIR)/LibreOffice$(if
> $(ENABLE_RELEASE_BUILD),,Dev).app/Contents/bin/InfoPlist_*.zip; do \
> > -               bn=`basename $$F .zip`; \
> > -               lang=$${bn#InfoPlist_}; \
> > +#
> > +# Create Resources/*.lproj directories for languages supported by OS X
> > +       set -x; for lang in ca cs da de el en es fi fr hr hu id it ja ko
> ms nl no pl pt pt_PT ro ru sk sv th tr uk vi zh_CN zh_TW; do \
> >                 lproj=$(TESTINSTALLDIR)/LibreOffice$(if
> $(ENABLE_RELEASE_BUILD),,Dev).app/Contents/Resources/$$lang.lproj; \
> >                 mkdir $$lproj; \
> > -               (cd $$lproj; unzip $$F); \
> >         done
> > +#
> >  # And remove the "bin" folder which should not be there
> >         rm -rf $(TESTINSTALLDIR)/LibreOffice$(if
> $(ENABLE_RELEASE_BUILD),,Dev).app/Contents/bin
> >  #
> _______________________________________________
> LibreOffice mailing list
> LibreOffice@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/libreoffice