New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Make Firebird distribution on Windows easier [CORE2243] #2670
Comments
Modified by: @hvladassignee: Vlad Khorsun [ hvlad ] |
Modified by: @hvladstatus: Open [ 1 ] => Open [ 1 ] Target: 2.1.2, 2.5 Beta 1 [ 10270, 10251 ] Fix Version: 2.1.2 [ 10270 ] Fix Version: 2.5 Beta 1 [ 10251 ] Planning Status: Unspecified => Considered for inclusion Development: Finished |
Commented by: Sean Leyne (seanleyne) I strongly disagree with this solution. There are 100000's of application which now need to install the MSVC 8 runtime package, and they simply have added it to their distribution and to the install package. 1 - Why do we need to fight a battle that countless of other have given up on? 2 - We do not know how MS will change their policy in the future, accordingly, we should not (as we would in the proposed solution) make assumptions about how MS will change the process oe potentially invalidate existing installs. |
Commented by: @hvlad It is too late to agree or disagree with it. This improvement was discussed and agreed in both fb-devel and fb-admins. 1. Ask users. Especially users of embedded package 2. This code currently active for MSVC8 run-time *only* |
Commented by: Miha Vrhovnik (mvrhov) Vlad on embedded This could be solved by statically linking everything into fbembed.dll. AFAIR there is ticket for that. all I can say is I tried it. This has at least two benefits: smaller distribution for at least 1,5MiB and easier distribution only 1 dll. Also the docs on how easy it is to distribute fb embedded would once again come true. |
Commented by: @hvlad Static linking was also discussed. We don't want to make our complex build process even more complex. I can add that we will reconsider build system in v3.0 |
Commented by: Miha Vrhovnik (mvrhov) Everything that I was unable to statically link was intl.dll. I should probably start a thread in dev mailing list with this and then we can put together some sort of HOW TO for others. |
Modified by: @pcisarstatus: Resolved [ 5 ] => Closed [ 6 ] |
Modified by: @pavel-zotovQA Status: No test |
Modified by: @pavel-zotovstatus: Closed [ 6 ] => Closed [ 6 ] QA Status: No test => Cannot be tested |
Submitted by: @hvlad
Since Firebird 2.1 we use MSVC 8 (VS2005) to build it on Windows. It creates dependency from MSVC8 run-time. If user decide to distribute MSVC8 run-time as private assembly he should put all its files in \bin, \udf and \intl folders to allow dynamic loading of required DLL's.
The improvement is to make possible distribution of just one copy of MSVC8 run-time in \bin folder only
Commits: c3fdb1b 83ec221 c68f941 685acf8
The text was updated successfully, but these errors were encountered: