<div dir="ltr"><div dir="ltr">Hi Stephan,</div><div dir="ltr"><br></div><div>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. </div><div><br></div><div>Best regards,<br>Andras</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Apr 26, 2019 at 9:14 AM Stephan Bergmann <<a href="mailto:sbergman@redhat.com">sbergman@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">I have some questions about <br>
<<a href="https://gerrit.libreoffice.org/plugins/gitiles/core/+/6e9a41dbc25402350e4b767d8f10e8e855f7531a%5E!/" rel="noreferrer" target="_blank">https://gerrit.libreoffice.org/plugins/gitiles/core/+/6e9a41dbc25402350e4b767d8f10e8e855f7531a%5E!/</a>> <br>
"create empty .lproj directories for languages supported by OS X" <br>
(quoted below; affecting only the non-default test-install make target, <br>
and only for macOS when configured with --enable-macosx-code-signing):<br>
<br>
* What is the reason to still generate those now-empty <br>
Contents/Resources/*.proj/ directories?<br>
<br>
* Since <br>
<<a href="https://gerrit.libreoffice.org/plugins/gitiles/core/+/7a08bfeabe21193e04b9747a831653efcfc63190%5E!/" rel="noreferrer" target="_blank">https://gerrit.libreoffice.org/plugins/gitiles/core/+/7a08bfeabe21193e04b9747a831653efcfc63190%5E!/</a>> <br>
"tdf#122244 Put InfoPlist.strings files at correct places on macOS", at <br>
least some of those Contents/Resources/*.proj/ directories in <br>
instset/LibreOffice*.app/ and in corresponding (--with-package-format=*) <br>
installation sets will contain empty InfoPlist.strings files.  I do not <br>
know if that is a problem for the --with-package-format=dmg installation <br>
sets, nor whether it is a problem here for this special <br>
--enable-macosx-code-signing test-install target.<br>
<br>
<br>
> commit 6e9a41dbc25402350e4b767d8f10e8e855f7531a<br>
> Author: Andras Timar <<a href="mailto:andras.timar@collabora.com" target="_blank">andras.timar@collabora.com</a>><br>
> Date:   Mon May 11 22:26:29 2015 +0200<br>
> <br>
>     create empty .lproj directories for languages supported by OS X<br>
>     <br>
>     The previous solution had problems. InfoPlist.strings were empty,<br>
>     because localization has not been working since the CFBundleTypeIconFile<br>
>     entries were removed from Info.plist. Icon file basename was the key<br>
>     in documents.ulf. So we packaged 0 bytes long files. The second problem<br>
>     was that we used LibreOffice language codes, and OS X language codes are<br>
>     different in some cases. This caused problems such as French strings on<br>
>     English UI (e.g. Open/Save dialogs), because the system did not<br>
>     recognize en-US.lproj and en-GB.lproj, and fell back to the next one: fr.<br>
>     <br>
>     Conflicts:<br>
>             Makefile.in<br>
>     <br>
>     (cherry picked from commit 91902ef1411943f65da296fefd15fff9170d9c0c)<br>
>     <br>
>     Change-Id: I9c502cdf737b497ca2ceef8f3c535ccfea2f6134<br>
> <br>
> diff --git a/Makefile.in b/Makefile.in<br>
> index 8e84f4682517..ff8f36ba0334 100644<br>
> --- a/Makefile.in<br>
> +++ b/Makefile.in<br>
> @@ -306,14 +306,13 @@ ifeq ($(OS_FOR_BUILD),WNT)<br>
>  else<br>
>         @$(SRCDIR)/solenv/bin/ooinstall $(TESTINSTALLDIR)<br>
>  ifneq ($(MACOSX_CODESIGNING_IDENTITY),)<br>
> -# Unzip bin/InfoPlist_*.zip files into corresponding Resources/*.lproj directories.<br>
> -       set -x; for F in $(TESTINSTALLDIR)/LibreOffice$(if $(ENABLE_RELEASE_BUILD),,Dev).app/Contents/bin/InfoPlist_*.zip; do \<br>
> -               bn=`basename $$F .zip`; \<br>
> -               lang=$${bn#InfoPlist_}; \<br>
> +#<br>
> +# Create Resources/*.lproj directories for languages supported by OS X<br>
> +       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 \<br>
>                 lproj=$(TESTINSTALLDIR)/LibreOffice$(if $(ENABLE_RELEASE_BUILD),,Dev).app/Contents/Resources/$$lang.lproj; \<br>
>                 mkdir $$lproj; \<br>
> -               (cd $$lproj; unzip $$F); \<br>
>         done<br>
> +#<br>
>  # And remove the "bin" folder which should not be there<br>
>         rm -rf $(TESTINSTALLDIR)/LibreOffice$(if $(ENABLE_RELEASE_BUILD),,Dev).app/Contents/bin<br>
>  #<br>
_______________________________________________<br>
LibreOffice mailing list<br>
<a href="mailto:LibreOffice@lists.freedesktop.org" target="_blank">LibreOffice@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/libreoffice" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/libreoffice</a></blockquote></div></div>