Ticket UUID: | 976520 | |||
Title: | Encoding search path | |||
Type: | Patch | Version: | None | |
Submitter: | dgp | Created on: | 2004-06-21 05:18:44 | |
Subsystem: | 38. Init - Library - Autoload | Assigned To: | dgp | |
Priority: | 9 Immediate | Severity: | ||
Status: | Closed | Last Modified: | 2004-12-02 02:26:37 | |
Resolution: | Accepted | Closed By: | dgp | |
Closed on: | 2004-11-30 19:34:40 | |||
Description: |
The attached patch represents work in progress toward separating the search path for encodings from the search path for the script library (init.tcl). It is not yet in acceptable shape to commit, but it should apply and run, so testing is encouraged. | |||
User Comments: |
vincentdarley added on 2004-12-02 02:26:37:
Logged In: YES user_id=32170 Please see bug 1077005 introduced by this patch. dgp added on 2004-12-01 02:34:40: Logged In: YES user_id=80530 committed for Tcl 8.5a2. dgp added on 2004-12-01 01:16:54: File Deleted - 110649: File Added - 110719: 976520.patch Logged In: YES user_id=80530 updated patch takes care of some code cleanup. dgp added on 2004-11-30 13:31:48: File Deleted - 110605: dgp added on 2004-11-30 13:31:47: File Added - 110649: 976520.patch dgp added on 2004-11-30 13:31:44: Logged In: YES user_id=80530 updated patch corrects a Windows compile error, and inlines the initScript. dgp added on 2004-11-30 04:42:10: File Deleted - 110236: File Added - 110605: 976520.patch dgp added on 2004-11-30 04:42:09: Logged In: YES user_id=80530 updated patch does that rename, fixes some remaining unixInit.test issues, and reformats as a unidiff. Other than a little code cleanup, this should be ready now. dgp added on 2004-11-29 23:07:38: Logged In: YES user_id=80530 Yes, that sounds like a good idea. dkf added on 2004-11-29 16:36:15: Logged In: YES user_id=79902 Should that be TclpGetEncodingNameFromEnvironment(), to emphasize that this is not returning an encoding but a string that names what encoding would be preferred? dgp added on 2004-11-27 12:19:03: File Deleted - 110016: File Added - 110236: 976520.patch Logged In: YES user_id=80530 Updated patch include a new routine TclpGetEncodingFromEnvironment() so that it is possible to learn from Tcl what it thinks the [encoding system] should be, even when a missing *.enc file, or an incomplete encoding search patch won't let the [encoding system] be properly initialized. dgp added on 2004-11-25 06:02:14: File Deleted - 109888: File Added - 110016: 976520.patch dgp added on 2004-11-25 06:02:12: Logged In: YES user_id=80530 updated patch corrects some Windows bugs (thanks, kbk), and adds some more progress, too. dgp added on 2004-11-25 02:38:34: Logged In: YES user_id=80530 Having multiple directories seems better to me, allowing apps or users to extend the encodings available without having to copy/reproduce all of them that come installed with Tcl. The patch taking shape here provides routines Tcl(Get|Set)EncodingSearchPath that can easily support the command, or be made public to support C apps. hobbs added on 2004-11-25 02:31:42: Logged In: YES user_id=72656 FWIW, I was going to TIP it as 'encoding directory', as in a single directory. 'searchpath' (or just 'path') implies multiple directories. Did we want to allow for that? dgp added on 2004-11-24 09:31:06: Logged In: YES user_id=80530 thanks for testing on unix. The unixInit-2.4 failure is a known glitch; it reflects a weakness in the test more than any real problem. The most effective way to fix that will actually be when the [encoding searchpath] (or whatever we decide to call it) sub-command becomes available. That TIP should be coming along soon after 8.5a2 release. msofer added on 2004-11-24 07:27:14: Logged In: YES user_id=148712 Confirming that the patch sets the correct encoding for ./tclsh. But I now get a failure of unixInit-2.4 dgp added on 2004-11-24 06:16:36: File Deleted - 109879: File Added - 109888: 976520.patch dgp added on 2004-11-24 06:16:35: Logged In: YES user_id=80530 D'OH! That patch would not even compile on WIndows. I think this one will, but need someone to test for sure. dgp added on 2004-11-24 05:41:45: File Deleted - 109748: dgp added on 2004-11-24 05:41:44: File Added - 109879: 976520.patch dgp added on 2004-11-24 05:41:40: Logged In: YES user_id=80530 more progress. standalone ./tclsh should get the right [encoding system] now. dgp added on 2004-11-23 05:25:26: File Deleted - 109645: File Added - 109748: 976520.patch dgp added on 2004-11-23 05:25:24: Logged In: YES user_id=80530 update to HEAD again. dgp added on 2004-11-22 06:45:43: File Deleted - 107736: File Added - 109645: 976520.patch Logged In: YES user_id=80530 Update to HEAD dgp added on 2004-11-06 00:20:12: File Deleted - 97961: File Added - 107736: 976520.patch Logged In: YES user_id=80530 Update to HEAD dgp added on 2004-08-16 23:15:42: File Deleted - 93267: File Added - 97961: 976520.patch dgp added on 2004-08-16 23:15:41: Logged In: YES user_id=80530 Update to HEAD. dgp added on 2004-07-09 00:21:34: File Deleted - 93265: dgp added on 2004-07-09 00:21:33: File Added - 93267: 976520.patch kennykb added on 2004-07-09 00:13:33: File Added - 93265: 976520.patch kennykb added on 2004-07-09 00:13:32: Logged In: YES user_id=99768 Uploaded a revised patch that fixes a couple of compilation errors on Windows dgp added on 2004-07-08 23:30:15: File Deleted - 92537: File Added - 93262: 976520.patch dgp added on 2004-07-08 23:30:13: Logged In: YES user_id=80530 updated again to resolve conflicts with the HEAD dgp added on 2004-07-01 21:48:57: File Deleted - 92298: File Added - 92537: 976520.patch Logged In: YES user_id=80530 updated to resolve conflicts with the HEAD. patthoyts added on 2004-06-30 21:46:21: Logged In: YES user_id=202636 Tested this patch with Win98 MSVC 6. The patched version raised no new test suite failures. dgp added on 2004-06-29 22:06:50: File Deleted - 92288: File Added - 92298: 976520.patch dgp added on 2004-06-29 22:06:49: Logged In: YES user_id=80530 updated again for testing dgp added on 2004-06-29 21:09:33: File Deleted - 91897: File Added - 92288: 976520.patch Logged In: YES user_id=80530 updated to HEAD dgp added on 2004-06-25 21:30:47: File Deleted - 91693: File Added - 91897: 976520.patch dgp added on 2004-06-25 21:30:46: Logged In: YES user_id=80530 just another update. kennykb added on 2004-06-24 04:04:49: File Added - 91693: 976520.patch Logged In: YES user_id=99768 Uploaded revised patch that corrects a few compilation problems on vc++ dgp added on 2004-06-24 03:48:59: Logged In: YES user_id=80530 That last comment isn't quite correct; but getting closer. dgp added on 2004-06-24 03:12:21: File Deleted - 91586: File Added - 91691: 976520.patch Logged In: YES user_id=80530 Updated patch reforms Tcl_FindExecutable so that it need not do encoding corrections for itself. It no longer needs the TclInitSubsystems call itself, but it is left in place until the proper alternative initialization triggers are identified. dgp added on 2004-06-23 11:52:54: File Deleted - 91536: File Added - 91586: 976520.patch Logged In: YES user_id=80530 updated patch to resolve conflicts dgp added on 2004-06-23 00:16:13: File Deleted - 91422: File Added - 91536: 976520.patch dgp added on 2004-06-23 00:16:12: Logged In: YES user_id=80530 Revised patch is considerably more ambitious in re-writing encoding initialization and other startup logic. Please test. dgp added on 2004-06-22 04:01:58: File Deleted - 91415: File Added - 91422: 976520.patch Logged In: YES user_id=80530 Revised patch incorporates kbk's fixes and only adds existing directories to the default encoding search path. kennykb added on 2004-06-22 03:27:24: File Added - 91415: esp-kbk.patch Logged In: YES user_id=99768 Attaching a revised patch that silences several compilation warnings in VC++. No new test failures in either VC++ or mingw with --enable-threads or without. dgp added on 2004-06-21 12:18:45: File Added - 91323: esp.patch |
Attachments:
- 976520.patch [download] added by dgp on 2004-12-01 01:16:54. [details]