GNU bug report logs - #55314
29.0.50; [PATCH] Add Greek translation of the TUTORIAL

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 55314 in the body.
You can then email your comments to 55314 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Protesilaos Stavrou <info <at> protesilaos.com>
To: bug-gnu-emacs <at> gnu.org
Subject: 29.0.50; [PATCH] Add Greek translation of the TUTORIAL
Date: Sun, 08 May 2022 16:11:12 +0300

[Message part 1 (text/plain, inline)]

Dear maintainers,

I have prepared a translation of the tutorial in Greek.  Please see the
attached patch.

The translation is a faithful copy of the TUTORIAL (in English).

Note that I named the file "TUTORIAL.el_GR" to avoid misrepresenting it
as an Elisp file.

All the best,
Protesilaos (or simply "Prot")

-- 
Protesilaos Stavrou
https://protesilaos.com

[0001-Add-Greek-translation-of-the-TUTORIAL.patch (text/x-patch, attachment)]

Message #8 received at 55314 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Protesilaos Stavrou <info <at> protesilaos.com>
Cc: 55314 <at> debbugs.gnu.org
Subject: Re: bug#55314: 29.0.50; [PATCH] Add Greek translation of the TUTORIAL
Date: Sun, 08 May 2022 17:14:21 +0300

> From: Protesilaos Stavrou <info <at> protesilaos.com>
> Date: Sun, 08 May 2022 16:11:12 +0300
> 
> I have prepared a translation of the tutorial in Greek.  Please see the
> attached patch.

Thanks.

> diff --git a/etc/NEWS b/etc/NEWS
> index 73e61aab32..afb297dde1 100644
> --- a/etc/NEWS
> +++ b/etc/NEWS
> @@ -612,6 +612,9 @@ available options can be restored by enabling this option.
>  
>  +++
>  ** 'imenu' is now bound to 'M-g i' globally.
> +** A new translation of the Emacs tutorial is available in Greek
> +Type 'C-u C-h t' to select it in case your language setup does not do
> +so automatically.

There should be an empty line before the new entry, and it should be
marked with "---", meaning that no changes are necessary for this
addition.

Also, I'd move this entry to a different place in NEWS; somehoe having
this near 'imenu' doesn't sound right to me.  Maybe we should rename
the "Editing complex text layout (CTL) scripts" section to be
something like "Internationalization changes", and then add this entry
there?

> diff --git a/lisp/language/greek.el b/lisp/language/greek.el
> index 58f4fe6fc4..edea582781 100644
> --- a/lisp/language/greek.el
> +++ b/lisp/language/greek.el
> @@ -79,7 +79,9 @@ (set-language-info-alist
>  	   (coding-priority greek-iso-8bit)
>  	   (nonascii-translation . iso-8859-7)
>  	   (input-method . "greek")
> -	   (documentation . t)))
> +	   (documentation . t)
> +           (sample-text . "Γειa σας")

sample-text should actually be the entire Greek line from etc/HELLO.
And can we please have a more meaningful value for documentation?

Message #11 received at 55314 <at> debbugs.gnu.org (full text, mbox):

From: Protesilaos Stavrou <info <at> protesilaos.com>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 55314 <at> debbugs.gnu.org
Subject: Re: bug#55314: 29.0.50; [PATCH] Add Greek translation of the TUTORIAL
Date: Sun, 08 May 2022 17:36:22 +0300

[Message part 1 (text/plain, inline)]

> From: Eli Zaretskii <eliz <at> gnu.org>
> Date: Sun, 08 May 2022 17:14:21 +0300
>
>> diff --git a/etc/NEWS b/etc/NEWS
>> index 73e61aab32..afb297dde1 100644
>> --- a/etc/NEWS
>> +++ b/etc/NEWS
>> @@ -612,6 +612,9 @@ available options can be restored by enabling this option.
>>  
>>  +++
>>  ** 'imenu' is now bound to 'M-g i' globally.
>> +** A new translation of the Emacs tutorial is available in Greek
>> +Type 'C-u C-h t' to select it in case your language setup does not do
>> +so automatically.
>
> There should be an empty line before the new entry, and it should be
> marked with "---", meaning that no changes are necessary for this
> addition.
>
> Also, I'd move this entry to a different place in NEWS; somehoe having
> this near 'imenu' doesn't sound right to me.  Maybe we should rename
> the "Editing complex text layout (CTL) scripts" section to be
> something like "Internationalization changes", and then add this entry
> there?

Thank you!  I prepended the "---" and filed it under the renamed
"Internationalization changes" heading.

>> diff --git a/lisp/language/greek.el b/lisp/language/greek.el
>> index 58f4fe6fc4..edea582781 100644
>> --- a/lisp/language/greek.el
>> +++ b/lisp/language/greek.el
>> @@ -79,7 +79,9 @@ (set-language-info-alist
>>  	   (coding-priority greek-iso-8bit)
>>  	   (nonascii-translation . iso-8859-7)
>>  	   (input-method . "greek")
>> -	   (documentation . t)))
>> +	   (documentation . t)
>> +           (sample-text . "Γειa σας")
>
> sample-text should actually be the entire Greek line from etc/HELLO.

Okay, I copied it.

> And can we please have a more meaningful value for documentation?

In my original patch, I kept it the way it was.  Now I updated it.
Please see the attached patch.

-- 
Protesilaos Stavrou
https://protesilaos.com

[0001-Add-Greek-translation-of-the-TUTORIAL.patch (text/x-patch, attachment)]

Message #16 received at 55314-done <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Protesilaos Stavrou <info <at> protesilaos.com>
Cc: 55314-done <at> debbugs.gnu.org
Subject: Re: bug#55314: 29.0.50; [PATCH] Add Greek translation of the TUTORIAL
Date: Sun, 08 May 2022 18:51:38 +0300

> From: Protesilaos Stavrou <info <at> protesilaos.com>
> Cc: 55314 <at> debbugs.gnu.org
> Date: Sun, 08 May 2022 17:36:22 +0300
> 
> Please see the attached patch.

Thanks, installed.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.