Re: Contributing French & Turkish translations to Pan

This is a multi-part message in MIME format.
--------------0EAwBdWjMdrxTdyTfaUSJJr6
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit

Hi there,

Here’s the feedback after my translation efforts: Pan is part of GNOME 
(Other Apps) and its translations are managed by the Damned Lies website 
of GNOME. So if you want to submit a new translation or update an 
existing one, you need to create an account there and join the 
corresponding language team. Let’s consider French: from 
<https://l10n.gnome.org/module/pan/> you’d go to 
<https://l10n.gnome.org/vertimus/pan/master/po/fr/> and after 
registering and joining the team, you’d reserve the translation from the 
drop down menu at the bottom of the page to make other members aware 
that you’re working on it. Download the gettext file, open it with say 
poedit and happy translating! When your work is finished, upload the 
translation on the same page and wait for someone to review and commit 
it. That’s all there is to it. The update for the French language, new 
translations for the help in French and Turkish have been submitted and 
are patiently waiting.

Additionally, to answer your following remark:

> I've applied your fr.po file to my git repo, and I get this kind of diff:
>  
> -#: ../pan/data/cert-store.cc:328
> +#: pan/data/cert-store.cc:359
>  msgid ""
>  "Error initializing Certificate Store. Check that the permissions for the "
> 
> 
> The path of the source file has changed in the comment (the ../ part is gone). 
> I've no idea if that may cause problems for the next French translator.
> 
> Could you check the impact ?

I asked to a fellow translator, and it seems that there's no impact. 
Anyway, translations will land from GNOME, so you shouldn't have to 
worry about that.

Also, I (sort of) worked out why the version compiled “by hand” didn’t 
take into account language changes on Ubuntu, whereas the version from 
the repositories happily switched languages according to the desktop 
settings (MATE in my case).

Firstly, I had to modify the CMakeLists.txt file at the root directory 
of the extracted tarball by adding
target_link_libraries(pan ${Gettext_LIBRARIES})
and later, just beneath
set(GETTEXT_PACKAGE pan)
I added
find_package(Gettext REQUIRED)
and
include_directories(${Gettext_INCLUDE_DIRS})
otherwise it just didn’t care about gettext files.

Then, Pan happily installs language files at 
/usr/local/share/locale/fr/LC_MESSAGES/pan.mo for French, and the help 
file goes to /usr/local/share/help/fr/pan/index.docbook for French 
again. But apparently it looks for these files in 
/usr/share/locale/<lang>/LC_MESSAGES/pan.mo, therefore never finds the 
translation (and translated help) files and falls back to English.

I played a little bit with the “main” CmakeLists.txt file to make Pan 
take into account the translations. I’m attaching a patch in case it 
would be useful. It can certainly be simplified, but as it does what I 
need I didn’t touch it any further. Besides, in order to build new help 
languages when they're committed, you may need to to update the
set(lang_list cs de es fr sv tr)
line of CMakeLists.txt file in the help subdirectory.

Finally, you may want to add the cmake-gui package to the Fedora 
dependency list.

I hope this is useful in some way.

Best regards.

--------------0EAwBdWjMdrxTdyTfaUSJJr6
Content-Type: text/x-patch; charset=UTF-8; name="mods.diff"
Content-Disposition: attachment; filename="mods.diff"
Content-Transfer-Encoding: base64

LS0tIC4vb3JpZ2luYWwvQ01ha2VMaXN0cy50eHQJMjAyNS0wNS0yNSAxNjoyNzoyOC4wMDAw
MDAwMDAgKzAzMDAKKysrIC4vbmV3L0NNYWtlTGlzdHMudHh0CTIwMjUtMDctMTIgMjM6NTc6
MjMuMzY2OTE1MTM1ICswMzAwCkBAIC0xNywxNyArMTcsMTcgQEAKICAgREVTQ1JJUFRJT04g
IlBhbiB1c2VuZXQgcmVhZGVyIgogICBIT01FUEFHRV9VUkwgImh0dHBzOi8vZ2l0bGFiLmdu
b21lLm9yZy9HTk9NRS9wYW4iCiApCi0KK21lc3NhZ2UoU1RBVFVTICJDTUFLRV9JTlNUQUxM
X1BSRUZJWCBpcyAke0NNQUtFX0lOU1RBTExfUFJFRklYfSIpCittZXNzYWdlKFNUQVRVUyAi
UEFOTE9DQUxFRElSIHdpbGwgYmUgJHtDTUFLRV9JTlNUQUxMX1BSRUZJWH0vc2hhcmUvbG9j
YWxlIikKICMgI2NtYWtlZGVmaW5lIGNvbXBsZXRlbHkgcmVmdXNlcyB0byByZWNvZ25pc2Ug
Q01BS0VfUFJPSkVDVF9WRVJTSU9OX3h4eHggb3IKICMgUFJPSkVDVF9WRVJTSU9OX3h4eHgK
IHNldChWRVJTSU9OX01BSk9SICR7UFJPSkVDVF9WRVJTSU9OX01BSk9SfSkKIHNldChWRVJT
SU9OX01JTk9SICR7UFJPSkVDVF9WRVJTSU9OX01JTk9SfSkKIHNldChWRVJTSU9OX1BBVENI
ICR7UFJPSkVDVF9WRVJTSU9OX1BBVENIfSkKIHNldChWRVJTSU9OX1RXRUFLICR7UFJPSkVD
VF9WRVJTSU9OX1RXRUFLfSkKLQorc2V0KExPQ0FMRURJUiAiJHtDTUFLRV9JTlNUQUxMX1BS
RUZJWH0vc2hhcmUvbG9jYWxlIiBDQUNIRSBQQVRIICJMb2NhbGUgZGlyZWN0b3J5IikKIHNl
dChQQU5fREJVU19TRVJWSUNFX05BTUUgICAgICAib3JnLmdub21lLnBhbiIpCiBzZXQoUEFO
X0RCVVNfU0VSVklDRV9QQVRIICAgICAgIi9uZXdzL3Bhbi9OWkIiKQotCiAjIFN0YW5kYXJk
IENNYWtlIG1vZHVsZXMKIGluY2x1ZGUoQ1Rlc3QpCiAKQEAgLTcwLDggKzcwLDEyIEBACiAK
ICMgaHR0cHM6Ly9jbWFrZS5vcmcvY21ha2UvaGVscC9sYXRlc3QvY29tbWFuZC9hZGRfZXhl
Y3V0YWJsZS5odG1sCiBhZGRfZXhlY3V0YWJsZShwYW4gcGFuL2d1aS9wYW4uY2MgcGFuL2d1
aS9ndWkuY2MpCit0YXJnZXRfbGlua19saWJyYXJpZXMocGFuICR7R2V0dGV4dF9MSUJSQVJJ
RVN9KQordGFyZ2V0X2NvbXBpbGVfZGVmaW5pdGlvbnMocGFuIFBSSVZBVEUgUEFOTE9DQUxF
RElSPVwiJHtDTUFLRV9JTlNUQUxMX1BSRUZJWH0vc2hhcmUvbG9jYWxlXCIpCiAjIHJlbW92
ZSB0aGlzIHNvIHBhbiBiaW5hcnkgbGFuZHMgaW4gYnVpbGQgZGlyLiBCdXQgdGhpcyByZXF1
aXJlcwogIyBmaXJzdCB0byByZW5hbWUgcGFuLyBkaXIgdG8gc3JjLwordGFyZ2V0X2xpbmtf
bGlicmFyaWVzKHBhbiAke0dldHRleHRfTElCUkFSSUVTfSkKKwogc2V0X3RhcmdldF9wcm9w
ZXJ0aWVzKHBhbiBQUk9QRVJUSUVTIFJVTlRJTUVfT1VUUFVUX0RJUkVDVE9SWSAicGFuL2d1
aSIpCiB0YXJnZXRfY29tcGlsZV9vcHRpb25zKHBhbiBQUklWQVRFICIke0NYWF9TVER9IiAi
LVdyZW9yZGVyIiAiLVd6ZXJvLWFzLW51bGwtcG9pbnRlci1jb25zdGFudCIpCiAKQEAgLTEw
Miw2ICsxMDYsOCBAQAogY2hlY2tfc3ltYm9sX2V4aXN0cyhsb2NhbHRpbWVfciAidGltZS5o
IiBIQVZFX0xPQ0FMVElNRV9SKQogCiBzZXQoR0VUVEVYVF9QQUNLQUdFIHBhbikKK2ZpbmRf
cGFja2FnZShHZXR0ZXh0IFJFUVVJUkVEKQoraW5jbHVkZV9kaXJlY3Rvcmllcygke0dldHRl
eHRfSU5DTFVERV9ESVJTfSkKIAogZmluZF9wYWNrYWdlKFBrZ0NvbmZpZyBSRVFVSVJFRCkK
IApAQCAtMjMyLDggKzIzOCw2IEBACiAjIGh0dHBzOi8vY21ha2Uub3JnL2NtYWtlL2hlbHAv
bGF0ZXN0L2NvbW1hbmQvY29uZmlndXJlX2ZpbGUuaHRtbAogY29uZmlndXJlX2ZpbGUoJHtD
TUFLRV9TT1VSQ0VfRElSfS9jb25maWctY21ha2UuaC5pbiAke0NNQUtFX0JJTkFSWV9ESVJ9
L2NvbmZpZy5oKQogCi10YXJnZXRfY29tcGlsZV9kZWZpbml0aW9ucyhwYW4gUFJJVkFURSAi
UEFOTE9DQUxFRElSPVwiJHtMT0NBTEVESVJ9XCIiKQotCiAjIGh0dHBzOi8vY21ha2Uub3Jn
L2NtYWtlL2hlbHAvbGF0ZXN0L2NvbW1hbmQvdGFyZ2V0X2xpbmtfbGlicmFyaWVzLmh0bWwK
IHRhcmdldF9saW5rX2xpYnJhcmllcyhwYW4KICAgcGFuLWNjLWd1aQo=

--------------0EAwBdWjMdrxTdyTfaUSJJr6
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: base64
Content-Disposition: inline

X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KUGFuLXVzZXJz
IG1haWxpbmcgbGlzdApQYW4tdXNlcnNAbm9uZ251Lm9yZwpodHRwczovL2xpc3RzLm5vbmdudS5v
cmcvbWFpbG1hbi9saXN0aW5mby9wYW4tdXNlcnMK

--------------0EAwBdWjMdrxTdyTfaUSJJr6--