Ticket UUID: | 2432057 | |||
Title: | Another path to case-tolerant package names | |||
Type: | Patch | Version: | 8.6 | |
Submitter: | jan.nijtmans | Created on: | 2008-12-15 23:59:55 | |
Subsystem: | 39. Package Manager | Assigned To: | jan.nijtmans | |
Priority: | 5 Medium | Severity: | Minor | |
Status: | Closed | Last Modified: | 2021-03-18 14:56:52 | |
Resolution: | Out of Date | Closed By: | jan.nijtmans | |
Closed on: | 2021-03-18 14:56:52 | |||
Description: |
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. | |||
User Comments: |
jan.nijtmans added on 2021-03-18 14:56:52:
Being handled now by TIP #595, but in a totally different way. So, closing. nijtmans added on 2010-07-27 21:29:53: 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. nijtmans added on 2010-07-27 04:20:03: File Added - 381220: 2432057.patch nijtmans added on 2010-07-27 04:19:23: File Deleted - 381219: nijtmans added on 2010-07-27 03:56:52: 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! nijtmans added on 2010-07-27 03:55:11: File Added - 381219: 2432057.patch nijtmans added on 2010-07-27 03:54:29: File Deleted - 305586: nijtmans added on 2008-12-17 06:03:10: 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") nijtmans added on 2008-12-17 05:57:32: File Added - 305586: tclPkg.c.diff nijtmans added on 2008-12-17 05:56:55: File Deleted - 305500: dgp added on 2008-12-17 01:48:51: 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. dgp added on 2008-12-17 01:46:57: Those are big and messy. Skip them. Crash in package-compat-1.5 . dgp added on 2008-12-17 01:34:43: Patch crashes in autoMkindex-1.2. nijtmans added on 2008-12-16 19:28:29: 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. nijtmans added on 2008-12-16 19:27:13: File Added - 305500: tclPkg.c.patch nijtmans added on 2008-12-16 19:26:31: File Deleted - 305450: nijtmans added on 2008-12-16 06:59:56: File Added - 305450: tclPkg.c.patch |
Attachments:
- 2432057.patch [download] added by nijtmans on 2010-07-27 04:20:03. [details]