Tcl Source Code

View Ticket
Login
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: