Raymond Toy pushed to branch master at cmucl / cmucl
Commits:
-
225940e4
by Raymond Toy at 2022-11-25T16:07:57+00:00
-
bea34994
by Raymond Toy at 2022-11-25T16:07:57+00:00
7 changed files:
- src/code/save.lisp
- src/code/unix.lisp
- src/general-info/release-21e.md
- src/i18n/locale/cmucl-unix.pot
- src/i18n/locale/cmucl.pot
- src/lisp/os-common.c
- tests/issues.lisp
Changes:
| ... | ... | @@ -142,6 +142,30 @@ |
| 142 | 142 | (file c-call:c-string)
|
| 143 | 143 | (initial-function (alien:unsigned #.vm:word-bits)))
|
| 144 | 144 | |
| 145 | +(defun set-up-locale-external-format ()
|
|
| 146 | + "Add external format alias for :locale to the format specified by
|
|
| 147 | + the locale as set by setlocale(3C)."
|
|
| 148 | + (let ((codeset (unix::unix-get-locale-codeset)))
|
|
| 149 | + (cond ((zerop (length codeset))
|
|
| 150 | + ;; Codeset was the empty string, so just set :locale to
|
|
| 151 | + ;; alias to the default external format.
|
|
| 152 | + (setf (gethash :locale stream::*external-format-aliases*)
|
|
| 153 | + *default-external-format*))
|
|
| 154 | + (t
|
|
| 155 | + (let ((codeset-format (intern codeset "KEYWORD")))
|
|
| 156 | + ;; If we know the format, we can set the alias.
|
|
| 157 | + ;; Otherwise, print a warning and use :iso8859-1 as the
|
|
| 158 | + ;; alias.
|
|
| 159 | + (setf (gethash :locale stream::*external-format-aliases*)
|
|
| 160 | + (if (stream::find-external-format codeset-format nil)
|
|
| 161 | + codeset-format
|
|
| 162 | + (progn
|
|
| 163 | + (warn "Unsupported external format; using :iso8859-1 instead: ~S"
|
|
| 164 | + codeset-format)
|
|
| 165 | + :iso8859-1)))))))
|
|
| 166 | + (values))
|
|
| 167 | + |
|
| 168 | +
|
|
| 145 | 169 | (defun save-lisp (core-file-name &key
|
| 146 | 170 | (purify t)
|
| 147 | 171 | (root-structures ())
|
| ... | ... | @@ -252,8 +276,13 @@ |
| 252 | 276 | ;; Set the runtime locale
|
| 253 | 277 | (unless (zerop (unix::unix-setlocale))
|
| 254 | 278 | (warn "os_setlocale failed"))
|
| 279 | + ;; Load external format aliases now so we can aliases to
|
|
| 280 | + ;; specify the external format.
|
|
| 281 | + (stream::load-external-format-aliases)
|
|
| 255 | 282 | ;; Set the locale for lisp
|
| 256 | 283 | (intl::setlocale)
|
| 284 | + ;; Set up :locale format
|
|
| 285 | + (set-up-locale-external-format)
|
|
| 257 | 286 | (ext::process-command-strings process-command-line)
|
| 258 | 287 | (setf *editor-lisp-p* nil)
|
| 259 | 288 | (macrolet ((find-switch (name)
|
| ... | ... | @@ -2915,3 +2915,10 @@ |
| 2915 | 2915 | 256)))
|
| 2916 | 2916 | (when (zerop result)
|
| 2917 | 2917 | (cast buf c-call:c-string)))))
|
| 2918 | + |
|
| 2919 | +(defun unix-get-locale-codeset ()
|
|
| 2920 | + _N"Get the codeset from the locale"
|
|
| 2921 | + (cast (alien-funcall
|
|
| 2922 | + (extern-alien "os_get_locale_codeset"
|
|
| 2923 | + (function (* char))))
|
|
| 2924 | + c-string)) |
| ... | ... | @@ -59,7 +59,7 @@ public domain. |
| 59 | 59 | * ~~#132~~ Ansi test `RENAME-FILE.1` no fails
|
| 60 | 60 | * ~~#134~~ Handle the case of `(expt complex complex-rational)`
|
| 61 | 61 | * ~~#136~~ `ensure-directories-exist` should return the given pathspec
|
| 62 | - * #139 `*default-external-format*` defaults to `:utf-8`
|
|
| 62 | + * #139 `*default-external-format*` defaults to `:utf-8`; add alias for `:locale` external format
|
|
| 63 | 63 | * ~~#140~~ External format for streams that are not `file-stream`'s
|
| 64 | 64 | * ~~#141~~ Disallow locales that are pathnames to a localedef file
|
| 65 | 65 | * ~~#142~~ `(random 0)` signals incorrect error
|
| ... | ... | @@ -1435,3 +1435,7 @@ msgid "" |
| 1435 | 1435 | " calling this so that the correct locale is returned."
|
| 1436 | 1436 | msgstr ""
|
| 1437 | 1437 | |
| 1438 | +#: src/code/unix.lisp
|
|
| 1439 | +msgid "Get the codeset from the locale"
|
|
| 1440 | +msgstr ""
|
|
| 1441 | + |
| ... | ... | @@ -6714,6 +6714,12 @@ msgid "" |
| 6714 | 6714 | "This is true if and only if the lisp was started with the -edit switch."
|
| 6715 | 6715 | msgstr ""
|
| 6716 | 6716 | |
| 6717 | +#: src/code/save.lisp
|
|
| 6718 | +msgid ""
|
|
| 6719 | +"Add external format alias for :locale to the format specified by\n"
|
|
| 6720 | +" the locale as set by setlocale(3C)."
|
|
| 6721 | +msgstr ""
|
|
| 6722 | + |
|
| 6717 | 6723 | #: src/code/save.lisp
|
| 6718 | 6724 | msgid ""
|
| 6719 | 6725 | "Saves a CMU Common Lisp core image in the file of the specified name. The\n"
|
| ... | ... | @@ -7,6 +7,7 @@ |
| 7 | 7 | |
| 8 | 8 | #include <assert.h>
|
| 9 | 9 | #include <errno.h>
|
| 10 | +#include <langinfo.h>
|
|
| 10 | 11 | #include <locale.h>
|
| 11 | 12 | #include <math.h>
|
| 12 | 13 | #include <netdb.h>
|
| ... | ... | @@ -796,3 +797,9 @@ os_get_lc_messages(char *buf, int len) |
| 796 | 797 | /* Return -1 if setlocale failed. */
|
| 797 | 798 | return locale ? 0 : -1;
|
| 798 | 799 | }
|
| 800 | + |
|
| 801 | +char *
|
|
| 802 | +os_get_locale_codeset()
|
|
| 803 | +{
|
|
| 804 | + return nl_langinfo(CODESET);
|
|
| 805 | +} |
| ... | ... | @@ -745,6 +745,11 @@ |
| 745 | 745 | (assert-equal (map 'list #'char-name string)
|
| 746 | 746 | (map 'list #'char-name (read-line s))))))
|
| 747 | 747 |
|
| 748 | +(define-test issue.139-locale-external-format
|
|
| 749 | + (:tag :issues)
|
|
| 750 | + ;; Just verify that :locale format exists
|
|
| 751 | + (assert-true (stream::find-external-format :locale nil)))
|
|
| 752 | + |
|
| 748 | 753 | ;;; Test stream-external-format for various types of streams.
|
| 749 | 754 | |
| 750 | 755 | (define-test issue.140.two-way-stream
|
| ... | ... | @@ -792,7 +797,6 @@ |
| 792 | 797 | (stream-external-format
|
| 793 | 798 | (make-broadcast-stream s1 s2 s3)))))))
|
| 794 | 799 | |
| 795 | - |
|
| 796 | 800 | (define-test issue.150
|
| 797 | 801 | (:tag :issues)
|
| 798 | 802 | (let ((ext:*gc-verbose* nil)
|