Skip to content
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

Closed
firebird-automations opened this issue Dec 15, 2008 · 11 comments
Closed

Make Firebird distribution on Windows easier [CORE2243] #2670

firebird-automations opened this issue Dec 15, 2008 · 11 comments

Comments

@firebird-automations
Copy link
Collaborator

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

@firebird-automations
Copy link
Collaborator Author

Modified by: @hvlad

assignee: Vlad Khorsun [ hvlad ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @hvlad

status: 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

@firebird-automations
Copy link
Collaborator Author

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.

@firebird-automations
Copy link
Collaborator Author

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*

@firebird-automations
Copy link
Collaborator Author

Modified by: @hvlad

status: Open [ 1 ] => Resolved [ 5 ]

resolution: Fixed [ 1 ]

@firebird-automations
Copy link
Collaborator Author

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.

@firebird-automations
Copy link
Collaborator Author

Commented by: @hvlad

Static linking was also discussed. We don't want to make our complex build process even more complex.
As you able to build statically linked embedded you always can do it for yourself, isn't is ?

I can add that we will reconsider build system in v3.0

@firebird-automations
Copy link
Collaborator Author

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.

@firebird-automations
Copy link
Collaborator Author

Modified by: @pcisar

status: Resolved [ 5 ] => Closed [ 6 ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @pavel-zotov

QA Status: No test

@firebird-automations
Copy link
Collaborator Author

Modified by: @pavel-zotov

status: Closed [ 6 ] => Closed [ 6 ]

QA Status: No test => Cannot be tested

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment