Skip to content

doc: add example on how to add EXTRA_QT_PLUGINS - #141

Merged
TheAssassin merged 2 commits into
linuxdeploy:masterfrom
pbek:patch-1
Apr 9, 2023
Merged

TheAssassin merged 2 commits into
linuxdeploy:masterfrom
pbek:patch-1

Conversation

@pbek

@pbek pbek commented Apr 8, 2023

Copy link
Copy Markdown
Contributor

No description provided.

@pbek

pbek commented Apr 8, 2023

Copy link
Copy Markdown
Contributor Author

...in regard of #140 (comment).

Comment thread README.md Outdated
**Qt specific:**
- `$QMAKE=/path/to/my/qmake`: use another `qmake` binary to detect paths of plugins and other resources (usually doesn't need to be set manually, most Qt environments ship scripts changing `$PATH`)
- `$EXTRA_QT_PLUGINS=pluginA;pluginB`: Plugins to deploy even if not found automatically by linuxdeploy-plugin-qt
- example: `$EXTRA_QT_PLUGINS=svg;` if you are using SVG icons

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You put it too simple. It needs to basically say "use svg if you want to use the module QtSvg".

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd would want to keep the EXTRA_QT_PLUGINS=svg; part to make it perfect clear what to put in your build script for the impatient 😉

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We could combine both, right? You can combine a short demo with an elaborate explanation on how this actually works.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Like my last commit?

@TheAssassin
TheAssassin merged commit cc77019 into linuxdeploy:master Apr 9, 2023
@TheAssassin

Copy link
Copy Markdown
Member

Perfect, thanks!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants