Tcl Source Code

View Ticket
Login
Ticket UUID: 403531
Title: Patch from Bug 219379 (re. Tcl_StaticPackage)
Type: Patch Version: None
Submitter: dgp Created on: 2001-01-31 23:10:00
Subsystem: 40. Dynamic Loading Assigned To: dgp
Priority: 5 Medium Severity:
Status: Closed Last Modified: 2008-04-15 23:42:21
Resolution: Invalid Closed By: dgp
    Closed on: 2008-04-15 16:42:21
User Comments: kennykb added on 2003-02-02 10:08:48:
Logged In: YES 
user_id=99768

Don, this patch is hopelessly out of date, and we decided that
219379 resulted from conflating the 'package' that [load] uses and
the 'package' that [package] uses, yes? Should we close this patch?

dgp added on 2002-10-24 21:36:38:
Logged In: YES 
user_id=80530

At the very least the patch needs updating.
It will not apply to the HEAD.

Will have to get back to this later...

kennykb added on 2002-10-24 21:10:49:
Logged In: YES 
user_id=99768

In a conversation that I had  with dgp on the Tcl'ers Chat last night, dgp
apparently convinced himself that the patch still isn't entirely the right thing.

There is apparently some confusion in the documentation between the
entrypoint name of the load moduleand the name of the package, if any,
that the DLL provides - leading to this patch.  The two really are distinct,
though, and it really would take a TIP to make changes to bring them in
line.

Certainly the fact that a slave interpreter can request a [load] that
is inconsistent with the master is a problem.  The correct solution is
not yet clear.

dkf added on 2002-10-24 16:59:56:
Logged In: YES 
user_id=79902

I don't know.  I don't remember.  :^D  If you feel that it
fixes things, then that's fine with me.

dgp added on 2002-10-24 05:09:25:
Logged In: YES 
user_id=80530

14 months later...

What else is needed from me?

dgp added on 2001-08-08 04:19:27:
Logged In: YES 
user_id=80530

Any more thoughts on this?

Was there some additional explanation or
documentation I was supposed to provide?

dgp added on 2001-04-04 08:11:11:

File Added - 4925: 403531.patch

Logged In: YES 
user_id=80530

Updated patch rewrites the man pages changes, and removes
the [load] error on failure to provide a package.

dkf added on 2001-02-27 18:34:36:
Logged In: YES 
user_id=79902

Some of the grammar in the documentation seems a little
shakey (and maybe you should add some change-bars too.) 
I've not the expertise to evaluate the technical portion of
the patch.

Attachments: