Tcl Source Code

View Ticket
Login
Ticket UUID: 1575837
Title: "make test" failures with 8.5a5
Type: Bug Version: obsolete: 8.5a5
Submitter: afredd Created on: 2006-10-12 11:09:39
Subsystem: 37. File System Assigned To: dgp
Priority: 9 Immediate Severity:
Status: Closed Last Modified: 2006-10-28 09:20:06
Resolution: Fixed Closed By: sf-robot
    Closed on: 2006-10-28 02:20:06
Description:
Just ran a "nmake test" of the just released
8.5a5 build on XP using VS6. The compiler gave some
warnings which look harmless enough (but you
might be interested):

eg.
        cl -nologo -c  -QI0f -D 
_CRT_SECURE_NO_DEPRECATE -D _CRT_NONSTDC_NO_DEPRECATE -
Fp.
\Release\tcl_Dynamic\ -W3 -O2  -Ot -Oi -Op -fp:strict -
Gs -GS -GL -YX -MD -I"..\win" -I"..
\generic" -I"..\libtommath"  -
DTCL_PIPE_DLL=\"tclpip.dll\" -DTCL_TOMMATH  -
DMP_PREC=4 -DTC
L_CFGVAL_ENCODING=\"cp1252\" -DTCL_CFG_OPTIMIZED -
Fo.\Release\tcl_Dynamic\tclTest.obj ..\g
eneric\tclTest.c
Command line warning D4002 : ignoring unknown option '-
fp:strict'
Command line warning D4002 : ignoring unknown option '-
GS'
Command line warning D4002 : ignoring unknown option '-
GL'

... and the linker also:

        link -nologo -machine:IX86 -ltcg -release -
opt:ref -opt:icf,3 -opt:nowin98 -subsys
tem:console -out:.\Release\cat32.exe -
stack:16384 .\Release\tcl_Dynamic\cat.obj  kernel32.
lib user32.lib
LINK : warning LNK4044: unrecognized option "ltcg"; 
ignored


Here are the results of the tests that failed (all
in fCmd):

==== fCmd-28.9 file link: success with file FAILED
==== Contents of test case:

    cd [temporaryDirectory]
    file delete -force abc.link
    set res [list [catch {file link abc.link abc.file} 
msg] $msg]
    cd [workingDirectory]
    set res

---- Result was:
0 D:/afredd/src/tcl/tcl8.5a5rc1-
src.tar/tcl8.5a5/win/abc.file
---- Result should have been (exact matching):
0 abc.file
==== fCmd-28.9 FAILED



==== fCmd-28.9.1 file link: success with file FAILED
==== Contents of test case:

    cd [temporaryDirectory]
    file delete -force abc.link
    set res {}
    file stat abc.file arr ; lappend res $arr(nlink)
    lappend res [catch {file link abc.link abc.file} 
msg] $msg
    file stat abc.file arr ; lappend res $arr(nlink)
    cd [workingDirectory]
    set res

---- Result was:
1 0 D:/afredd/src/tcl/tcl8.5a5rc1-
src.tar/tcl8.5a5/win/abc.file 2
---- Result should have been (exact matching):
1 0 abc.file 2
==== fCmd-28.9.1 FAILED



==== fCmd-28.11 file link: success with directory 
FAILED
==== Contents of test case:

    cd [temporaryDirectory]
    file delete -force abc.link
    set res [list [catch {file link abc.link abc.dir} 
msg] $msg]
    cd [workingDirectory]
    set res

---- Result was:
0 D:/afredd/src/tcl/tcl8.5a5rc1-
src.tar/tcl8.5a5/win/abc.dir
---- Result should have been (exact matching):
0 abc.dir
==== fCmd-28.11 FAILED



==== fCmd-28.19 file link: relative paths FAILED
==== Contents of test case:

    cd [temporaryDirectory]
    file mkdir d1/d2/d3
    set res [list [catch {file link d1/l2 d1/d2} err] 
$err]
    lappend res [catch {file delete -force d1} err] 
$err

---- Result was:
0 D:/afredd/src/tcl/tcl8.5a5rc1-
src.tar/tcl8.5a5/win/d1/d2 0 {}
---- Result should have been (exact matching):
0 d1/d2 0 {}
==== fCmd-28.19 FAILED
User Comments: sf-robot added on 2006-10-28 09:20:06:
Logged In: YES 
user_id=1312539

This Tracker item was closed automatically by the system. It was
previously set to a Pending status, and the original submitter
did not respond within 14 days (the time period specified by
the administrator of this Tracker).

coldstore added on 2006-10-13 20:00:12:
Logged In: YES 
user_id=19214

Applied afredd-supplied patches, with thanks.

afredd added on 2006-10-13 16:41:21:

File Added - 198002: tclWinFile.c189-patch

Logged In: YES 
user_id=1386588


Attached patch passes tests.

Needed to avoid reassiging toPtr, as it used as
the return value.

afredd added on 2006-10-13 15:47:05:
Logged In: YES 
user_id=1386588


The change 1.89 of tclWinFile.c does _not_ appear
to fix the problem. I still get the same 4 fails:
fCmd-28.9, fCmd-28.9.1, fCmd-28.11 & fCmd-28.19.

coldstore added on 2006-10-13 12:27:43:
Logged In: YES 
user_id=19214

I misinterpreted the windows [file link] code while trying
to protect against a potential NULL return error.

I've reworked the fix.  Should work now.  Please test (I
don't have Windows.)

dgp added on 2006-10-13 00:34:55:
Logged In: YES 
user_id=80530


Best guess is that the
2006-08-29 commit from
coldstore is to blame.
Please review.

dgp added on 2006-10-13 00:20:43:
Logged In: YES 
user_id=80530

ok, so sometime since
8.5a4, [file link] on
Windows has started returning
only absolute paths.  Gotta 
locate the source of that
regression.

afredd added on 2006-10-12 22:59:54:
Logged In: YES 
user_id=1386588

>> Are any of these failures also seen in the 8.4.14rc1?
>
>All test passed for me with that build.

fwiw - just tried 8.5a4, and this too passed all tests.

afredd added on 2006-10-12 21:36:01:
Logged In: YES 
user_id=1386588


> Are any of these failures also seen in the 8.4.14rc1?

All test passed for me with that build.

NB. There is no fCmd-28.9.1 or fCmd-28.19 in
8.4.14's fCmd.test.

dgp added on 2006-10-12 21:25:42:
Logged In: YES 
user_id=80530


Are any of these failures also
seen in the 8.4.14rc1 ?

Attachments: