Tcl Source Code

View Ticket
Login
Ticket UUID: 1222788
Title: pkg_mkIndex produces comments only
Type: Bug Version: obsolete: 8.4.10
Submitter: pmmartin Created on: 2005-06-17 17:15:46
Subsystem: 39. Package Manager Assigned To: dgp
Priority: 5 Medium Severity:
Status: Closed Last Modified: 2005-07-05 22:37:28
Resolution: Invalid Closed By: dgp
    Closed on: 2005-07-05 15:37:28
Description:
pkgIndex.tcl file created with only comments.
See also comments by LV at http://mini.net/tcl/3388.

Error message in -verbose mode indicated "dir" was not
defined at the time of invoking "source".

An adequate workaround was to add the following line to
proc pkg_mkIndex in package.tcl, just before the large
$c eval {...} block:

$c eval [list set dir "$dir"]

This line defines "dir" in the slave interpreter's
namespace, prior to invoking the "source" command.
User Comments: dgp added on 2005-07-05 22:37:27:
Logged In: YES 
user_id=80530


the log attached indicates something
wrong with the rc.tcl file you're trying
to index.  Apparently it expects a
variable "dir" to be defined before
the file is [source]d ?

Without the rc.tcl file contents, I can't
be more specific about how to correct
things, but I don't see anything here
suggesting a bug in [pkg_mkIndex].

Your suggested fix is effective, but
misplaced.  The indexer is not the
place to put workarounds for bugs
in the code being indexed.

pmmartin added on 2005-06-18 02:07:02:

File Added - 138798: 1222788_ScenarioResults.txt

pmmartin added on 2005-06-18 02:07:01:
Logged In: YES 
user_id=319460

Concur on likelihood of pilot error.

Attached file, 1222788_ScenarioResults.txt, shows before and
after behavior.

dgp added on 2005-06-18 00:47:17:
Logged In: YES 
user_id=80530


this is probably pilot error.

To be sure, please report exactly
what arguments were passed
to [pkg_mkIndex], along with
relevant details about the file(s)
being indexed, and the reports
from -verbose.

Willing to consider improvements
to [pkg_mkIndex], but for quick
success, you're almost certainly
better off ignoring the command
completely, and just creating your
index script directly.

Attachments: