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.

Sign up for GitHub

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

Jump to bottom

windows sdk_install.bat file generated after compile seems to incorrectly generate dll file copy command #557

Closed
junaid-ebrahim opened this issue Dec 23, 2015 · 4 comments
Closed

windows sdk_install.bat file generated after compile seems to incorrectly generate dll file copy command #557

junaid-ebrahim opened this issue Dec 23, 2015 · 4 comments

Comments

@junaid-ebrahim
Copy link

I see line 94 of install_sdk.pro , when changed to the below seems to produce the correct dll copy command

win32:sdk_install.commands = $$quote($$COPY $$system_path($$BUILD_DIR/bin/Qt_AV_.dll) $$system_path($$[QT_INSTALL_BINS]/))

Note the * added between the Qt and AV
@junaid-ebrahim
Copy link
Author

sorry, the asterisks got converted to markdown, not sure how to escape the like correctly for this issue reporting system, anyway , the "AV" in italics should have a asterisk before and after

@wang-bin
Copy link
Owner

No. On windows only QtAV1.dll and QtAVWidgets1.dll are created. What's your Qt version? Maybe Qt5.4 will create Qt5AV1.dll.

@junaid-ebrahim
Copy link
Author

Yes, I am using Qt 5.4.1 msvc2013 edition, the dll created is Qt5AV1.dll and Qt5AVd1.dll for debug versions. They are missed during the install_sdk run because of the naming convention

wang-bin added a commit that referenced this issue Dec 26, 2015
@wang-bin
deploy: fix dll copy for 5.4 #557
f73e538 
fucking qt bug
@wang-bin
Copy link
Owner

It's a qt bug. Anyway, the latest code should work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@wang-bin @junaid-ebrahim