unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: "Daniel Martín via Bug reports for GNU Emacs, the Swiss army knife of text editors" <bug-gnu-emacs@gnu.org>
To: Eli Zaretskii <eliz@gnu.org>
Cc: nisoni@algon.dk, 51321@debbugs.gnu.org
Subject: bug#51321: 29.0.50; date in modelines
Date: Sat, 23 Oct 2021 18:00:39 +0200	[thread overview]
Message-ID: <m11r4bvihk.fsf@yahoo.es> (raw)
In-Reply-To: <m1v91nykch.fsf@yahoo.es> ("Daniel Martín via \"Bug reports for GNU Emacs, the Swiss army knife of text editors\""'s message of "Sat, 23 Oct 2021 14:52:14 +0200")

[-- Attachment #1: Type: text/plain, Size: 310 bytes --]

Daniel Martín via "Bug reports for GNU Emacs, the Swiss army knife of
text editors" <bug-gnu-emacs@gnu.org> writes:

>
> I've attached a patch that solves the issue.  If you think it's a good
> idea, please install it for me.  Thanks.

Sorry, there was a small mistake.  I attach a corrected patch.


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-Set-LC_ALL-to-the-current-locale-in-the-NS-port.patch --]
[-- Type: text/x-patch, Size: 1127 bytes --]

From c27c60d344da64dac42c452e3a3aa7c35522a007 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Daniel=20Mart=C3=ADn?= <mardani29@yahoo.es>
Date: Sat, 23 Oct 2021 14:44:31 +0200
Subject: [PATCH] Set LC_ALL to the current locale in the NS port

* src/nsterm.m (ns_init_locale): If not already set, set LC_ALL to the
current locale ID so that dates, currencies, etc. use the settings
configured in the operating system.  (Bug#51321)
---
 src/nsterm.m | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/nsterm.m b/src/nsterm.m
index aa29c13eb2..19ed750d21 100644
--- a/src/nsterm.m
+++ b/src/nsterm.m
@@ -543,8 +543,10 @@ - (NSColor *)colorUsingDefaultColorSpace
       NSString *localeID = [NSString stringWithFormat:@"%@.UTF-8",
                                      [locale localeIdentifier]];
 
-      /* Set LANG to locale, but not if LANG is already set.  */
+      /* Set LANG and LC_ALL to locale, but not if the variables are
+         already set.  */
       setenv("LANG", [localeID UTF8String], 0);
+      setenv("LC_ALL", [localeID UTF8String], 0);
     }
   @catch (NSException *e)
     {
-- 
2.31.0


  reply	other threads:[~2021-10-23 16:00 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-10-21 15:24 bug#51321: 29.0.50; date in modelines Niels Søndergaard
2021-10-21 17:00 ` Eli Zaretskii
2021-10-21 18:09   ` Niels Søndergaard
2021-10-21 18:13     ` Niels Søndergaard
2021-10-21 18:18     ` Eli Zaretskii
2021-10-22  1:37       ` Niels Søndergaard
2021-10-22  6:34         ` Eli Zaretskii
2021-10-22 13:27           ` Niels Søndergaard
2021-10-22 14:11             ` Eli Zaretskii
2021-10-23 12:16               ` Daniel Martín via Bug reports for GNU Emacs, the Swiss army knife of text editors
2021-10-23 12:23                 ` Eli Zaretskii
2021-10-23 12:52                   ` Daniel Martín via Bug reports for GNU Emacs, the Swiss army knife of text editors
2021-10-23 16:00                     ` Daniel Martín via Bug reports for GNU Emacs, the Swiss army knife of text editors [this message]
2021-10-30 15:41                       ` Stefan Kangas
2021-10-30 18:09                         ` Alan Third
2021-10-31  9:06                           ` Eli Zaretskii
2021-10-31 10:14                             ` Alan Third
2021-10-31 13:01                               ` Eli Zaretskii
2021-11-01 13:04                               ` Filipp Gunbin
2021-11-01 13:33                                 ` Eli Zaretskii
2021-11-01 14:21                                   ` Filipp Gunbin
2021-11-01 14:34                                     ` Eli Zaretskii
2021-11-01 14:45                                       ` Filipp Gunbin
2021-11-01 21:17                                       ` Alan Third
2021-11-01 22:17                                         ` Daniel Martín via Bug reports for GNU Emacs, the Swiss army knife of text editors
2021-11-02  3:40                                           ` Niels Søndergaard
2021-11-02 16:10                                           ` Filipp Gunbin
2021-11-02 18:22                                           ` Stefan Kangas
2021-11-09  7:52                                             ` Stefan Kangas
2021-11-09 14:13                                               ` Filipp Gunbin
2022-09-10  5:09                                                 ` Lars Ingebrigtsen
2021-11-02  1:10                                         ` Filipp Gunbin
2021-10-23 12:34                 ` Niels Søndergaard
2021-10-23 12:55                   ` Daniel Martín via Bug reports for GNU Emacs, the Swiss army knife of text editors
2021-10-23 12:58                     ` Niels Søndergaard
2021-10-22 14:40         ` Lars Ingebrigtsen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=m11r4bvihk.fsf@yahoo.es \
    --to=bug-gnu-emacs@gnu.org \
    --cc=51321@debbugs.gnu.org \
    --cc=eliz@gnu.org \
    --cc=mardani29@yahoo.es \
    --cc=nisoni@algon.dk \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).