--- Begin Message ---
Subject: |
[PATCH] Add support for the Modi script |
Date: |
Mon, 16 May 2022 05:03:59 +0530 |
I have added the modi script to Emacs this time.
I have also renamed Hindi to Devanagari in etc/HELLO, since this script is also used by
languages other than Hindi, like: Marathi and Nepali. If this change is not warranted you can
revert it.
Please review the patch.
0001-Add-support-for-the-Modi-script.patch
Description: Text Data
--- End Message ---
--- Begin Message ---
Subject: |
Re: bug#55439: [PATCH] Add support for the Modi script |
Date: |
Tue, 17 May 2022 16:24:15 +0300 |
> From: समीर सिंह Sameer Singh <lumarzeli30@gmail.com>
> Date: Tue, 17 May 2022 04:55:42 +0530
> Cc: 55439@debbugs.gnu.org
>
> Can you tell how many more old scripts you intend to add? I'm
> thinking perhaps we should make a separate section for them in NEWS,
> instead of basically repeating the same text over and over and over
> again
>
> I am planning to add about 31 more scripts.
Thanks, I therefore rewrote the NEWS entries for the scripts we
already installed to make them more concise. Please follow this
template for the future additions.
> I'm okay with the change, but please add the same sample-text to the
> Devanagari language environment in indian.el.
>
> Done.
>
> I have also:
> 1. added sample texts to other scripts which didn't have them.
> 2. replaced the spaces with a tab in the sample texts which I had added.
> 3. renamed Punjabi to Gurmukhi in its set-language-info-alist.
>
> In etc/HELLO
> I have:
> 1. added a Gurumukhi greeting.
> 2. replaced the Odia greeting, it was ଶୁଣିବେ (tl. Listen) earlier, which
> I don't think is a greeting, therefore I
> have replaced it with ନମସ୍କାର (namaskaar)
>
> The "Hindi" greeting had a character at its end, U+0964, which
> disappeared when you moved the line to Devanagari. Is that
> intentional?
>
> Yes, that was intentional, it was a full stop. I didn't think that it was
> necessary, the other greetings also did not
> have it.
>
> Period missing at the end of this line.
>
> Oops. Fixed it. The earlier news entry about Syloti Nagri did not have it
> too, so I have also added it there.
Installed, thanks.
--- End Message ---