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. |