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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxx> 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@xxxxxxxxxxxxx>
> 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@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/libreoffice
_______________________________________________
LibreOffice mailing list
LibreOffice@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/libreoffice

[Index of Archives]     [LARTC]     [Bugtraq]     [Yosemite Forum]     [Photo]

  Powered by Linux