| Ticket UUID: | 3335120320518c9535503985c5278f328a833880 | ||
| Title: | makefile.vc: package with include folder requires this folder to exist | ||
| Type: | Bug | Created on: | 2025-05-09 15:35:13 |
| Submitter: | oehhar | Assigned to: | nobody |
| Subsystem: | 53. Configuration and Build Tools | Severity: | Minor |
| Priority: | 5 Medium | Last modified: | 2025-05-20 15:54:22 |
| Status: | Closed | Closed by: | oehhar |
| Resolution: | Fixed | Closed on: | 2025-05-20 15:54:22 |
| Version: | 9.0.1 | ||
| Description: | ||||
|
VS2022 64 bit prompt on Win11 x64 OS: A compiled and installed TCL 9.0.1 is in c:\myprograms\tcl9.0_x64. A checkout of tdbc-main is in C:\test\fossil\tdbc\tdbc-main. An VS2022 64 bit native prompt gets the following commands: % nmake -f makefile.vc TCLDIR=c:\myprograms\tcl9.0_x64 % nmake -f makefile.vc install TCLDIR=c:\myprograms\tcl9.0_x64 INSTALLDIR=..\..\install_x64 ... Installing headers to '..\..\install_x64\..\include' When looking in the install folder (e.g. "..\.."), there is:
It might be helpful, to create the folder '..\..\install_x64\..\include', if it does not exist. The relevant entries in makefile.vc are: PRJ_HEADERS_PUBLIC = \ $(GENERICDIR)\tdbc.h \ $(GENERICDIR)\tdbcDecls.h I am reporting this here, as IMHO, the machinery for this is here and not in tdbc-main. Thanks for all, Harald | ||||
| User Comments: | ||||
apnadkarni added on 2025-05-18 15:53:58:
Proposed fix in branch apn-bug-3335120320. apnadkarni added on 2025-05-20 05:28:00:
Fixed in [761808a5e8]. Harald, please verify for your case. oehhar added on 2025-05-20 15:54:22:
Thanks for the fix, it works. What I still find interesting is, that the folder "include" is created beside the given INSTALLDIR, not in the installdir. It is also interesting, that the mostly useless documentation files (*.n) are written into the installation folder. Also, tdbcstub.lib and tdbcConfig.sh are written into the installation folder, which are IMHO expected in lib. So, anything what is normally in doc/lib goes into the installation folder, with the exception of the include folder. I suppose, this is done to work as bundled package. Anyway, great fix. Thanks for all, Harald | ||||