using embedded resources
Bug #1768932 reported by
djcj
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
qpdfview |
Fix Released
|
Wishlist
|
Adam Reichold |
Bug Description
The html help files and compiled localizations can be read from Qt's embedded resources system. But that requires adding ":/" as prefix manually. I've attached a patch that does this automatically if qpdfview is build with -DSTATIC_RES, otherwise it'll look for external resource files like it already does.
This allows me to cross-compile a single static binary for Windows.
Changed in qpdfview: | |
status: | Incomplete → Fix Committed |
Changed in qpdfview: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
Hello djcj,
thank for your contribution! I would like to request that you make two changes to this patch:
* Instead of defining STATIC_RES and deriving RESPFX from that, just define RESOURCE_PREFIX to be empty by default and ":/" for embedded resources.
* The change seems incomplete? Usually I would like to do "qmake CONFIG+ =static_ resources" which does actually include an RCC manifest to really package these files as embedded resources and sets the necessary preprocessor definitions like RESOURCE_PREFIX.
Best regards,
Adam.
P.S.: Please consider using Launchpad's merge request feature to submit more complex changes as it greatly simplifies review and discussion and avoids having to open a bug report for each change even if it does not actually fix a bug.