Tcl Source Code

View Ticket
Login
Ticket UUID: 3543020
Title: Thread extension not building with makefile. when in release
Type: Bug Version: obsolete: 8.6b3
Submitter: decosterjos Created on: 2012-07-12 14:02:11
Subsystem: 80. Thread Package Assigned To: nijtmans
Priority: 9 Immediate Severity:
Status: Closed Last Modified: 2012-07-18 13:52:32
Resolution: Fixed Closed By: nijtmans
    Closed on: 2012-07-18 06:52:32
Description:
The Threads package has a makefile.vc to build with Visual Studio in directory win/vc. When adding the Thread package to the pkgs directory for a 'batteries included' Tcl release, the makefile.vc file is moved from directory win/vc to directory win. This invalidates the ROOT variable used in the makefile.vc. It still has value ..\.. where after copying it should become ..
User Comments: nijtmans added on 2012-07-18 13:52:32:

allow_comments - 1

Thanks for testing! Closing

decosterjos added on 2012-07-18 13:34:53:
OK now for Tcl and for Itcl, Thread and Tdbc in the pkgs dir. Thanks.

nijtmans added on 2012-07-17 16:16:00:
Fixed in trunk. OK now?

decosterjos added on 2012-07-17 13:54:57:
I attached a patch to 'Makefile.in' which updates the changed windows patch in 'make dist'

decosterjos added on 2012-07-17 13:54:13:

File Added - 449112: Makefile.in.patch

nijtmans added on 2012-07-17 02:18:52:
Done as suggested by Jos. OK?

Thanks, Jos!

decosterjos added on 2012-07-16 21:54:30:
The attached file 'vc.diff' is the 'fossil diff' output after moving the files from the win.vc to the win directory and updating the README file.

decosterjos added on 2012-07-16 21:51:14:

File Added - 449050: vc.diff

decosterjos added on 2012-07-16 21:01:51:
To build Thread in a 'Betteries' included' Tcl, under the pkgs directories, one of the following needs to be done:

- put all files of tcl/pkgs/thread/win/vc in thread/win and adapt the ROOT in the thread makefile.vc 

or

- modify tcl/win/makefile.vc to also look in pkgs/???/win/vc directories

I prefer to remove the thread/win/vc directory.

vasiljevic added on 2012-07-13 00:57:16:
Don, I have, as already stated, no experience with Windows build.
I must say I did not understand your answer :-(
Do I need to do anything? I normally fresh-clone the distro, make 
changes, commit and push.

dgp added on 2012-07-13 00:44:57:
This was corrected before in checkin 6593e21b9c.

It got broken again in checkin 51cf545767.

There's a natural tug-o-war going on between making
builds work in a fossil checkout and making them work
in the released distribution.  Unless there's some value
in having those two keep different directory layouts, I
recommend getting rid of the vc subdir from the very beginning.

Otherwise we'll just keep fixing this again and again.

Attachments: