Menu
▾
▴
#581 Another path to case-tolerant package names
Here is a patch, meant as a contribution to the TIP #339
followup discussion. It is an alternative, providing 100%
compatibility, solving an easier problem than the original.
In stead of trying to make package handling case-insensitive,
it only tries to handle the situation that a FooBar package
is provided, and then let the core provide a wrapper package
"foobar" which can be "required" in stead of the original.
Uploaded new patch file, in which autoMkindex.test crash is fixed,
apart from that, "package names" doesn't report phony packages any
more, and a few test cases are added.
Patch crashes in autoMkindex-1.2.
Those are big and messy. Skip them.
Crash in package-compat-1.5 .
oops. not in it. following it.
I'm giving up. Wanted to give this
a shot on the assumption it was
ready to go. If not, I'd rather
take a different tack.
New attempt (there was one Tcl_EventuallyFree() too much ....,
on Windows it didn't crash.)
However, I wouldn't commit this until at least a new TIP handling
this has been accepted. It's too late for Tcl 8.6 anyway, let's
see for Tcl 8.7. I will continue to add more tests, and any
review is welcome (I still don't know what "Those are big and messy.
Skip them" means: this patch is much smaller then anything else
I have seen...... Also curious about the "different track")
Here is a new patch, against current HEAD. And fixed a problem related to
circular dependancies: Phony packages should not be counted on that.
With this version, all test cases pass!
patch against current CVS HEAD
B.T.W: I am not fully confident about this yet. It looks like the auto_mkIndex tool produces two lines for each package with this change. It would be better to modify auto_mkIndex to recognise this situation, and deliver the same output as before.