QuasarApp / CQtDeployer

This project is used to deploy applications written using QML, qt or other С / С++ frameworks.
https://quasarapp.ddns.net:3031/docs/QuasarApp/CQtDeployer/latest/
GNU Lesser General Public License v3.0
576 stars 87 forks source link

Add description of the bin option. #520

Closed EndrII closed 3 years ago

EndrII commented 3 years ago

why it so happened that the required library to run is not indicated in the list of dependencies.

It might be a run-time dynamic linking library used by dlopen , LoadLibrary or something equivalent.

Before seeing this post, I had problem with the OpenSSL library. I had to manually copy "libcrypto-1_1-x64.dll" and "libssl-1_1-x64.dll" to the target directory every time. For now, I can use the option "-bin" and "-libDir" like this:

> cqt -bin 'path\to\libssl-1_1-x64.dll,myProgram.exe' -libDir 'path\to\the\parent\dir\of\libcrypto-1_1-x64.dll'

The library "libssl-1_1-x64.dll" depends on "libcrypto-1_1-x64.dll", so CQtDeployer will copy them together to the target directory. Nice!

However, I never thought I could use the option "-bin" like this and I had no idea why a list can be its value before. This part of the document needs to be improved, in my opinion.


And one more thing, the doc said:

-extraLibs [list,params] Sets the mask of the library name for forced copying. Example: "-extraLib mySql" - forces to copy all libraries whose names contain mySql to the project folder. This option is case sensitive.

@EndrII explained further:

As for the extraLibs option, this option is intended for forced copying of dependencies from the system environment similar to deploySystem. https://github.com/QuasarApp/CQtDeployer/issues/394#issuecomment-656678601

It would be better to add the above explanation to the document.

Originally posted by @pzhlkj6612 in https://github.com/QuasarApp/CQtDeployer/issues/394#issuecomment-774440191

pzhlkj6612 commented 3 years ago

@EndrII Hi, don't forget -extraLibs . The title needs to be changed.

EndrII commented 3 years ago

yes. thanks

EndrII commented 3 years ago

@pzhlkj6612 check the #521 PR.