Tcl Source Code

View Ticket
Login
Ticket UUID: 1c6f6503d4fb512d4d3d6c9cd1cfa50d34c5308c
Title: Fix auto_execok on Windows
Type: Patch Version: 8.6.5
Submitter: danckaert Created on: 2016-03-09 16:26:57
Subsystem: 38. Init - Library - Autoload Assigned To: jan.nijtmans
Priority: 5 Medium Severity: Minor
Status: Open Last Modified: 2017-05-24 08:40:32
Resolution: None Closed By: nobody
    Closed on:
Description:

There are several problems with auto_execok on Windows. See for example these tickets: 4ab2ccc4a2, 1244733, e21fc32c2a. Another issue is that the search path constructed by [auto_execok] is quite ad-hoc, while the path used by [exec] may be dependent on the OS version and even on registry values, as documented on http://msdn.microsoft.com/en-us/library/windows/desktop/aa365527(v=vs.85).aspx.

All this can be solved by aligning the search procedure of [auto_execok] with [exec]. To do this, a small helper proc is needed to do call the appropriate C functions from [auto_execok].

I'm attaching a patch to do this. Note that I didn't know where precisely to declare and create the helper functions, so this may require some review.

User Comments: jan.nijtmans added on 2017-05-24 08:40:32:
I like this solution! Not enough severity to rush it into 8.6.7, but I'll have a closer look soon.

danckaert added on 2017-05-24 07:37:51:

I've replaced the patch with an updated version. Previous one did not always return a list.


Attachments: